Enzo MartellucciandClaude Sonnet 5 192ddf9a6d fix(dataset): let OAuth2RedirectError survive DatasetRestApi.post ...

msyavuz's review noted that CreateDatasetCommand's OAuth2RedirectError
re-raise doesn't survive the API layer: DatasetRestApi.post is wrapped
in flask-appbuilder's @safe, which catches any uncaught exception and
flattens it into an opaque 500, dropping the url/tab_id extras the
frontend needs to start the OAuth2 dance. Mirror the pattern already
used by DatabaseRestApi (no @safe + explicit except OAuth2RedirectError:
raise) so the app-level SupersetErrorException handler can build the
correct 403 response instead.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

2026-08-28 11:19:18 +02:00

..

common

refactor: update Connection.execute() to use queries with text() (#40277)

2026-06-22 15:36:15 -07:00

consts

…

example_data

refactor: replace Engine.execute() with Connection.execute() (#41917)

2026-07-13 09:40:33 -07:00

fixtures

…

integration_tests

fix(security): surface extra_editors in dashboard/chart lists (#43392)

2026-08-24 11:49:29 +02:00

unit_tests

fix(dataset): let OAuth2RedirectError survive DatasetRestApi.post

2026-08-28 11:19:18 +02:00

__init__.py

…

conftest.py

…