Skip to content

website/integrations: change nextcloud scope name to avoid confusion#15050

Merged
BeryJu merged 1 commit intogoauthentik:mainfrom
Keksmo:patch-2
Jun 16, 2025
Merged

website/integrations: change nextcloud scope name to avoid confusion#15050
BeryJu merged 1 commit intogoauthentik:mainfrom
Keksmo:patch-2

Conversation

@Keksmo
Copy link
Contributor

@Keksmo Keksmo commented Jun 15, 2025

This PR changes the Nextcloud Docs by replacing the proposed scope name 'profile' to 'nextcloud'. The reason for this is, that the name 'profile' is very close to the default scope 'profiles', which had lead me (and probably many others) to hours of debugging even though the issue is very simple and can easily be avoided by renaming the scope, which eliminates this confusion.

Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make website)

Signed-off-by: Marlin <77961876+Keksmo@users.noreply.github.com>
@Keksmo Keksmo requested a review from a team as a code owner June 15, 2025 17:16
@netlify
Copy link

netlify bot commented Jun 15, 2025

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 94525d9
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/684effd24c6b5b000881a5fe
😎 Deploy Preview https://deploy-preview-15050--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Jun 15, 2025

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit 94525d9
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/684effd285216f00083fbdba
😎 Deploy Preview https://deploy-preview-15050--authentik-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@tanberry
Copy link
Contributor

Thanks @Keksmo for this contribution! Although authentik doesn't really show "profiles" scope as a default (we don't suggest one at all), "profiles" is a common OpenID Connect scope ... so I can understand why it's potentially confusing for us to have suggested "profile" as the scope name! Thank you for clarifying this point in our docs.

@codecov
Copy link

codecov bot commented Jun 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.76%. Comparing base (ee47803) to head (94525d9).
Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #15050   +/-   ##
=======================================
  Coverage   92.76%   92.76%           
=======================================
  Files         815      815           
  Lines       42117    42117           
=======================================
+ Hits        39068    39070    +2     
+ Misses       3049     3047    -2     
Flag Coverage Δ
e2e 47.59% <ø> (+<0.01%) ⬆️
integration 24.35% <ø> (+<0.01%) ⬆️
unit 90.65% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tanberry
Copy link
Contributor

Thanks @Keksmo for this contribution! Although authentik doesn't really show "profiles" scope as a default (we don't suggest one at all), "profiles" is a common OpenID Connect scope ... so I can understand why it's potentially confusing for us to have suggested "profile" as the scope name! Thank you for clarifying this point in our docs.

1 similar comment
@tanberry
Copy link
Contributor

Thanks @Keksmo for this contribution! Although authentik doesn't really show "profiles" scope as a default (we don't suggest one at all), "profiles" is a common OpenID Connect scope ... so I can understand why it's potentially confusing for us to have suggested "profile" as the scope name! Thank you for clarifying this point in our docs.

Copy link
Contributor

@tanberry tanberry left a comment

Choose a reason for hiding this comment

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

Thanks again @Keksmo and sorry about the lost hours debugging... :-(

@BeryJu BeryJu changed the title Changed Scope naming of Nextcloud Docs ("profile" to "nextcloud" website/integrations: change nextcloud scope name to avoid confusion Jun 16, 2025
@BeryJu BeryJu merged commit bce70a1 into goauthentik:main Jun 16, 2025
99 checks passed
kensternberg-authentik added a commit that referenced this pull request Jun 16, 2025
* main: (43 commits)
  Web/cleanup/empty state better slot handling (#14289)
  website/docs: release notes for `2025.6.2` (#15065)
  website/docs: remove commented out config options (#15064)
  website/docs: postgres troubleshooting: get PGPASSWORD from POSTGRES_PASSWORD_FILE (#15039)
  core: bump goauthentik/fips-python from 3.13.4-slim-bookworm-fips to 3.13.5-slim-bookworm-fips (#15058)
  website: bump the eslint group in /website with 2 updates (#15059)
  web: bump the wdio group across 1 directory with 3 updates (#14593)
  web: bump @sentry/browser from 9.28.1 to 9.29.0 in /web in the sentry group across 1 directory (#15061)
  web: bump the eslint group across 2 directories with 2 updates (#15062)
  core: bump axllent/mailpit from v1.26.0 to v1.26.1 in /tests/e2e (#15060)
  core: bump pydantic from 2.11.5 to 2.11.7 (#15063)
  providers/oauth2: bug fixes from conformance testing (#15056)
  website/integrations: change nextcloud scope name to avoid confusion (#15050)
  web/flow: cleanup WebAuthn helper functions (#14460)
  web/elements: fix typo in localeComparator (#15054)
  stages/authenticator_webauthn: Update FIDO MDS3 & Passkey aaguid blobs (#15049)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in es (#15047)
  core, web: update translations (#15048)
  blueprints: add section support for organisation (#15045)
  core, web: update translations (#15043)
  ...
cavefire pushed a commit to cavefire/authentik that referenced this pull request Jun 24, 2025
…oauthentik#15050)

changed "profile" to "nextcloud"

Signed-off-by: Marlin <77961876+Keksmo@users.noreply.github.com>
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.

3 participants