Skip to content

website/docs: Fix CSP syntax#12124

Merged
BeryJu merged 1 commit intogoauthentik:mainfrom
thegcat:patch-1
Nov 25, 2024
Merged

website/docs: Fix CSP syntax#12124
BeryJu merged 1 commit intogoauthentik:mainfrom
thegcat:patch-1

Conversation

@thegcat
Copy link
Contributor

@thegcat thegcat commented Nov 21, 2024

Scheme sources need to not have quotes https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy#scheme-source

Signed-off-by: Felix Schäfer <felix.schaefer@tu-dortmund.de>
@thegcat thegcat requested a review from a team as a code owner November 21, 2024 16:14
@netlify
Copy link

netlify bot commented Nov 21, 2024

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 54cd619
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/673f5c6d6a616100088ea2cb
😎 Deploy Preview https://deploy-preview-12124--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Nov 21, 2024

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit 54cd619
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/673f5c6d65c7c00008c83fc8
😎 Deploy Preview https://deploy-preview-12124--authentik-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@BeryJu BeryJu changed the title Fix CSP syntax website/docs: Fix CSP syntax Nov 21, 2024
@BeryJu BeryJu requested a review from gergosimonyi November 21, 2024 16:23
@codecov
Copy link

codecov bot commented Nov 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.62%. Comparing base (7f224cb) to head (54cd619).
Report is 28 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12124      +/-   ##
==========================================
+ Coverage   92.60%   92.62%   +0.01%     
==========================================
  Files         761      761              
  Lines       38024    38024              
==========================================
+ Hits        35213    35218       +5     
+ Misses       2811     2806       -5     
Flag Coverage Δ
e2e 49.06% <ø> (-0.01%) ⬇️
integration 24.83% <ø> (ø)
unit 90.22% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

Copy link
Collaborator

@gergosimonyi gergosimonyi left a comment

Choose a reason for hiding this comment

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

Ah, of course. Thank you!

@BeryJu BeryJu merged commit 6c1ad98 into goauthentik:main Nov 25, 2024
@thegcat thegcat deleted the patch-1 branch November 25, 2024 17:59
kensternberg-authentik added a commit that referenced this pull request Nov 25, 2024
* main:
  website/docs: Fix CSP syntax (#12124)
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