Skip to content
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

website/integrations: Add documentation for Drupal #12925

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

justafish
Copy link

Details

Adds documentation for authentication with Drupal


Checklist

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

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

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

@justafish justafish requested a review from a team as a code owner February 3, 2025 14:02
Copy link

netlify bot commented Feb 3, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit be824d2
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/67bdd69769498700089b0945

Copy link

netlify bot commented Feb 3, 2025

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit be824d2
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/67bdd69632319b00081e031b
😎 Deploy Preview https://deploy-preview-12925--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 site configuration.

@BeryJu BeryJu changed the title Add documentation for Drupal website/integrations: Add documentation for Drupal Feb 3, 2025
Copy link
Contributor

@dominic-r dominic-r left a comment

Choose a reason for hiding this comment

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

Hi, thanks for your contribution. Before reviewing this PR, would it be possible to make sure your contribution follows the template outlined in the documentation? Thanks.

@justafish
Copy link
Author

@dominic-r I've made this change (however the authentik configuration needs to happen before the Drupal configuration)

Copy link
Contributor

@dominic-r dominic-r left a comment

Choose a reason for hiding this comment

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

Hi @justafish . Thanks for your contribution. Here are a few comments. Please do let me know if you have any questions.

:::


## authentik configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

Note to tana: reminder: will update format in a later PR

@dominic-r
Copy link
Contributor

Hi @justafish. Just checking in to see if there are any developments related to this PR. thanks

@justafish
Copy link
Author

@dominic-r I've addressed your feedback 👍

@justafish justafish requested a review from dominic-r February 25, 2025 14:14
@justafish justafish requested a review from dominic-r February 25, 2025 14:42
## Drupal configuration

- From the Admin Toolbar or admin page at https://drupal.company/admin go to
Configuration -> OpenID Connect or directly at https://drupal.company/admin/config/services/openid-connect
Copy link
Contributor

@tanberry tanberry Feb 25, 2025

Choose a reason for hiding this comment

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

Suggested change
Configuration -> OpenID Connect or directly at https://drupal.company/admin/config/services/openid-connect
**Configuration -> OpenID Connect** or directly at https://drupal.company/admin/config/services/openid-connect.


- From the Admin Toolbar or admin page at https://drupal.company/admin go to
Configuration -> OpenID Connect or directly at https://drupal.company/admin/config/services/openid-connect
- Input the Client ID and Secret you noted above
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Input the Client ID and Secret you noted above
- Input the Client ID and Secret you noted above.

Configuration -> OpenID Connect or directly at https://drupal.company/admin/config/services/openid-connect
- Input the Client ID and Secret you noted above
- Fill out the following endpoints:
- Authorization endpoint: <kbd>https://<em>authentik.company</em>/application/o/authorize/</kbd>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Authorization endpoint: <kbd>https://<em>authentik.company</em>/application/o/authorize/</kbd>
- **Authorization endpoint**: <kbd>https://<em>authentik.company</em>/application/o/authorize/</kbd>

- Input the Client ID and Secret you noted above
- Fill out the following endpoints:
- Authorization endpoint: <kbd>https://<em>authentik.company</em>/application/o/authorize/</kbd>
- Token endpoint: <kbd>https://<em>authentik.company</em>/application/o/token/</kbd>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Token endpoint: <kbd>https://<em>authentik.company</em>/application/o/token/</kbd>
- **Token endpoint**: <kbd>https://<em>authentik.company</em>/application/o/token/</kbd>

- Fill out the following endpoints:
- Authorization endpoint: <kbd>https://<em>authentik.company</em>/application/o/authorize/</kbd>
- Token endpoint: <kbd>https://<em>authentik.company</em>/application/o/token/</kbd>
- UserInfo endpoint: <kbd>https://<em>authentik.company</em>/application/o/userinfo/</kbd>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- UserInfo endpoint: <kbd>https://<em>authentik.company</em>/application/o/userinfo/</kbd>
- **UserInfo endpoint**: <kbd>https://<em>authentik.company</em>/application/o/userinfo/</kbd>

If you are developing Drupal locally with DDEV and authentik is also running
locally, use `host.docker.internal:9000` as the hostname for the Token and UserInfo endpoints.
:::
- Select the "Override registration settings" checkbox
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Select the "Override registration settings" checkbox
- Select the "Override registration settings" checkbox.

locally, use `host.docker.internal:9000` as the hostname for the Token and UserInfo endpoints.
:::
- Select the "Override registration settings" checkbox
- Enable the OpenID button on user login form
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Enable the OpenID button on user login form
- Enable the OpenID button on user login form.

- Create an application e.g. "Drupal" and set the Provider field to the provider
created above

## Drupal configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

@justafish ideally these steps would be numbered, not bullets.

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 so much @justafish for this contribution. I was shocked that we didn't already have an integration guide for Drupal; now we do, thanks to you!

I made a few more minor edits, once those are in let's get this merged!

Copy link

codecov bot commented Feb 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.64%. Comparing base (7c944b9) to head (be824d2).
Report is 203 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12925      +/-   ##
==========================================
- Coverage   92.75%   90.64%   -2.11%     
==========================================
  Files         769      792      +23     
  Lines       38929    40232    +1303     
==========================================
+ Hits        36109    36469     +360     
- Misses       2820     3763     +943     
Flag Coverage Δ
e2e 48.02% <ø> (-0.55%) ⬇️
integration 24.13% <ø> (-0.48%) ⬇️
unit 88.17% <ø> (-2.22%) ⬇️

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.

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