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

Feature/config #91

Closed
wants to merge 8 commits into from
Closed

Conversation

jubeaz
Copy link
Contributor

@jubeaz jubeaz commented May 24, 2024

Hello,

I've modified arsenal so that config can be defined by user in their own ~/.arsenal.conf which if not present is a copy of the one found in data.

User can decide to use or not the cheats provided in data/cheats

Finally I've made color configurable.

Regards,

@jubeaz
Copy link
Contributor Author

jubeaz commented May 24, 2024

hello can someone check why I do have this vul in my pull request
2024-05-24_18-24

I've found that https://medium.com/@paul.brabban/handling-cve-2018-20225-1bc348caf6e1

@v1k1ngfr
Copy link
Collaborator

v1k1ngfr commented Aug 5, 2024

Hello @jubeaz,

If a user doesn't want to use the cheats in data/cheats, those cheats can simply be deleted / replaced.

Thanks

@Guilhem7
Copy link
Collaborator

Thanks for the idea, and for your PR, however we will keep arsenal simpler for the configuration part.

@Guilhem7 Guilhem7 closed this Nov 13, 2024
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