website/integrations: nextcloud: add SSE warning#11976
Merged
BeryJu merged 3 commits intogoauthentik:mainfrom Nov 12, 2024
Merged
website/integrations: nextcloud: add SSE warning#11976BeryJu merged 3 commits intogoauthentik:mainfrom
BeryJu merged 3 commits intogoauthentik:mainfrom
Conversation
Signed-off-by: Dis <397465+disconn3ct@users.noreply.github.com>
✅ Deploy Preview for authentik-storybook canceled.
|
✅ Deploy Preview for authentik-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
tanberry
reviewed
Nov 11, 2024
tanberry
reviewed
Nov 11, 2024
|
|
||
| - [via OIDC Connect (OAuth)](#openid-connect-auth) | ||
| - [via SAML](#saml-auth) | ||
| - via LDAP outpost |
Contributor
There was a problem hiding this comment.
Since we are adding a third option here, we need to add a section below for it, like the other two. This new sectin could be quite short, just a sentence explaining the process conceptually, and link off to this doc: https://docs.goauthentik.io/docs/add-secure-apps/outposts/#more-about-outpost-integrations.
Or link to the separate docs for K9s vs Docker...
Contributor
Author
There was a problem hiding this comment.
It is not trivial and I do not have a clean configuration to offer, much less a reasonable process. I thought the data loss warranted a notice. I am not agreeing to expand the scope of the volunteer work I am doing at this time.
Contributor
There was a problem hiding this comment.
No worries, I can do it later, after we merge your addition! Thanks again for adding the important info.
tanberry
previously requested changes
Nov 11, 2024
Contributor
tanberry
left a comment
There was a problem hiding this comment.
Thanks so much for this. A suggestion and a few edits, please, then let's get it merged.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11976 +/- ##
==========================================
- Coverage 92.64% 92.57% -0.07%
==========================================
Files 761 761
Lines 37816 37841 +25
==========================================
- Hits 35033 35032 -1
- Misses 2783 2809 +26
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: Dis <397465+disconn3ct@users.noreply.github.com>
BeryJu
reviewed
Nov 12, 2024
Signed-off-by: Jens L. <jens@beryju.org>
BeryJu
approved these changes
Nov 12, 2024
kensternberg-authentik
added a commit
that referenced
this pull request
Nov 14, 2024
* main: providers/ldap: fix global search_full_directory permission not being sufficient (#12028) website/docs: 2024.10.2 release notes (#12025) lifecycle: fix ak exit status not being passed (#12024) core: use versioned_script for path only (#12003) core, web: update translations (#12020) core: bump google-api-python-client from 2.152.0 to 2.153.0 (#12021) providers/oauth2: fix manual device code entry (#12017) crypto: validate that generated certificate's name is unique (#12015) core, web: update translations (#12006) core: bump google-api-python-client from 2.151.0 to 2.152.0 (#12007) translate: Updates for file locale/en/LC_MESSAGES/django.po in zh-Hans (#12011) translate: Updates for file locale/en/LC_MESSAGES/django.po in zh_CN (#12010) translate: Updates for file web/xliff/en.xlf in zh-Hans (#12012) translate: Updates for file web/xliff/en.xlf in zh_CN (#12013) providers/proxy: fix Issuer when AUTHENTIK_HOST_BROWSER is set (#11968) website/docs: move S3 ad GeoIP to System Management/Operations (#11998) website/integrations: nextcloud: add SSE warning (#11976)
kensternberg-authentik
added a commit
that referenced
this pull request
Nov 14, 2024
* main: providers/ldap: fix global search_full_directory permission not being sufficient (#12028) website/docs: 2024.10.2 release notes (#12025) lifecycle: fix ak exit status not being passed (#12024) core: use versioned_script for path only (#12003) core, web: update translations (#12020) core: bump google-api-python-client from 2.152.0 to 2.153.0 (#12021) providers/oauth2: fix manual device code entry (#12017) crypto: validate that generated certificate's name is unique (#12015) core, web: update translations (#12006) core: bump google-api-python-client from 2.151.0 to 2.152.0 (#12007) translate: Updates for file locale/en/LC_MESSAGES/django.po in zh-Hans (#12011) translate: Updates for file locale/en/LC_MESSAGES/django.po in zh_CN (#12010) translate: Updates for file web/xliff/en.xlf in zh-Hans (#12012) translate: Updates for file web/xliff/en.xlf in zh_CN (#12013) providers/proxy: fix Issuer when AUTHENTIK_HOST_BROWSER is set (#11968) website/docs: move S3 ad GeoIP to System Management/Operations (#11998) website/integrations: nextcloud: add SSE warning (#11976)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Details
Nextcloud server-side encryption requires access to the user's plaintext password. This adds a warning to the integration doc.
Closes #11975
Checklist
ak test authentik/)make lint-fix)If an API change has been made
make gen-build)If changes to the frontend have been made
make web)If applicable
make website)