Skip to content

Nextcloud.md updated & Added the tutorial for OIDC & Authelia#369

Merged
nitnelave merged 8 commits intolldap:mainfrom
Normanras:nextcloud_docs
Nov 18, 2022
Merged

Nextcloud.md updated & Added the tutorial for OIDC & Authelia#369
nitnelave merged 8 commits intolldap:mainfrom
Normanras:nextcloud_docs

Conversation

@Normanras
Copy link
Copy Markdown
Contributor

Added Tutorial for Nextcloud/OIDC/Authelia/LLDAP & Updated minor changes on Nextcloud.md

@Normanras
Copy link
Copy Markdown
Contributor Author

Forgot to run cargo... closing this PR and resubmitting.

@Normanras Normanras closed this Nov 11, 2022
@Normanras Normanras reopened this Nov 11, 2022
@Normanras
Copy link
Copy Markdown
Contributor Author

no errors with cargo fmt

Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud.md
Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Copy link
Copy Markdown
Member

@nitnelave nitnelave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the many comments, but it's a rather complicated setup, so I want to try to eliminate as much ambiguity as possible.

Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud.md Outdated
Comment thread example_configs/nextcloud.md
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
@Normanras
Copy link
Copy Markdown
Contributor Author

Sorry for the many comments, but it's a rather complicated setup, so I want to try to eliminate as much ambiguity as possible.

No problem at all! I would rather go through and get it right instead of having to go back to redo the work in a new PR. I'm not sure why there are so many backtick errors though, I made sure to preview everything locally and in my fork and it looked normal. Thanks for finding and commenting on them.

@Normanras
Copy link
Copy Markdown
Contributor Author

Normanras commented Nov 15, 2022

Okay, I feel like we're going down a path here, but for this PR I've implemented all the requested changes and added some extra links and references (such as Authelia's client secret/id). I also removed over-explaining lines and added a section at the top detailing assumptions.

I don't have time in the coming weeks, but I think revamping this with a bit more explanation of the Authelia setup might add some clarity. I'm open to your feedback, but let me know what you see in this commit. As I said, I'd rather it be right this time rather than go back after its merged. I'm mostly concerned that the backticks errors will persist, so I'll have to open it in another editor if it continues to revert to apostrophes.

…users > nextcloud_users, clarity to certain sections.
Copy link
Copy Markdown
Member

@nitnelave nitnelave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good! Can you just remove the example_configs/images/.DS_Store file?

Comment thread example_configs/nextcloud.md Outdated
@Normanras
Copy link
Copy Markdown
Contributor Author

It looks good! Can you just remove the example_configs/images/.DS_Store file?

Yes, done.

Copy link
Copy Markdown
Member

@nitnelave nitnelave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost there! It should be the last round. Sorry I didn't catch these earlier.

Comment thread example_configs/nextcloud_oidc_authelia.md Outdated
Comment thread example_configs/nextcloud_oidc_authelia.md
Comment thread example_configs/nextcloud_oidc_authelia.md
@nitnelave
Copy link
Copy Markdown
Member

Thanks a lot!

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 18, 2022

Codecov Report

Merging #369 (750b404) into main (03ad10d) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #369   +/-   ##
=======================================
  Coverage   68.60%   68.60%           
=======================================
  Files          32       32           
  Lines        6236     6236           
=======================================
  Hits         4278     4278           
  Misses       1958     1958           

@nitnelave nitnelave merged commit ffc59af into lldap:main Nov 18, 2022
@Normanras Normanras deleted the nextcloud_docs branch November 18, 2022 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants