Skip to content

feat(config): bring back secure text fields - #1271

Open
DirectivesMods wants to merge 1 commit into
Polyfrost:v1from
DirectivesMods:v1
Open

DirectivesMods wants to merge 1 commit into
Polyfrost:v1from
DirectivesMods:v1

Conversation

@DirectivesMods

Copy link
Copy Markdown

Description

Brings back @Text(secure = true) from v0, which got lost in the rewrite. It masks the value with * and adds an eye button to reveal it (useful for API keys).

Also adds secureMask, secureShowLast and secureShowChars to customize the mask, e.g. ****-****-****-abcd. Works with KtConfig and the DSL too.

Tested on 1.8.9.

Related Issue(s)

None

Checklist

  • I made a clear description of what was changed
  • I stated why these changes were necessary
  • I updated documentation or said what needs to be updated
  • I made sure these changes are backwards compatible
  • This pull request is for one feature/bug fix

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review 🔄 Running since 2026-10-27T04:29:45.979150Z bd89992 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant