Skip to content

💥 Change --password-file behavior#2156

Draft
ChanTsune wants to merge 1 commit intomainfrom
cli/password-read-strategy
Draft

💥 Change --password-file behavior#2156
ChanTsune wants to merge 1 commit intomainfrom
cli/password-read-strategy

Conversation

@ChanTsune
Copy link
Copy Markdown
Owner

Previously, the entire file content was used as the password. Now, only the first non-empty line is read, and trailing newlines are ignored, to align with common tool behavior.

@github-actions github-actions Bot added cli This issue is about cli application break API braking change labels Sep 26, 2025
@ChanTsune ChanTsune force-pushed the cli/password-read-strategy branch from 5e052b7 to cd0e0bc Compare October 13, 2025 13:34
@github-actions github-actions Bot removed the break API braking change label Oct 13, 2025
Previously, the entire file content was used as the password.  Now, only the first non-empty line is read, and trailing newlines are ignored,  to align with common tool behavior.
@ChanTsune ChanTsune force-pushed the cli/password-read-strategy branch from cd0e0bc to 7836e51 Compare November 21, 2025 13:35
@github-actions github-actions Bot added break API braking change and removed break API braking change labels Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli This issue is about cli application

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant