Skip to content

Conversation

@dpgaspar
Copy link
Member

@dpgaspar dpgaspar commented Apr 26, 2021

SUMMARY

New FAB 3.2.3 has 2 new security manager APIs, get_user_permissions and get_role_permissions

These APIs generate a performant query to the DB (eager loading) and take into account builtin roles and public users

note:
FAB to 3.2.3 with the following change log since 3.2.1:

Improvements and Bug fixes on 3.2.3

fix: improve performance for get role permissions (#1624) [Daniel Gaspar]
feat: get user permissions API (#1620) [Daniel Gaspar]
fix: Ignore LDAP search referrals (#1602) [Fred Thomsen]
fix: relax AzureAD mandatory fields (#1608) [hyunjong.lee]

Improvements and Bug fixes on 3.2.2

docs: fix, errors in BaseModelView docstring (#1591) [Xiaodong DENG]
fix: load user info for okta (#1589) [QP Hou]

Improvements and Bug fixes on 3.2.1

docs: improve contributing run single test (#1579) [Daniel Vaz Gaspar]
fix: sqlalchemy 1.4.0 breaking changes (#1586) [Daniel Vaz Gaspar]

ADDITIONAL INFORMATION

  • Has associated issue:
  • 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
Copy link

codecov bot commented Apr 26, 2021

Codecov Report

Merging #14348 (9d108a2) into master (437be4b) will decrease coverage by 0.15%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14348      +/-   ##
==========================================
- Coverage   77.06%   76.90%   -0.16%     
==========================================
  Files         954      954              
  Lines       48141    48139       -2     
  Branches     5987     5987              
==========================================
- Hits        37098    37020      -78     
- Misses      10846    10922      +76     
  Partials      197      197              
Flag Coverage Δ
hive ?
mysql 81.04% <85.71%> (-0.01%) ⬇️
postgres 81.07% <85.71%> (-0.01%) ⬇️
python 81.15% <85.71%> (-0.30%) ⬇️
sqlite 80.68% <85.71%> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
superset/views/utils.py 83.66% <85.71%> (-0.13%) ⬇️
superset/db_engines/hive.py 0.00% <0.00%> (-82.15%) ⬇️
superset/db_engine_specs/hive.py 70.80% <0.00%> (-16.80%) ⬇️
superset/views/database/mixins.py 81.03% <0.00%> (-1.73%) ⬇️
superset/db_engine_specs/presto.py 83.36% <0.00%> (-1.06%) ⬇️
superset/db_engine_specs/base.py 87.68% <0.00%> (-0.44%) ⬇️
superset/connectors/sqla/models.py 88.37% <0.00%> (-0.25%) ⬇️
superset/utils/core.py 88.65% <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 437be4b...9d108a2. Read the comment docs.

@amitmiran137
Copy link
Member

could we wait with this one after 1.2?

@dpgaspar
Copy link
Member Author

could we wait with this one after 1.2?

not really, why?

@nytai nytai merged commit b4e527a into apache:master Apr 30, 2021
@dpgaspar dpgaspar deleted the fix/bootstrap-permissions branch April 30, 2021 18:06
@yousoph
Copy link
Member

yousoph commented Apr 30, 2021

🏷️ 2021.17

amitmiran137 pushed a commit to nielsen-oss/superset that referenced this pull request May 3, 2021
* master: (38 commits)
  refactor(native-filters): allow cascading only for filter_select (apache#14441)
  test(maximize-chart): Add tests to maximize chart action (apache#14371)
  fix: fixing mysql error message (apache#14416)
  feat: Logic added to limiting factor column in Query model (apache#13521)
  change relationship (apache#14435)
  fix: bootstrap data permissions (apache#14348)
  fix: parse simple string error message values (apache#14360)
  chore: add stack trace to all calls of logger.error (apache#14382)
  update README with new docs and recordings (apache#14432)
  Renamed impyla from implya in impala.mdx and Renamed PIP package impyla from impala in index.mdx (apache#14425)
  fix(native-filters): fix filter scope error (apache#14426)
  feat: Adding limiting_factor column to Query model (apache#14234)
  feat: Add etag caching to dashboard APIs (apache#14357)
  chore: Moves Card to the components folder (apache#14139)
  feat: Dynamic imports for the Icons component (apache#14318)
  feat: Support env vars configuration for WebSocket server (apache#14398)
  fix: SQLLab role permissions (apache#14372)
  fix(native-filters): always show filters without dataset (apache#14409)
  fix error getting partitionQuery from table.partition (apache#14369)
  refactor: Boostrap to AntD - Tabs (apache#14048)
  ...
henryyeh pushed a commit to preset-io/superset that referenced this pull request May 3, 2021
hughhhh pushed a commit that referenced this pull request May 4, 2021
cccs-RyanS pushed a commit to CybercentreCanada/superset that referenced this pull request Dec 17, 2021
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 29, 2021
cccs-rc pushed a commit to CybercentreCanada/superset that referenced this pull request Mar 6, 2024
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.3.0 First shipped in 1.3.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels preset:2021.17 preset-io size/S 🚢 1.3.0 First shipped in 1.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants