chore: Bump FAB to 3.4.0

[kamalkeshavani-aiinside]

SUMMARY

Bumping FAB to latest 3.4.0
Changes:

• chore: pin down WTForms dpgaspar/Flask-AppBuilder#1735 [Daniel Vaz Gaspar]
• fix: ModuleNotFoundError from wtforms 3.0.0 dpgaspar/Flask-AppBuilder#1733 [Ke Zhu]
• fix: add .env for docker-compose dpgaspar/Flask-AppBuilder#1728 [Daniel Vaz Gaspar]
• fix: OAuth login flow dpgaspar/Flask-AppBuilder#1707 [Daniel Vaz Gaspar]

[codecov]

Codecov Report

Merging #17420 (c6291dd) into master (ffa55f7) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #17420      +/-   ##
==========================================
- Coverage   77.01%   76.80%   -0.22%     
==========================================
  Files        1040     1040              
  Lines       56077    56077              
  Branches     7738     7738              
==========================================
- Hits        43190    43069     -121     
- Misses      12629    12750     +121     
  Partials      258      258              

Flag	Coverage Δ
hive	?
mysql	81.89% <ø> (ø)
postgres	81.90% <ø> (ø)
presto	?
python	81.98% <ø> (-0.42%)	⬇️
sqlite	81.57% <ø> (ø)

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

Impacted Files	Coverage Δ
superset/db_engines/hive.py	0.00% <0.00%> (-85.19%)	⬇️
superset/db_engine_specs/hive.py	69.49% <0.00%> (-16.99%)	⬇️
superset/db_engine_specs/presto.py	83.47% <0.00%> (-6.91%)	⬇️
superset/views/database/mixins.py	81.03% <0.00%> (-1.73%)	⬇️
superset/connectors/sqla/models.py	86.35% <0.00%> (-1.62%)	⬇️
superset/models/core.py	89.26% <0.00%> (-0.74%)	⬇️
superset/db_engine_specs/base.py	88.20% <0.00%> (-0.39%)	⬇️
superset/utils/core.py	89.98% <0.00%> (-0.13%)	⬇️

---

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 ffa55f7...c6291dd. Read the comment docs.

[dpgaspar]

LGTM, Thank you!

[kamalkeshavani-aiinside]

@junlincc Please tag this for v1.4

[villebro]

@kamalkeshavani-aiinside FYI I updated the description to link to the PRs on the FAB GH repo (previously the numbered tags were pointing to issues on this repo)

[usamaB]

I tried the 1.4.0rc1 but the login/logout flow hasn't been fixed. I'm unable to logout. It just automatically logs in and brings me to the welcome screen. And the login flow is incorrect as well. If you have a single provider it doesn't show login UI but takes directly to enter email/pass UI(google in my case).

And if I do grep for Flask-AppBuilder in the pod shell(I'm using helm) it shows it has v3.3.0 and not v.3.4.0.

[dpgaspar]

@usamaB with FAB 3.3.0 that behaviour is expected, have you rebuilt your container with this commit included?