Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented OS keyring storage for credential file password #434

Draft
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

unapproachable
Copy link
Collaborator

This eliminates the need for the password to be entered every time the bot starts. As long as a user doesn't lose control of their physical computer and their local account password, it seems as safe as entering the text at startup and still prevents any copy/paste credential incidents from the community.

…ing the need for the password to be entered every time the bot starts. As long as a user doesn't lose control of their physical computer
Copy link
Collaborator

@DakkJaniels DakkJaniels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks reasonable - if you are all set with it, merge to dev.

@unapproachable unapproachable marked this pull request as draft December 22, 2020 15:08
@i3asm
Copy link

i3asm commented Jul 12, 2021

Can you guys rerun the action auto build to check the why is it not accepting this? the error output is expired.
Thanks.

@DakkJaniels
Copy link
Collaborator

the only thing it tells us if the code doesn't conform to black formatting. It has nothing to do with if the code is functional.

@i3asm
Copy link

i3asm commented Jul 12, 2021

This is the first time I hear about black formatting. I am reading their page right now, TIL!
anyway, thanks for maintaining this project.

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.

3 participants