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: envoy support #3793
feat: envoy support #3793
Conversation
ArtifactsThese changes are published for testing on Buildkite, DockerHub and GitHub Container Registry. Docker Container
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #3793 +/- ##
==========================================
+ Coverage 69.17% 69.27% +0.09%
==========================================
Files 258 258
Lines 16228 16252 +24
Branches 434 435 +1
==========================================
+ Hits 11226 11258 +32
+ Misses 4313 4306 -7
+ Partials 689 688 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
✅ Deploy Preview for authelia-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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, great work with this!
2e605d9
to
99f6fc1
Compare
Likewise, we knocked this out so damn quickly. Really helps to have a few sets of eyes. |
11315e5
to
fc791ca
Compare
fc791ca
to
191d916
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.
Looks like we left a couple debug log lines in since we were trying to fix this.
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.
b6a9ff9
to
19a8cdb
Compare
This effectively adds support for envoy. Docs to come.