Skip to content
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

Support for 'expect' header #55

Open
systemmonkey42 opened this issue Oct 5, 2022 · 1 comment
Open

Support for 'expect' header #55

systemmonkey42 opened this issue Oct 5, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@systemmonkey42
Copy link

systemmonkey42 commented Oct 5, 2022

Hi,

I don't know if this is actively developed, but I struck an issue with a WEBDAV server where it sent an expect: 100-continue header which caused authelia to emit HTTP/1.1 100 Continue before sending the authentication results.

The auth-request.lua script immediately disconnected as soon as it saw the HTTP/1.1 100 Continue with the message

haproxy[231]: Invalid status code in auth-request backend 'be_authelia': 100

Adding the line

headers["expect"] = nil

fixes the problem.

Thanks

@TimWolla
Copy link
Owner

TimWolla commented Oct 5, 2022

I don't know if this is actively developed,

Not actively developed, because I consider this to be feature complete, I'll however try to respond to issues and to fix bugs (if any).

fixes the problem.

That sounds reasonable, would you consider creating a PR that fixes the issue for you? If possible include a VTest test that verifies the behavior. Explanation is here: #54 (comment)

@TimWolla TimWolla added the enhancement New feature or request label Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants