Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add domain separation for originator signatures #235

Merged

Conversation

neekolas
Copy link
Contributor

No description provided.

Copy link
Contributor Author

neekolas commented Oct 16, 2024

Copy link

graphite-app bot commented Oct 16, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “Queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “Hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@neekolas neekolas marked this pull request as ready for review October 16, 2024 23:46
@neekolas neekolas requested a review from a team as a code owner October 16, 2024 23:46
PAYER_DOMAIN_SEPARATION_LABEL = "payer|"
JWT_DOMAIN_SEPARATION_LABEL = "jwt|"
PAYER_DOMAIN_SEPARATION_LABEL = "payer|"
ORIGINATOR_DOMAIN_SEPARATION_LABEL = "originator|"
Copy link
Contributor

Choose a reason for hiding this comment

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

Same Q about making this shorter!

@neekolas neekolas force-pushed the 10-16-recover_signer_from_client_envelopes branch from 1244b3c to 1e9a07a Compare October 17, 2024 23:05
@neekolas neekolas force-pushed the 10-16-add_domain_separation_for_originator_signatures branch from 9657d8c to 5a4631f Compare October 17, 2024 23:05
Copy link
Contributor Author

neekolas commented Oct 18, 2024

Merge activity

  • Oct 17, 5:18 PM PDT: A user started a stack merge that includes this pull request via Graphite.
  • Oct 17, 5:23 PM PDT: Graphite rebased this pull request as part of a merge.
  • Oct 17, 5:25 PM PDT: A user merged this pull request with Graphite.

@neekolas neekolas force-pushed the 10-16-recover_signer_from_client_envelopes branch from 1e9a07a to 3a6f2db Compare October 18, 2024 00:18
Base automatically changed from 10-16-recover_signer_from_client_envelopes to main October 18, 2024 00:22
@neekolas neekolas force-pushed the 10-16-add_domain_separation_for_originator_signatures branch from 5a4631f to 93719d9 Compare October 18, 2024 00:22
@neekolas neekolas merged commit 161823d into main Oct 18, 2024
7 checks passed
@neekolas neekolas deleted the 10-16-add_domain_separation_for_originator_signatures branch October 18, 2024 00:25
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