-
Notifications
You must be signed in to change notification settings - Fork 207
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
feat: extract-base64
cmd
#1522
feat: extract-base64
cmd
#1522
Conversation
help
|
@fukusuket Thanks so much! Looking really good, just a few small things:
|
@Shirofune-Security @YamatoSecurity
Sorry ... This is a correct Base64 string, so there is no way around this FP at this time :( |
@Shirofune-Security @YamatoSecurity
Not a complete workaround, but by adding a simple string match, I was able to exclude False Positive! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fukusuket LGTM! Thank you!
What Changed
extract-base64
command #1512Evidence
Integration-Test
I would appreciate it if you could check it out when you have time🙏