authentik icon indicating copy to clipboard operation
authentik copied to clipboard

website/integrations: add Chronograf

Open tomlawesome opened this issue 1 year ago β€’ 5 comments

Add Chronograf oauth integration to website.

Details

Chronograf is not listed yet in integrations, but it's working well for me using their generic oauth and Authentik's oauth endpoints generated by simply creating an oauth provider.


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)
  • [ ] The translation files have been updated (make i18n-extract)

If applicable

  • [x] The documentation has been updated
  • [ ] The documentation has been formatted (make website)

tomlawesome avatar Dec 03 '23 19:12 tomlawesome

Deploy Preview for authentik-storybook ready!

Name Link
Latest commit 8ac6761920bab879b03b2b5ce3b9da3075f95953
Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/657476f53f0b6200080ad881
Deploy Preview https://deploy-preview-7766--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 site configuration.

netlify[bot] avatar Dec 03 '23 19:12 netlify[bot]

Deploy Preview for authentik ready!

Name Link
Latest commit 8ac6761920bab879b03b2b5ce3b9da3075f95953
Latest deploy log https://app.netlify.com/sites/authentik/deploys/657476f5cf227400082995a3
Deploy Preview https://deploy-preview-7766--authentik.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse
1 paths audited
Performance: 97 (no change from production)
Accessibility: 90 (no change from production)
Best Practices: 100 (no change from production)
SEO: 80 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

netlify[bot] avatar Dec 03 '23 19:12 netlify[bot]

Thanks so much @tomlawesome for this contribution! I'll do a copy-edit on the docs, and get a tech review on it. Always nice to add more integrations.

tanberry avatar Dec 04 '23 16:12 tanberry

Codecov Report

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

Comparison is base (6bdbdaf) 92.64% compared to head (8ac6761) 92.63%. Report is 47 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7766      +/-   ##
==========================================
- Coverage   92.64%   92.63%   -0.01%     
==========================================
  Files         588      588              
  Lines       29141    29200      +59     
==========================================
+ Hits        26997    27049      +52     
- Misses       2144     2151       +7     
Flag Coverage Ξ”
e2e 50.65% <ΓΈ> (-0.07%) :arrow_down:
integration 25.91% <ΓΈ> (-0.04%) :arrow_down:
unit 89.70% <ΓΈ> (-0.02%) :arrow_down:

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 Dec 11 '23 07:12 codecov[bot]

Hey, thanks a lot for this PR, this helped a lot.

However I had to add USE_ID_TOKEN=true for the setup to properly work (as per https://docs.influxdata.com/chronograf/v1/administration/managing-security/#jwks-signature-verification-optional).

PierreBtz avatar Feb 18 '24 17:02 PierreBtz

Hi @tomlawesome can you take a look at the last few points/edits, and then see about adding the tip that @PierreBtz made? Also, you'll need to run make website or make website-lint-fix before pushing your final commit, so that the build check for linting passes.

Apologies we let this one sit for a bot; we are looking forward to your reply and getting it merged, thanks!

tanberry avatar Feb 23 '24 18:02 tanberry