-
-
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
OIDC integration contributions docs: Seafile (update) and Roundcube (new) #6195
OIDC integration contributions docs: Seafile (update) and Roundcube (new) #6195
Conversation
Thanks for choosing to contribute @sphakka. We lint all PR's with golangci-lint and eslint, I may add a review to your PR with some suggestions. You are free to apply the changes if you're comfortable, alternatively you are welcome to ask a team member for advice. ArtifactsThese changes once approved by a team member will be 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 configuration. |
Err, apparently GitHub doesn't like commits signed externally with my personal key :-( |
70785a5
to
1e02108
Compare
I think you need to add the public key to the account but it should be fine as is. I will look at it as I get some time this week. |
1e02108
to
e9b179f
Compare
Tested on latest stable Authelia and Seafile. Added extra info for SSO auth bypass when using Seafile WebDAV extension with nginx. Signed-off-by: Marco Emilio "sphakka" Poleggi <[email protected]>
Full stack configuration Roundcube via Nginx, Dovecot and Postfix. Signed-off-by: Marco Emilio "sphakka" Poleggi <[email protected]>
FIxed typos. Signed-off-by: Marco Emilio Poleggi <[email protected]>
Fixed bad formatting. Signed-off-by: Marco Emilio Poleggi <[email protected]>
e9b179f
to
671bb6b
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.
Really appreciate the PR! Mostly LGTM, few changes though. Let me know what you think.
Removed Nginx snippets as latest docs provide all needed info. Reworked doc structure to start with Authelia, then all other configurations. Signed-off-by: Marco Emilio "sphakka" Poleggi <[email protected]>
It should be better now ^^ |
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
Only documentation for OIDC integration is affected: