Skip to content

feat(integrations): mark SMTP password and Dependency-Track API key as password#3117

Open
Piskoo wants to merge 2 commits into
chainloop-dev:mainfrom
Piskoo:PFM-5981
Open

feat(integrations): mark SMTP password and Dependency-Track API key as password#3117
Piskoo wants to merge 2 commits into
chainloop-dev:mainfrom
Piskoo:PFM-5981

Conversation

@Piskoo
Copy link
Copy Markdown
Collaborator

@Piskoo Piskoo commented May 14, 2026

Tag the SMTP password and Dependency-Track API key registration fields with format=password.

Tag the SMTP password and Dependency-Track API key registration fields with
format=password so frontends rendering the JSON schema can present them as
masked password inputs.

Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

migmartri
migmartri previously approved these changes May 14, 2026
// The URL of the Dependency-Track instance
InstanceURI string `json:"instanceURI" jsonschema:"format=uri,description=The URL of the Dependency-Track instance"`
APIKey string `json:"apiKey" jsonschema:"description=The API key to use for authentication"`
APIKey string `json:"apiKey" jsonschema:"format=password,description=The API key to use for authentication"`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice, does this updates the generated json schema?

Copy link
Copy Markdown
Collaborator Author

@Piskoo Piskoo May 14, 2026

Choose a reason for hiding this comment

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

yes, it does

Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
@Piskoo Piskoo marked this pull request as ready for review May 14, 2026 16:53
@Piskoo Piskoo requested a review from a team May 14, 2026 16:53
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