Skip to content

Add support for AES-192-GCM and AES-256-GCM#7

Merged
BenCookie95 merged 2 commits into
masterfrom
aes-gcm
May 12, 2025
Merged

Add support for AES-192-GCM and AES-256-GCM#7
BenCookie95 merged 2 commits into
masterfrom
aes-gcm

Conversation

@BenCookie95

Copy link
Copy Markdown

Summary

Cherry picked from commit 342698e of the source repo.

Before the change I can reproduce the error in the ticket:

{"timestamp":"2025-05-09 13:24:00.416 -04:00","level":"error","msg":"An error occurred while parsing the response from the Identity Provider. Please contact your System Administrator.","caller":"web/context.go:123","path":"/login/sso/saml","request_id":"cy6ji9zg17gfjq4mjfd5yjk14h","ip_addr":"::1","user_id":"","method":"POST","err_where":"SamlInterfaceImpl.ValidateResponse","http_code":302,"error":"SamlInterfaceImpl.ValidateResponse: An error occurred while parsing the response from the Identity Provider. Please contact your System Administrator., error validating response: unable to decrypt encrypted assertion: unknown symmetric encryption method \"http://www.w3.org/2009/xmlenc11#aes256-gcm\""}

With the change I can login successfully.

Ticket Link

https://mattermost.atlassian.net/browse/MM-64222

@BenCookie95

Copy link
Copy Markdown
Author

This repo hasn't been updated in a while, it's on an old GO version and it's quite a few commits behind the source repo. We probably need a separate ticket to get this repo up to date.

@hanzei hanzei added the 2: Dev Review Requires review by a core committer label May 12, 2025
@BenCookie95 BenCookie95 merged commit f77801b into master May 12, 2025
4 checks passed
@BenCookie95 BenCookie95 deleted the aes-gcm branch May 12, 2025 16:09
@hanzei hanzei added 3: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3: Reviews Complete All reviewers have approved the pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants