-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
feat(server): allowed hosts #3680
base: master
Are you sure you want to change the base?
Conversation
ArtifactsThese changes are published for testing on Buildkite, DockerHub and GitHub Container Registry. Docker Container
|
✅ Deploy Preview for authelia-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
09d91ab
to
200adb9
Compare
This implements an allowed hosts middleware which helps prevent communication with Authelia via unknown host headers.
200adb9
to
a74aea1
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #3680 +/- ##
==========================================
+ Coverage 68.90% 68.95% +0.04%
==========================================
Files 257 258 +1
Lines 16184 16215 +31
Branches 431 431
==========================================
+ Hits 11152 11181 +29
- Misses 4337 4340 +3
+ Partials 695 694 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
7d3cba7
to
a0fdee4
Compare
406fc3a
to
771bfea
Compare
fe5e950
to
1359082
Compare
0b320e9
to
270c99e
Compare
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.
Only a couple of minor nits.
Co-authored-by: Amir Zarrinkafsh <nightah@me.com>
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.
LGTM.
This implements an allowed hosts middleware which helps prevent communication with Authelia via unknown host headers.