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

Remove bad references (dead links) #6139

Merged
merged 4 commits into from Oct 29, 2015
Merged

Conversation

wchen-r7
Copy link
Contributor

These links are no longer available (404). They are dead links.

These bad links were originally picked up by tools/modules/module_reference.rb, and then manually verified one by one while I was updating the modules.

  • To verify, you can manually try the links.

These links are no longer available. They are dead links.
@@ -46,8 +46,7 @@ def initialize(info = {})
'References' => [
[ 'CVE', '2015-0964' ], # XSS vulnerability
[ 'CVE', '2015-0965' ], # CSRF vulnerability
[ 'CVE', '2015-0966' ], # "techician/yZgO8Bvj" web interface backdoor
[ 'URL', 'https://community.rapid7.com/rapid7_blogpostdetail?id=a111400000AanBs' ] # Original disclosure
Copy link
Contributor

Choose a reason for hiding this comment

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

@wchen-r7
Copy link
Contributor Author

Ok, cool I will update them. Thanks!

@jlee-r7
Copy link
Contributor

jlee-r7 commented Oct 28, 2015

For the record, I much prefer to see commas after the last element of Arrays so diffs like this aren't so big and are easier to read.

@wchen-r7
Copy link
Contributor Author

All links mentioned above are updated. Thanks.

@lsato-r7 lsato-r7 self-assigned this Oct 29, 2015
@lsato-r7 lsato-r7 merged commit 95920b7 into rapid7:master Oct 29, 2015
lsato-r7 added a commit that referenced this pull request Oct 29, 2015
@wchen-r7 wchen-r7 deleted the bad_links branch August 22, 2016 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants