Skip to content

Conversation

@pvanderknyff
Copy link
Contributor

Makes two changes:

  • If the field is marked secure, and the field name is not password, fail packager validation. These connectors will not work in Tableau.
  • If the field has token, password, or secret in it, and is not marked secure, fail the validation, since that suggests that field would leak customer secrets. Open to making this a non-failing warning, but given we already have a warning I'd kind of rather err on the side of caution

Also adds two new test cases to cover these scenarios.

For work item W-19338086

@pvanderknyff pvanderknyff merged commit cdf746b into master Aug 29, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants