Skip to content

Feature/cross origin embedder policy#164

Merged
GaProgMan merged 10 commits intomainfrom
feature/cross-origin-embedder-policy
Dec 30, 2024
Merged

Feature/cross origin embedder policy#164
GaProgMan merged 10 commits intomainfrom
feature/cross-origin-embedder-policy

Conversation

@GaProgMan
Copy link
Copy Markdown
Owner

Rationale for this PR

This PR adds the following header: Cross-Origin-Opener-Policy. This recommended value header is documented here, and the header itself is documented here.

This PR closes #74

The following is a minimal code sample for the new feature:

// in Program.cs
app.UseSecureHeadersMiddleware();

PR Checklist

Feel free to either check the following items (by place an x inside of the square brackets) or by replacing the square brackets with a relevant emoji from the following list:

  • ✅ to indicate that you have checked something off
  • ❎ to indicate that you haven't checked something off
  • ❓ to indicate that something might not be relevant (writing tests for documentation changes, for instance)

Essential

These items are essential and must be completed for each commit. If they are not completed, the PR may not be accepted.

  • [ ✅ ] I have added tests to the OwaspHeaders.Core.Tests project
  • [ ✅ ] I have run the dotnet-format command and fixed any .editorconfig issues
  • [ ✅ ] I have ensured that the code coverage has not dropped below 65%
  • [ ✅ ] I have increased the version number in OwaspHeaders.Core.csproj (only relevant for code changes)

Optional

  • [ ✅ ] I have documented the new feature in the docs directory
  • [ ✅ ] I have provided a code sample, showing how someone could use the new code

Any Other Information

This section is optional, but it might be useful to list any other information you think is relevant.

@github-actions
Copy link
Copy Markdown
Contributor

Code Coverage

Package Line Rate Branch Rate Complexity Health
OwaspHeaders.Core 87% 74% 303
OwaspHeaders.Core 87% 74% 303
Summary 87% (1242 / 1422) 74% (290 / 392) 606

@GaProgMan GaProgMan merged commit 1a8e56f into main Dec 30, 2024
@GaProgMan GaProgMan deleted the feature/cross-origin-embedder-policy branch December 30, 2024 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cross-Origin-Embedder-Policy not supported

2 participants