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 https prefix to module URL references #19099

Merged

Conversation

sjanusz-r7
Copy link
Contributor

Quick win, this PR adds in the https prefix to module references that were missing them (only two), similar to the PR here: #18923

Fixes an issue where the module couldn't be selected in Pro.

Testing

require 'URI'
ref = 'www.geutebrueck.com'
URI.parse(ref).host
ref = 'https://www.geutebrueck.com'
URI.parse(ref).host

Verification

  • Run the code snippet above
  • Ensure the host is not nil

@sjanusz-r7 sjanusz-r7 added the rn-no-release-notes no release notes label Apr 17, 2024
@adfoster-r7
Copy link
Contributor

cc @cgranleese-r7 - this should probably be added to the Metasploit validation script - #17804

@adfoster-r7 adfoster-r7 merged commit 2116457 into rapid7:master Apr 17, 2024
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn-no-release-notes no release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants