Skip to content

fix: Bytelixir collector rejects valid session at /#71

Merged
GeiserX merged 1 commit into
mainfrom
fix/bytelixir-path-check
May 23, 2026
Merged

fix: Bytelixir collector rejects valid session at /#71
GeiserX merged 1 commit into
mainfrom
fix/bytelixir-path-check

Conversation

@GeiserX
Copy link
Copy Markdown
Owner

@GeiserX GeiserX commented May 23, 2026

Summary

  • The session expiry check used an allowlist of paths (/en, /es, /dashboard, /home) but the authenticated dashboard actually stays at /
  • Changed to only detect expiry when redirected to /login
  • Tested: cookie returns 200 with balance $0.87770 — parsing works correctly

Test plan

  • Verified inside container: cookie fetches dashboard, balance parses to 0.87770
  • After deploy, Bytelixir should show "connected" in CashPilot UI

…at /

The session expiry check was using an allowlist of paths (/en, /es,
/dashboard, /home) but the authenticated dashboard actually stays at /.
Changed to only reject if redirected to /login, which is the actual
indicator of an expired session.
@GeiserX GeiserX merged commit 5e0c0a8 into main May 23, 2026
4 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 23, 2026

Warning

Review limit reached

@GeiserX, we couldn't start this review because you've used your available PR reviews for now.

Your plan currently allows 1 review/hour. Refill in 19 minutes and 10 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more review capacity refills, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d3ec8153-2003-4d7d-a368-a8059efdc443

📥 Commits

Reviewing files that changed from the base of the PR and between 4603fe7 and 1a37f0b.

📒 Files selected for processing (1)
  • app/collectors/bytelixir.py
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bytelixir-path-check

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@GeiserX GeiserX deleted the fix/bytelixir-path-check branch May 23, 2026 19:51
@codecov
Copy link
Copy Markdown

codecov Bot commented May 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.10%. Comparing base (4603fe7) to head (1a37f0b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #71   +/-   ##
=======================================
  Coverage   91.10%   91.10%           
=======================================
  Files          25       25           
  Lines        2733     2733           
=======================================
  Hits         2490     2490           
  Misses        243      243           
Files with missing lines Coverage Δ
app/collectors/bytelixir.py 79.72% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant