caddyserver / website

The Caddy website
156 stars 153 forks source link

[doc] Update forward_auth document to reflect latest change in authelia #421

Closed terrytw closed 1 month ago

terrytw commented 1 month ago

The document on caddy and authelia is inconsistent, using different example config:

https://caddyserver.com/docs/caddyfile/directives/forward_auth#authelia https://www.authelia.com/integration/proxies/caddy/#subpath

I have searched around for a bit and it seems authelia changed its api: https://fossies.org/diffs/authelia/4.38.9_vs_4.38.10/docs/content/blog/release-notes-4.38/index.md-diff.html

So maybe it is better to update caddy's doc to reflect that?

francislavoie commented 1 month ago

/cc @james-d-elliott if you can help us out with this to confirm?

james-d-elliott commented 1 month ago

I'll take a close look at this today. Thanks for pinging me.

james-d-elliott commented 1 month ago

Just confirming that the uri /api/verify?rd=https://auth.example.com should be updated to uri /api/authz/forward-auth.

Technically the /api/verify example still works, just it's not our recommended option anymore.

francislavoie commented 1 month ago

Okay sounds good. Is the redirect target configured on authelia now instead?

james-d-elliott commented 1 month ago

Correct, this was added as part of the multi-domain config. There is still a query argument but it's changed for those new endpoints and is intended on just being an override.

francislavoie commented 1 month ago

Updated, thanks!

https://github.com/caddyserver/website/commit/12e02d62b0534da21e1bf26112a3a92e00846c17