Skip to content

chore: return method not supported via Gateway when /v2 is absent#13121

Merged
masih merged 2 commits intomasterfrom
masih/graceful-v2-gateway-not-supported
May 11, 2025
Merged

chore: return method not supported via Gateway when /v2 is absent#13121
masih merged 2 commits intomasterfrom
masih/graceful-v2-gateway-not-supported

Conversation

@masih
Copy link
Member

@masih masih commented May 10, 2025

When there is no /v2 api offered by the full-node gracefully fallback to returning method not supported via Gateway.

Fixes #13119

When there is no /v2 api offered by the full-node gracefully fallback
to returning `method not supported` via Gateway.
@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz May 10, 2025
@masih masih requested a review from rvagg May 10, 2025 21:14
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@masih masih changed the title Return method not supported via Gateway when /v2 is absent chore: return method not supported via Gateway when /v2 is absent May 10, 2025
@github-project-automation github-project-automation bot moved this from 📌 Triage to ⌨️ In Progress in FilOz May 10, 2025
@masih masih added the skip/changelog This change does not require CHANGELOG.md update label May 10, 2025
@github-actions github-actions bot dismissed their stale review May 10, 2025 21:15

PR title now matches the required format.

Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested, works well, just needs the log fix and maybe this should also get a CHANGELOG entry so we can ship a release that may have only this fix (can have more, but if we did just this change then it should say something)

@github-project-automation github-project-automation bot moved this from ⌨️ In Progress to ✔️ Approved by reviewer in FilOz May 11, 2025
@masih masih removed the skip/changelog This change does not require CHANGELOG.md update label May 11, 2025
@masih masih enabled auto-merge (squash) May 11, 2025 14:59
@masih masih merged commit c3b48f5 into master May 11, 2025
100 checks passed
@masih masih deleted the masih/graceful-v2-gateway-not-supported branch May 11, 2025 15:02
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz May 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

Unversioned FULLNODE_API_INFO on a client resolving to a v2 endpoint

3 participants