Skip to content

fix: send Photon requests to the Helius RPC root - #2390

Open
sergeytimoshin wants to merge 1 commit into
mainfrom
fix/photon-root-json-rpc
Open

fix: send Photon requests to the Helius RPC root#2390
sergeytimoshin wants to merge 1 commit into
mainfrom
fix/photon-root-json-rpc

Conversation

@sergeytimoshin

Copy link
Copy Markdown
Contributor

Helius Gatekeeper routes Photon calls through JSON-RPC at the RPC root URL. The Rust client appended method paths such as /getIndexerHealth, preventing clients from using https://mainnet.helius-rpc.com/?api-key=....

Send all Rust apis::default_api calls to the configured URL, with the method in the JSON body. Preserve custom endpoint paths, other query parameters, and encoded API keys. The TypeScript client already posts to the root URL; add regression coverage and document mainnet configuration.

Validation:

  • cargo test -p photon-api --locked: 14 tests passed.
  • TypeScript vitest run tests/unit/rpc/json-rpc-transport.test.ts: 15 tests passed.
  • Live read-only requests to mainnet.helius-rpc.com/ succeeded for getIndexerSlot and getCompressedAccount.
  • git diff --check passed.

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: d685b4c9-d506-46d2-8f93-bfcd821645d4

📥 Commits

Reviewing files that changed from the base of the PR and between ad5964f and dd9e55b.

⛔ Files ignored due to path filters (3)
  • js/stateless.js/README.md is excluded by none and included by none
  • js/stateless.js/src/rpc.ts is excluded by none and included by none
  • js/stateless.js/tests/unit/rpc/json-rpc-transport.test.ts is excluded by none and included by none
📒 Files selected for processing (2)
  • sdk-libs/photon-api/CHANGELOG.md
  • sdk-libs/photon-api/src/lib.rs

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.

@sergeytimoshin sergeytimoshin changed the title fix(photon): use root JSON-RPC requests for Helius Gatekeeper fix: send Photon requests to the Helius RPC root Sep 8, 2026
@sergeytimoshin
sergeytimoshin marked this pull request as ready for review September 8, 2026 12:11
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-08T12:15:48.018257Z dd9e55b Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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