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.

Sign up for GitHub

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

Support URIBLs #11

Open

theel0ja wants to merge 1 commit into silverwind:master from theel0ja:uribl

theel0ja commented Dec 5, 2019 •

edited

Loading

Fixes #10

Code I used for dev purposes: https://gist.github.com/theel0ja/28f9b2c6fbe3d6d5fc4547680b211fcc

includeTxt is broken, though.

I guess it's this:

Lines 35 to 37 in 5c340a5

 } catch (err) { 

 return opts.includeTxt ? {listed: false, txt: []} : false; 

 }

It returns "false" even if the A record is returned, if no TXT record exists on the URIBL


Support URIBLs

1b3726b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment