Skip to content

feat: support for import/export masked_encrypted_extra (backend)#38077

Merged
rusackas merged 4 commits into
masterfrom
feat/masked-encrypted-extra-import-export-backend
Mar 4, 2026
Merged

feat: support for import/export masked_encrypted_extra (backend)#38077
rusackas merged 4 commits into
masterfrom
feat/masked-encrypted-extra-import-export-backend

Conversation

@Vitor-Avila
Copy link
Copy Markdown
Contributor

@Vitor-Avila Vitor-Avila commented Feb 18, 2026

SUMMARY

This PR adds support for exporting/importing masked_encrypted_extra info from DB connections. This is useful for:

  • BQ connections
  • GSheets connection
  • OAuth connections
  • Any DB connection that includes sensitive info in secure_extra

Stacked diffs implementation

The feature was split into 3 PRs:

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Videos added to #38078

TESTING INSTRUCTIONS

Test coverage added

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

@Vitor-Avila Vitor-Avila marked this pull request as draft February 18, 2026 20:14
@github-actions github-actions Bot added the api Related to the REST API label Feb 18, 2026
@bito-code-review
Copy link
Copy Markdown
Contributor

bito-code-review Bot commented Feb 18, 2026

Bito Automatic Review Skipped - Branch Excluded

Bito didn't auto-review because the source or target branch is excluded from automatic reviews.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the branch exclusion settings here, or contact your Bito workspace admin at evan@preset.io.

@dosubot dosubot Bot added the change:backend Requires changing the backend label Feb 18, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 18, 2026

Deploy Preview for superset-docs-preview ready!

Name Link
🔨 Latest commit df23c4a
🔍 Latest deploy log https://app.netlify.com/projects/superset-docs-preview/deploys/69961da06f90b900085e25f9
😎 Deploy Preview https://deploy-preview-38077--superset-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread superset/databases/api.py Outdated
Comment thread tests/unit_tests/utils/json_tests.py Outdated
@Vitor-Avila Vitor-Avila marked this pull request as ready for review February 18, 2026 20:33
@dosubot dosubot Bot added the data:connect Namespace | Anything related to db connections / integrations label Feb 18, 2026
@Vitor-Avila Vitor-Avila force-pushed the feat/masked-encrypted-extra-import-export-backend branch from df23c4a to c6258bb Compare February 19, 2026 05:36
Base automatically changed from feat/label-encrypted-fields to master February 23, 2026 16:23
Copy link
Copy Markdown
Member

@betodealmeida betodealmeida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

@rusackas rusackas merged commit 8c9efe5 into master Mar 4, 2026
56 checks passed
@rusackas rusackas deleted the feat/masked-encrypted-extra-import-export-backend branch March 4, 2026 19:26
aminghadersohi pushed a commit to aminghadersohi/superset that referenced this pull request Mar 17, 2026
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Related to the REST API change:backend Requires changing the backend data:connect Namespace | Anything related to db connections / integrations size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants