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

Use same credentials for MAC based protection at Downstream #106

Merged
merged 8 commits into from
Sep 10, 2024

Conversation

kiron-mx
Copy link
Collaborator

Addresses issue #91

  • Introduced new container class MessageContext to store persistency context and verification credentials of incoming messages
  • If outgoing message is configured to be reprotected, any configuration of outgoing credentials is ignored and the verification credentials are used instead
  • In case of failed protection validation the configured outgoing credentials are used for the outgoing error message

@kiron-mx kiron-mx linked an issue Apr 30, 2024 that may be closed by this pull request
@kiron-mx kiron-mx requested a review from Akretsch April 30, 2024 11:15
@kiron-mx kiron-mx force-pushed the 91-response-protection-not-correctly-chosen branch from 1dc863e to d5df3f1 Compare April 30, 2024 15:36
Copy link

sonarcloud bot commented Apr 30, 2024

import org.bouncycastle.asn1.pkcs.PBKDF2Params;
import org.bouncycastle.asn1.pkcs.PKCSObjectIdentifiers;

public class SharedSecretCredentials implements SharedSecretCredentialContext {
Copy link
Collaborator

Choose a reason for hiding this comment

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

rename to RespondingSharedSecretCredentials or similar

@kiron-mx kiron-mx force-pushed the 91-response-protection-not-correctly-chosen branch from d5df3f1 to 1dc863e Compare May 17, 2024 15:06
@kiron-mx kiron-mx force-pushed the 91-response-protection-not-correctly-chosen branch from 1dc863e to b2090d9 Compare August 28, 2024 19:09
Copy link

sonarcloud bot commented Sep 10, 2024

@Akretsch Akretsch merged commit 041ef55 into main Sep 10, 2024
5 checks passed
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.

Response protection not correctly chosen
2 participants