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

Malformed URL when retrieving files #246

Open
JonZeolla opened this issue Nov 23, 2016 · 1 comment
Open

Malformed URL when retrieving files #246

JonZeolla opened this issue Nov 23, 2016 · 1 comment
Labels
bug Known bug in the code. Enhancement
Milestone

Comments

@JonZeolla
Copy link
Contributor

JonZeolla commented Nov 23, 2016

There appears to be an issue handling the oinkcode where it doesn't add a / appropriately between it and the domain.

Use of uninitialized value $Snort_path in -B at ./pulledpork.pl line 1773.
Checking latest MD5 for emerging.rules.tar.gz....
A 403 error occurred, please wait for the 15 minute timeout
to expire before trying again or specify the -n runtime switch
You may also wish to verfiy your oinkcode, tarball name, and other configuration options
Error 403 when fetching http://rules.emergingthreats.netopen/suricata->3.0.1/emerging.rules.tar.gz.md5 at ./pulledpork.pl line 534.
main::md5file('open', 'emerging.rules.tar.gz', '/tmp/', >'http://rules.emergingthreats.netopen/suricata-3.0.1/') called at ./pulledpork.pl line 2006

In my conf I have

rule_url=http://rules.emergingthreats.net|emerging.rules.tar.gz|open

If I change that to

rule_url=http://rules.emergingthreats.net|emerging.rules.tar.gz|/open

It appears to work properly.

@JonZeolla
Copy link
Contributor Author

This also works when you use:

rule_url=http://rules.emergingthreats.net/|emerging.rules.tar.gz|open

I don't believe that / used to be required, but now it is. Perhaps it's worth detecting if the domain ends in /, and if not, add it.

@shirkdog shirkdog added bug Known bug in the code. Enhancement labels Dec 6, 2017
@shirkdog shirkdog added this to the 0.8.0 milestone Dec 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Known bug in the code. Enhancement
Projects
None yet
Development

No branches or pull requests

2 participants