Skip to content

Support URIBLs#11

Open
theel0ja wants to merge 1 commit intosilverwind:masterfrom
theel0ja:uribl
Open

Support URIBLs#11
theel0ja wants to merge 1 commit intosilverwind:masterfrom
theel0ja:uribl

Conversation

@theel0ja
Copy link
Copy Markdown

@theel0ja theel0ja commented Dec 5, 2019

Fixes #10

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

includeTxt is broken, though.

I guess it's this:

dnsbl/index.js

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

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.

Support URIBLs

1 participant