authentik icon indicating copy to clipboard operation
authentik copied to clipboard

website/integrations: add cloudflare access

Open dominic-r opened this issue 1 year ago • 7 comments

Details

closes #9710

documents how to protect selfhosted applications with cloudflare access and authentik


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)

dominic-r avatar May 13 '24 22:05 dominic-r

Deploy Preview for authentik-docs ready!

Name Link
Latest commit 8ee578503110e31ca403c8ff6edb078c7863be01
Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/66565f6774490c0008a4e35f
Deploy Preview https://deploy-preview-9713--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.

netlify[bot] avatar May 13 '24 22:05 netlify[bot]

Deploy Preview for authentik-storybook canceled.

Name Link
Latest commit 8ee578503110e31ca403c8ff6edb078c7863be01
Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/66565f674e5fd90008856139

netlify[bot] avatar May 13 '24 22:05 netlify[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.63%. Comparing base (fd66be9) to head (8ee5785). Report is 65 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9713      +/-   ##
==========================================
- Coverage   92.64%   92.63%   -0.01%     
==========================================
  Files         710      710              
  Lines       34706    34744      +38     
==========================================
+ Hits        32152    32185      +33     
- Misses       2554     2559       +5     
Flag Coverage Δ
e2e 49.66% <ø> (-0.01%) :arrow_down:
integration 25.49% <ø> (-0.02%) :arrow_down:
unit 90.09% <ø> (+<0.01%) :arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 13 '24 22:05 codecov[bot]

I've noticed that integration documentation varies greatly in style—from the inclusion of images to the use of numbers in headers, and more. I'd be happy to update older documentation to match the new style guide you mentioned once it's released, if needed.

dominic-r avatar May 28 '24 22:05 dominic-r

@4d62ext you're right, the variation amongst the Integrations docs is wild... I have been sorta kinda trying to get a hold on it all, but haven't managed to yet. Your help, as much or as little as you have time for, would be greatly appreciated.

I think that the biggest variations are in the headers (as you pointed out), the capitalization, and the ordering of the sections (sometimes authentik is first, sometimes it is last). And yes, also the use of numbered Steps. That is something I struggle with even in our regular docs; whether or not to use numbered steps. It has become less and less common, but my old-fashioned self thinks they are very useful. Sometimes several steps can be grouped together into one section, and that section labeled Step 1 ... as opposed to numbering literally every line and having 27 steps, lol. But it is a tricky one. I'd like your input on whether it makes sense in the Integration docs.

Finally, I did just push a PR #9900 with our baby Style Guide... we are open to input on that as well; it's just a start.

Thank you for all your help with authentik!

tanberry avatar May 29 '24 15:05 tanberry

Oh, forgot to say... in general, I'd like to limit the use of screenshots/images in the docs (too much maintenance). The only time we want to use them is if there is a particularly difficult-to-find UI element, and we need to show where it is.

tanberry avatar May 29 '24 15:05 tanberry

@tanberry I personally find it better if it is streamlined it into just a list of steps without necessarily distinguishing between the two sections. It feels less cramped and also allows proper headers and sections (Ex: steps directly in h2 instead of authentik Settings being h2 and a step being h3) As for the numbering in headers, I'm inclined to agree with you. Most documentation pages are self-explanatory and a chunk of the images could be removed. I'll open a few pull requests once the style guide is released and my contribution in #9899 is marked ready for review. It is simply renaming functions so it should not take too much time. Mabye a few test fixes as well.

dominic-r avatar May 29 '24 23:05 dominic-r

Thanks again, @4d62ext ! I am not sure if I prefer the way you did it, with the H2s being steps that continue in numbering through the two sections, or keeping the authentik config and service config as separate sections with separately numbered steps. I'll have a chat with the team and tweak the template (that no one can find anyway haha) to reflect the decision.

But in the meantime let's get this important integration doc out there! :-)

tanberry avatar May 30 '24 16:05 tanberry