Skip to content

fix: Earn.fm Supabase auth, exchange rates redirect, CSP#72

Merged
GeiserX merged 1 commit into
mainfrom
fix/earnfm-exchange-csp
May 23, 2026
Merged

fix: Earn.fm Supabase auth, exchange rates redirect, CSP#72
GeiserX merged 1 commit into
mainfrom
fix/earnfm-exchange-csp

Conversation

@GeiserX
Copy link
Copy Markdown
Owner

@GeiserX GeiserX commented May 23, 2026

Summary

  • Earn.fm collector rewritten: UUID token was for Docker client only, not API access. Now uses Supabase email/password auth → verified working ($5.71 balance)
  • Exchange rates fixed: Frankfurter API returns 301 but httpx wasn't following redirects → only USD was available in currency selector
  • CSP fix: Chart.js source map request to cdn.jsdelivr.net was blocked by connect-src 'self'

Test plan

- Rewrite Earn.fm collector to use Supabase email/password auth instead
  of the broken UUID token (which is only for the Docker bandwidth client)
- Fix exchange rates: Frankfurter API returns 301 but httpx wasn't
  following redirects, causing empty fiat rates (only USD available)
- Fix CSP: allow connect-src to cdn.jsdelivr.net for Chart.js source maps
@GeiserX GeiserX merged commit bd8e8fd into main May 23, 2026
3 of 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 8 minutes and 41 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: fce447dc-2557-40f2-9c94-64be8a4ca1b6

📥 Commits

Reviewing files that changed from the base of the PR and between 5e0c0a8 and 86bdd49.

📒 Files selected for processing (5)
  • app/collectors/__init__.py
  • app/collectors/earnfm.py
  • app/exchange_rates.py
  • app/main.py
  • services/bandwidth/earnfm.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/earnfm-exchange-csp

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/earnfm-exchange-csp branch May 23, 2026 20:02
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