feat: Added authentik as new identity provider

Yoyasp commented 1 year ago

Description

Added Authentik as new identity provider.

This PR includes:

Authentik as new provider in the security manager.
Role mapping from Authentik groups
JWKS validation on Authentik tokens if provided in the config.py.
New exception if token signature cannot be validated.
Tests for the authentication for parsing the correct info, checking if the signature is valid and allowing unsigned tokens if specified in the config.

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Is CRUD MVC related.
[x] Is Auth, RBAC security related.
[ ] Changes the security db schema.
[x] Introduces new feature
[ ] Removes existing feature

codecov[bot] commented 1 year ago

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (59db85d) 79.31% compared to head (7d63e4a) 79.32%.

Files	Patch %	Lines
flask_appbuilder/security/manager.py	81.25%	6 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #2168 +/- ## ======================================= Coverage 79.31% 79.32% ======================================= Files 72 72 Lines 8974 9007 +33 ======================================= + Hits 7118 7145 +27 - Misses 1856 1862 +6 ``` | [Flag](https://app.codecov.io/gh/dpgaspar/Flask-AppBuilder/pull/2168/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Daniel+Vaz+Gaspar) | Coverage Δ | | |---|---|---| | [python](https://app.codecov.io/gh/dpgaspar/Flask-AppBuilder/pull/2168/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Daniel+Vaz+Gaspar) | `79.32% <82.35%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Daniel+Vaz+Gaspar#carryforward-flags-in-the-pull-request-comment) to find out more.

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

Yoyasp commented 1 year ago

Some how flake8 keeps giving me errors in the CI pipeline, however i dont get any errors while running flake8 on my dev host... Any help is appreciated.

dpgaspar commented 9 months ago

Sorry for the delay, please remove the requests dependency

forget it we already depend on it, I'll open a PR to include it on extras

dpgaspar / Flask-AppBuilder