feat: add secure credential storage via OS keyring#66
Open
mquandalle wants to merge 1 commit intoazerpas:mainfrom
Open
feat: add secure credential storage via OS keyring#66mquandalle wants to merge 1 commit intoazerpas:mainfrom
mquandalle wants to merge 1 commit intoazerpas:mainfrom
Conversation
Add support for storing passwords securely using the operating system's native keyring (macOS Keychain, Windows Credential Manager, Linux Secret Service). This enables automated/cron usage without interactive prompts. New CLI options: - `bourso config --save-password`: Store password in OS keyring - `bourso config --delete-password`: Remove password from keyring - `bourso config --show`: Display current configuration status Password resolution now follows a fallback chain: 1. OS keyring (if available) 2. settings.json (backward compatibility, with migration warning) 3. Interactive prompt (or error if non-interactive) Closes azerpas#63 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Owner
|
Once again, awesome thanks! I'll check this out after #67 :) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Asked @claude to give a shot at #63. I tested locally and it works as expected, at least before I was blocked by #61
Summary
New Commands
Password Resolution
Fallback chain:
Platform Support
Closes #63
🤖 Generated with Claude Code