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 **many** things (refactor core, add cache, ETA, multiple password file options, ...) #74

Closed
wants to merge 2 commits into from

Conversation

synacktiv-antoineg
Copy link

@synacktiv-antoineg synacktiv-antoineg commented Feb 18, 2024

for another day

@synacktiv-antoineg synacktiv-antoineg closed this by deleting the head repository Feb 18, 2024
@knavesec
Copy link
Owner

knavesec commented Feb 19, 2024

@synacktiv-antoineg Any particular reason you closed this? Looks really interesting on face value

@synacktiv-antoineg
Copy link
Author

@knavesec Yes, the code was not reviewed and field-tested enough internally by my company, so we decided to postpone the PR in order to serve better quality code. I will re-open it soon

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.

2 participants