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

Df 3668 url validator #70

Merged
merged 3 commits into from Jan 13, 2020
Merged

Df 3668 url validator #70

merged 3 commits into from Jan 13, 2020

Conversation

tylerslijboom-r7
Copy link
Contributor

@tylerslijboom-r7 tylerslijboom-r7 commented Dec 30, 2019

Description

Searches files for URLs, and then runs a HTTP HEAD request to test that the URL exists. Reports back any that give an HTTP return code in the 400s.

https://issues.corp.rapid7.com/browse/DF-3668

Testing

Tested on the following plugins: mimecast, jenkins, wordpress, att_cybersecurity_alienvault_otx, threatstack, thehive, trufflehog, digitalocean, docker_engine, dumbno, netmiko and a few others. The attached file below highlights the 3 cases in order:

URLS that exist and are valid
return code 403 URLs
rc 404 URLs

url-validator-script.txt

ScreenShot sample:

Screen Shot 2020-01-03 at 4 00 19 PM

@tylerslijboom-r7 tylerslijboom-r7 added WIP and removed WIP labels Dec 30, 2019
icon_validator/rules/url_validator.py Show resolved Hide resolved
icon_validator/rules/url_validator.py Show resolved Hide resolved
icon_validator/rules/url_validator.py Outdated Show resolved Hide resolved
@lmilby-r7
Copy link
Contributor

Could we also get a screen shot of the validator running.

@jschipp-r7
Copy link
Contributor

Could we also get a screen shot of the validator running.

I prefer text output that I can copy and paste from but along with Luke's same reasoning.

Copy link
Contributor Author

@tylerslijboom-r7 tylerslijboom-r7 left a comment

Choose a reason for hiding this comment

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

Could we also get a screen shot of the validator running.

There is a txt file in the PR Testing section that is a terminal script output...

icon_validator/rules/url_validator.py Show resolved Hide resolved
icon_validator/rules/url_validator.py Show resolved Hide resolved
@tylerslijboom-r7 tylerslijboom-r7 merged commit 3c71ba9 into master Jan 13, 2020
@tylerslijboom-r7 tylerslijboom-r7 deleted the df-3668-url-validator branch January 13, 2020 17:35
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.

None yet

5 participants