fix: load user info for okta

[houqp]

Description

Fix okta integration

ADDITIONAL INFORMATION

• Has associated issue: close Okta Oauth - Fails with 'Response' object has no attribute 'data' #1577
• Is CRUD MVC related.
• Is Auth, RBAC security related.
• Changes the security db schema.
• Introduces new feature
• Removes existing feature

[codecov]

Codecov Report

Merging #1589 (c64c735) into master (dbe1ede) will decrease coverage by 4.73%.
The diff coverage is 88.23%.

❗ Current head c64c735 differs from pull request most recent head c0b8851. Consider uploading reports for the commit c0b8851 to get more accurate results

@@            Coverage Diff             @@
##           master    #1589      +/-   ##
==========================================
- Coverage   74.76%   70.02%   -4.74%     
==========================================
  Files          46       54       +8     
  Lines        6585     7869    +1284     
==========================================
+ Hits         4923     5510     +587     
- Misses       1662     2359     +697     

Flag	Coverage Δ
python	70.02% <88.23%> (-4.74%)	⬇️

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

Impacted Files	Coverage Δ
flask_appbuilder/baseviews.py	89.86% <ø> (ø)
flask_appbuilder/security/manager.py	72.05% <0.00%> (-0.09%)	⬇️
flask_appbuilder/__init__.py	100.00% <100.00%> (ø)
flask_appbuilder/models/filters.py	88.19% <100.00%> (-0.62%)	⬇️
flask_appbuilder/models/sqla/filters.py	76.66% <100.00%> (+5.53%)	⬆️
flask_appbuilder/security/mongoengine/models.py	0.00% <0.00%> (-89.34%)	⬇️
flask_appbuilder/models/mongoengine/interface.py	0.00% <0.00%> (-77.50%)	⬇️
flask_appbuilder/models/mongoengine/filters.py	0.00% <0.00%> (-68.58%)	⬇️
flask_appbuilder/security/mongoengine/manager.py	0.00% <0.00%> (-60.35%)	⬇️
flask_appbuilder/models/mongoengine/fields.py	41.66% <0.00%> (-27.78%)	⬇️
... and 12 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 302e4ab...c0b8851. Read the comment docs.

[jonathonbattista]

What is the status on this?

[houqp]

waiting for project maintainer to approve and merge, but we are already running this in production :)

[jonathonbattista]

@houqp How did you go about packaging this?

[houqp]

pip install from my own git branch

[thesuperzapper]

@dpgaspar can you review this PR?

[dpgaspar]

Thank you for the fix!

[jonathonbattista]

When will a new release be cut?

[dpgaspar]

Next week

[thesuperzapper]

@houqp @dpgaspar do you think we also need to do this for "azure"?
https://github.com/dpgaspar/Flask-AppBuilder/blob/master/flask_appbuilder/security/manager.py#L591-L609

[houqp]

I don't have the answer because I don't have the environment to test. I recommend giving it a try and file issue to report any bug you found.

[thesuperzapper]

Issue is I don't have any azure setup either!

Anyone who does, please comment!

[dpgaspar]

RC1 for 3.2.2 is out: https://pypi.org/project/Flask-AppBuilder/3.2.2rc1/

3.2.2 will be out today/tomorrow

[aliuyusuf003]

I am using azure oauth provider for superset on my localhost and I am getting this error "'Response' object has no attribute 'data'
"
Can some one point me to the fix. Many thanks.