Skip to content

[BREAKING] Python: Refine SecretString handling - #8

Closed
eavanvalkenburg wants to merge 1 commit into
mainfrom
secretstring-masking
Closed

eavanvalkenburg wants to merge 1 commit into
mainfrom
secretstring-masking

Conversation

@eavanvalkenburg

Copy link
Copy Markdown
Owner

Motivation & Context

Make settings-value behavior more consistent across Python formatting and provider integrations.

Description & Review Guide

  • What are the major changes? Refine SecretString behavior and align provider credential handling and type annotations.
  • What is the impact of these changes? SecretString is now a value wrapper rather than a str subclass. Callers needing a plain string should use get_secret_value(). Plain-string settings inputs remain supported.
  • What do you want reviewers to focus on? Consistent settings behavior and compatibility at provider boundaries.

Related Issue

N/A.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • All unit tests pass, and I have added new tests where possible
  • The PR follows the Contribution Guidelines
  • This PR is linked to an issue and there is no other open PR for this issue (see Related Issue above).
  • This is not a breaking change. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

This branch had an error being deployed

1 failed deployment
github-app-auth — 4ceefd0b Deployed Sep 7, 2026 by eavanvalkenburg via team_check #6
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