Skip to content

fix: if not forced, reprotect response to MAC-protected requests#101

Open
Akretsch wants to merge 3 commits into
mainfrom
fix_force_ReprotectMode
Open

fix: if not forced, reprotect response to MAC-protected requests#101
Akretsch wants to merge 3 commits into
mainfrom
fix_force_ReprotectMode

Conversation

@Akretsch

Copy link
Copy Markdown
Collaborator

@DDvO

DDvO commented Oct 28, 2025

Copy link
Copy Markdown
Member

Please rebase and extend the new sentence currently ending in
(regardless of the configuration related to reprotection or output credentials).

to point out that this is only if EnforceReprotectMode is false.

@Akretsch
Akretsch force-pushed the fix_force_ReprotectMode branch from 2781634 to 3703819 Compare October 28, 2025 15:02
@Akretsch

Akretsch commented Oct 28, 2025

Copy link
Copy Markdown
Collaborator Author

Please rebase and extend the new sentence currently ending in (regardless of the configuration related to reprotection or output credentials).

to point out that this is only if EnforceReprotectMode is false.

Rebase and text proposal done.

@DDvO DDvO left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Further corrections, for precisely describing the behavior.

Comment thread doc/config/README.md Outdated
Comment thread doc/config/README.md Outdated
Comment thread doc/config/README.md Outdated
@DDvO

DDvO commented Nov 4, 2025

Copy link
Copy Markdown
Member

As we just aligned, I updated my improvements on the documentation.
Please check if they are to-the-point.

Co-authored-by: David von Oheimb <DDvO@users.noreply.github.com>

@ZMaradics ZMaradics left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved.

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.

3 participants