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

BeVigil Source Added #1180

Merged
merged 4 commits into from
Aug 13, 2022
Merged

BeVigil Source Added #1180

merged 4 commits into from
Aug 13, 2022

Conversation

alt-glitch
Copy link
Contributor

BeVigil scans mobile applications and makes OSINT data available for security researchers.
Let me know if the code can be improved a bit :))

@L1ghtn1ng
Copy link
Collaborator

@alt-glitch Thank your for your request, you have included your api key so that needs removing and your will want to revoke that key as its now compromised due to the accidental inclusion of it. You will want to remove it and see if if you can remove it from git history. Unfortunately I do not know how to do that so cannot help on that one.

@alt-glitch
Copy link
Contributor Author

Hi
Yes I think I mistakenly saw the api_keys.yaml file in the .gitignore so thought it wouldn't get added. I realised my mistake and have revoked the API key. Thank you for the heads up!

Copy link
Collaborator

@L1ghtn1ng L1ghtn1ng left a comment

Choose a reason for hiding this comment

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

Only a few things that need slightly changing but looks good 👍

theHarvester/discovery/bevigil.py Outdated Show resolved Hide resolved
theHarvester/discovery/bevigil.py Outdated Show resolved Hide resolved
alt-glitch and others added 2 commits August 14, 2022 00:04
Co-authored-by: J.Townsend <L1ghtn1ng@users.noreply.github.com>
Co-authored-by: J.Townsend <L1ghtn1ng@users.noreply.github.com>
@L1ghtn1ng L1ghtn1ng merged commit 0b65a63 into laramies:master Aug 13, 2022
@L1ghtn1ng
Copy link
Collaborator

Thank you for adding this 👍

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