fix: check embedded feature flag in request loader

[lilykuang]

SUMMARY

• move the feature flag check to request loader

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #18628 (e508457) into master (51f716f) will increase coverage by 0.00%.
The diff coverage is 83.33%.

@@           Coverage Diff           @@
##           master   #18628   +/-   ##
=======================================
  Coverage   66.31%   66.31%           
=======================================
  Files        1595     1595           
  Lines       62596    62599    +3     
  Branches     6297     6297           
=======================================
+ Hits        41510    41513    +3     
  Misses      19440    19440           
  Partials     1646     1646           

Flag	Coverage Δ
hive	52.14% <83.33%> (+<0.01%)	⬆️
javascript	51.33% <ø> (ø)
mysql	81.26% <83.33%> (+<0.01%)	⬆️
postgres	81.31% <83.33%> (+<0.01%)	⬆️
presto	51.98% <83.33%> (+<0.01%)	⬆️
python	81.75% <83.33%> (+<0.01%)	⬆️
sqlite	81.01% <83.33%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
superset/security/manager.py	92.30% <83.33%> (+0.04%)	⬆️

---

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 51f716f...e508457. Read the comment docs.