Skip to content

Fix SignatureParser accepting duplicate parameters in HTTP Signature header#37375

Merged
ClearlyClaire merged 5 commits intomastodon:mainfrom
shleeable:patch-38
Jan 8, 2026
Merged

Fix SignatureParser accepting duplicate parameters in HTTP Signature header#37375
ClearlyClaire merged 5 commits intomastodon:mainfrom
shleeable:patch-38

Conversation

@shleeable
Copy link
Contributor

@shleeable shleeable commented Jan 2, 2026

Detected by SCA, fix by myself. please validate.

From [RFC 7235, Section 2.1](https://datatracker.ietf.org/doc/html/rfc7235#section-2.1): Authentication parameters are name=value pairs, where the name token is matched case-insensitively, and each parameter name MUST only occur once per challenge.

ref:

raise Mastodon::SignatureVerificationError, 'Error parsing signature parameters'

@ClearlyClaire ClearlyClaire changed the title SignatureParser accepts duplicate parameters in HTTP Signature header Fix SignatureParser accepting duplicate parameters in HTTP Signature header Jan 5, 2026
@ClearlyClaire ClearlyClaire added the to backport PR needed to be backported label Jan 8, 2026
@ClearlyClaire ClearlyClaire added this pull request to the merge queue Jan 8, 2026
Merged via the queue into mastodon:main with commit e0cca5c Jan 8, 2026
35 checks passed
@shleeable shleeable deleted the patch-38 branch January 8, 2026 11:23
ClearlyClaire added a commit that referenced this pull request Jan 16, 2026
ClearlyClaire added a commit that referenced this pull request Jan 16, 2026
ClearlyClaire added a commit that referenced this pull request Jan 16, 2026
ClearlyClaire added a commit that referenced this pull request Jan 19, 2026
ClearlyClaire added a commit that referenced this pull request Jan 19, 2026
ClearlyClaire added a commit that referenced this pull request Jan 19, 2026
@ClearlyClaire ClearlyClaire removed the to backport PR needed to be backported label Jan 19, 2026
mistydemeo pushed a commit to mistydemeo/mastodon that referenced this pull request Jan 20, 2026
kmycode pushed a commit to kmycode/mastodon that referenced this pull request Jan 20, 2026
mimikun pushed a commit to mimikun/mastodon that referenced this pull request Jan 23, 2026
mimikun pushed a commit to mimikun/mastodon that referenced this pull request Jan 23, 2026
Ember-ruby pushed a commit to Ember-ruby/mastodon-glitch that referenced this pull request Jan 24, 2026
mo-rijndael pushed a commit to mastodon-ml/mastodon that referenced this pull request Jan 25, 2026
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.

2 participants

Comments