Skip to content

fix: mark dsn as isSecret in config schema#48

Merged
btipling merged 1 commit into
mainfrom
bt/secret-audit-postgresql
Jun 15, 2026
Merged

fix: mark dsn as isSecret in config schema#48
btipling merged 1 commit into
mainfrom
bt/secret-audit-postgresql

Conversation

@btipling

Copy link
Copy Markdown
Contributor

Summary

Marks credential field(s) dsn (the DSN connection string embeds the database password) as secret in this connector's config so the credential is stored and handled as a secret rather than plaintext.

This connector defines its config in Go (field package), not a config.yaml. Secret-ness is the field.WithIsSecret(true) option on the field — the authoritative source. There is no committed config_schema.json in this repo, and the generated conf.gen.go is a mapstructure struct that carries no secret metadata, so the one-line config.go change is the complete fix and a CI regen produces no additional diff (no race).

BREAKING

BREAKING: adding isSecret: true to these fields changes how existing configurations are stored. Customers with existing connector configurations will need to re-enter credentials after this change is deployed.

Audit

Part of the connector config secret-ness audit (phase 13). Found via systematic review of credential fields lacking isSecret.

Do not merge — left open for review.

@btipling btipling requested a review from a team June 13, 2026 02:29
@github-actions

Copy link
Copy Markdown
Contributor

Connector PR Review: fix: mark dsn as isSecret in config schema

Blocking Issues: 0 | Suggestions: 0 | Threads Resolved: 0
Review mode: full
View review run

Review Summary

Single-line change adds field.WithIsSecret(true) to the dsn config field in pkg/config/config.go. The DSN embeds database credentials, so marking it as secret is correct and necessary. The breaking change for existing configurations is clearly documented in the PR description. No issues found.

Security Issues

None found.

Correctness Issues

None found.

Suggestions

None.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No blocking issues found.

@btipling btipling merged commit 649c611 into main Jun 15, 2026
5 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.

1 participant