Skip to content

RM-9658 Add parsing for csp header#1418

Merged
noble-rubicon merged 1 commit intodevelopfrom
feature/RM-9658-add-csp-parser-for-header
Mar 23, 2026
Merged

RM-9658 Add parsing for csp header#1418
noble-rubicon merged 1 commit intodevelopfrom
feature/RM-9658-add-csp-parser-for-header

Conversation

@noble-rubicon
Copy link
Contributor

No description provided.

Copy link
Member

@MichaelKetting MichaelKetting left a comment

Choose a reason for hiding this comment

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

Mostly formatting and some ASCII questions that are meant as question not a definite change it. If you agree with my reasoning, please do change it, though.

@noble-rubicon noble-rubicon force-pushed the feature/RM-9658-add-csp-parser-for-header branch 2 times, most recently from 924eeca to a17c9c2 Compare March 19, 2026 14:21
Copy link
Member

@MichaelKetting MichaelKetting left a comment

Choose a reason for hiding this comment

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

Sorry, I managed to find a consistency problem with the existing code. I think it's a quick and easy fix.

@noble-rubicon noble-rubicon force-pushed the feature/RM-9658-add-csp-parser-for-header branch 3 times, most recently from 1ee8502 to 56d45cc Compare March 23, 2026 10:49
@noble-rubicon noble-rubicon force-pushed the feature/RM-9658-add-csp-parser-for-header branch from 56d45cc to 1708cf2 Compare March 23, 2026 14:37
@noble-rubicon noble-rubicon merged commit b92d301 into develop Mar 23, 2026
1 check passed
@noble-rubicon noble-rubicon deleted the feature/RM-9658-add-csp-parser-for-header branch March 23, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants