fix(embedded): Guest Role Permissions

[sinhashubham95]

SUMMARY

This fixed the fetching of roles over a guest token. Using this the embedded dashboard can get the roles associated with the guest token.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

curl --location --request GET 'http://localhost:8080/api/v1/me/roles/'
--header 'accept: application/json'
--header 'X-guesttoken: {guest_token}'

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

[sinhashubham95]

@geido please check.

[codecov]

Codecov Report

Merging #21208 (6c63d71) into master (c61a507) will decrease coverage by 0.00%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master   #21208      +/-   ##
==========================================
- Coverage   66.40%   66.40%   -0.01%     
==========================================
  Files        1783     1783              
  Lines       68084    68087       +3     
  Branches     7261     7261              
==========================================
+ Hits        45214    45215       +1     
- Misses      21005    21007       +2     
  Partials     1865     1865              

Flag	Coverage Δ
hive	53.11% <0.00%> (-0.01%)	⬇️
mysql	81.00% <33.33%> (-0.01%)	⬇️
postgres	81.05% <33.33%> (-0.01%)	⬇️
presto	53.01% <0.00%> (-0.01%)	⬇️
python	81.53% <33.33%> (-0.01%)	⬇️
sqlite	79.57% <33.33%> (-0.01%)	⬇️
unit	50.67% <0.00%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
superset/security/manager.py	95.07% <33.33%> (-0.30%)	⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[sinhashubham95]

@geido can you please check?

[sinhashubham95]

@villebro @jinghua-qa can you please check this pr?

[sinhashubham95]

@cccs-tom @dpgaspar @eschutho can you please review this pr?

[sinhashubham95]

@bkyryliuk can you please check this pull request as you recently changed this file?

[sinhashubham95]

@geido @hughhhh @eschutho @villebro @jinghua-qa @bkyryliuk @cccs-tom @dpgaspar can you please check this PR?

[hgrami]

Thanks for addressing this, I'm trying to implement the guest token and I'm facing the same issue. This approach is great! Hope someone reviews this PR soon.

[villebro]

I started CI, will review soon

[villebro]

LGTM, thanks for fixing this!