-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
website/docs: Fix websocket default config for nginx proxy manager #11621
Conversation
Resolves: - #10010 - #7323 - #11453 - https://www.reddit.com/r/Authentik/comments/1c5sf6l/authentik_with_nginx_proxy_manager_not_possible/ Signed-off-by: Mahmoud AlyuDeen <mahmoudalyudeen@gmail.com>
✅ Deploy Preview for authentik-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for authentik-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from being baffled how this standard nginx configuration breaking NPM, if anything I dont think it should be commented out but rather removed, as it being commented out without a reason makes it even more confusing
Signed-off-by: Mahmoud AlyuDeen <mahmoudalyudeen@gmail.com>
Makes sense. I've replaced the current setup with a working and tested I updated the PR title and description accordingly. |
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11621 +/- ##
=======================================
Coverage 92.75% 92.75%
=======================================
Files 736 736
Lines 36609 36609
=======================================
+ Hits 33956 33957 +1
+ Misses 2653 2652 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
* main: (68 commits) core: extract object matching from flow manager (#11458) admin: store version history (#11520) web: bump API Client version (#11706) providers/oauth2: add initial JWE support (#11344) core, web: update translations (#11703) tests/e2e: add forward auth e2e test (#11374) web/admin: fix duplicate flow labels (#11689) providers/saml: fix incorrect ds:Reference URI (#11699) website/docs: Fix websocket default config for nginx proxy manager (#11621) core, web: update translations (#11692) core: bump uvicorn from 0.31.1 to 0.32.0 (#11693) core: bump github.com/prometheus/client_golang from 1.20.4 to 1.20.5 (#11694) website/docs: add missing file to sidebar (#11695) website/docs: rewrote too long sentence (#11696) translate: Updates for file locale/en/LC_MESSAGES/django.po in fr (#11697) translate: Updates for file web/xliff/en.xlf in fr (#11698) stages/authenticator_webauthn: Update FIDO MDS3 & Passkey aaguid blobs (#11683) core, web: update translations (#11682) core: bump github.com/getsentry/sentry-go from 0.29.0 to 0.29.1 (#11684) core: bump github.com/redis/go-redis/v9 from 9.6.1 to 9.6.2 (#11685) ...
Details
Update suggested configuration replacing
commenting outlines that cause Nginx Proxy Manager host to go offline.Resolves:
Checklist
ak test authentik/
)make lint-fix
)