posit-dev / posit-dev/rsconnect-python
Store API keys in keyring, if available
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 37
- Forks
- 28
- Avg merge
- 1d 3h
- Merged PRs (30d)
- 7
Description
#773 added an optional keyring dependency for storing oauth tokens. We should use that for API keys in rsconnect add too, so that API keys don't live in plain text in a hidden file somewhere
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Review the implementation from #773 and trace the rsconnect add flow where API keys are stored. Confirm how optional keyring support is detected, then verify that API keys use it when available and are not written as plain text when the feature is supported.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- cli, security
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100