Skip to content

website/docs: improve flow inspector docs#8993

Merged
BeryJu merged 1 commit intomainfrom
website/docs/improve-flow-inspector-docs
Mar 21, 2024
Merged

website/docs: improve flow inspector docs#8993
BeryJu merged 1 commit intomainfrom
website/docs/improve-flow-inspector-docs

Conversation

@BeryJu
Copy link
Member

@BeryJu BeryJu commented Mar 21, 2024

Details

REPLACE ME


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)

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu requested a review from a team as a code owner March 21, 2024 16:22
@netlify
Copy link

netlify bot commented Mar 21, 2024

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit b61a828
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/65fc5eafae039b00082ea77b

@netlify
Copy link

netlify bot commented Mar 21, 2024

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit b61a828
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/65fc5eafcdb0520008edfa09
😎 Deploy Preview https://deploy-preview-8993--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.

When running a flow with the inspector enabled, the flow is still executed normally. This means that for example, a [User write](../flow/stages/user_write.md) stage will _actually_ write user data.
:::

### Accessing the inspector
Copy link
Member

Choose a reason for hiding this comment

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

So should we allow users with the debug attribute as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

it should probably be an RBAC permission instead of just flat-out superuser, but the reason why this works the way it does is because you need to be logged in a super user to then start the flow with inspector as otherwise the user wouldn't be authenticated

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.

Thank you, much better!

@BeryJu BeryJu merged commit bcb7c72 into main Mar 21, 2024
@BeryJu BeryJu deleted the website/docs/improve-flow-inspector-docs branch March 21, 2024 18:03
kensternberg-authentik added a commit that referenced this pull request Mar 26, 2024
* main:
  web: bump API Client version (#9021)
  sources/ldap: add ability to disable password write on login (#8377)
  web: bump API Client version (#9020)
  lifecycle: migrate: ensure template schema exists before migrating (#8952)
  website/integrations: Update nextcloud Admin Group Expression (#7314)
  web/flow: general ux improvements (#8558)
  website: bump @types/react from 18.2.67 to 18.2.69 in /website (#9016)
  core: bump requests-oauthlib from 1.4.0 to 2.0.0 (#9018)
  web: bump the sentry group in /web with 2 updates (#9017)
  web/admin: small fixes (#9002)
  website: bump webpack-dev-middleware from 5.3.3 to 5.3.4 in /website (#9001)
  core: bump ruff from 0.3.3 to 0.3.4 (#8998)
  website/docs: Upgrade nginx reverse porxy config (#8947)
  website/docs: improve flow inspector docs (#8993)
  website/deverlop-docs website/integrations: add links to integrations template (#8995)
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