Skip to content

fix(rpc): support Accept parameters and media ranges - #343

Open
huklaa wants to merge 1 commit into
circlefin:mainfrom
huklaa:fix-341-accept-header-negotiation
Open

fix(rpc): support Accept parameters and media ranges#343
huklaa wants to merge 1 commit into
circlefin:mainfrom
huklaa:fix-341-accept-header-negotiation

Conversation

@huklaa

@huklaa huklaa commented Sep 4, 2026

Copy link
Copy Markdown

Fixes #341.

Summary

  • parse comma-separated "Accept" media ranges instead of treating the whole header as one media type
  • ignore supported ranges with "q=0"
  • keep "application/json" and "/" as default-version fallbacks
  • add regression coverage for parameters, multiple ranges, zero quality, malformed quality values, and unsupported versions

Testing

Focused parser regression tests were added under "crates/malachite-app/src/rpc/version.rs".

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.

API version negotiation should support Accept parameters and multiple media ranges

1 participant