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

Add infile to options defaults #84

Closed
qha opened this issue Dec 8, 2021 · 6 comments
Closed

Add infile to options defaults #84

qha opened this issue Dec 8, 2021 · 6 comments

Comments

@qha
Copy link

qha commented Dec 8, 2021

Add infile=None to OPTIONS_DEFAULTS in diceware/config.py to permit configuring custom word lists that are not installed in diceware/wordlists.

@ulif
Copy link
Owner

ulif commented Dec 8, 2021

I do not understand. None is already the default for the infile argument. Could you explain what is not working for you?

@qha
Copy link
Author

qha commented Dec 8, 2021

get_config_dict only reads configuration settings that occur in defaults_dict, i take it that's why setting infile in .diceware.ini had no effect for me unlike other settings.

@ulif
Copy link
Owner

ulif commented Dec 8, 2021

Ok, so you want to use a different wordlist by default? The usual way to do this is putting the list into the local wordlist dir (displayed by --help). Is this an option for you?

@qha
Copy link
Author

qha commented Dec 9, 2021

Sure, that is an option, i just prefer to keep my own configuration where i can write it without elevating my priveliges.

@drebs
Copy link
Contributor

drebs commented Jul 29, 2022

Sure, that is an option, i just prefer to keep my own configuration where i can write it without elevating my priveliges.

How about if, instead, Diceware included ${XDG_DATA_HOME}/diceware/wordlists in its search for available wordlists?

@qha
Copy link
Author

qha commented Jul 29, 2022

That sounds like it would accomplish everything i wished for when opening this ticket.

@ulif ulif closed this as completed in 05565e7 Sep 12, 2024
ulif added a commit that referenced this issue Sep 12, 2024
Follow XDG base file specification for configuration files and to find
additional wordlist files.
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

No branches or pull requests

3 participants