Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Error parsing multiple ASNs from reverse lookup #19

Closed
rshipp opened this issue Feb 22, 2018 · 5 comments
Closed

Error parsing multiple ASNs from reverse lookup #19

rshipp opened this issue Feb 22, 2018 · 5 comments

Comments

@rshipp
Copy link

rshipp commented Feb 22, 2018

Looks like a bug when whatever ASN lookup you do returns multiple numbers:

2018/02/22 09:43:48 Error doing reverse lookup on %v: %v 103.235.224.237 Could not parse ASN (4808 23724 56282): %!s(MISSING): strconv.Atoi: parsing "4808 23724 56282": invalid syntax
2018/02/22 09:43:52 Error doing reverse lookup on %v: %v 103.75.156.237 Could not parse ASN (134877 135216): %!s(MISSING): strconv.Atoi: parsing "134877 135216": invalid syntax
2018/02/22 09:43:59 Error doing reverse lookup on %v: %v 106.75.101.69 Could not parse ASN (4808 23724): %!s(MISSING): strconv.Atoi: parsing "4808 23724": invalid syntax
2018/02/22 09:43:59 Error doing reverse lookup on %v: %v 106.75.18.132 Could not parse ASN (4808 23724): %!s(MISSING): strconv.Atoi: parsing "4808 23724": invalid syntax

These IPs are from the CI Army blacklist, http://cinsscore.com/list/ci-badguys.txt, there's about 160 IPs out of the 15k on the list that trigger the error. GOSINT version is latest master as of today, on Ubuntu 16.04.2 LTS, installed through the bash install script.

@blueteam1
Copy link
Collaborator

Hey Ryan,

Nice catch, I've seen this in our use too. This is actually a bug with the library we use for that which is https://github.com/ammario/ipisp. I haven't dug deeper yet to work on a fix, but so far it doesn't seem to cause much issue with it operating with GOSINT.

Thanks

@rshipp
Copy link
Author

rshipp commented Feb 22, 2018

Got it, thanks. 👍 I'll close this issue then. Should I open one upstream or are they already aware of the issue?

@rshipp rshipp closed this as completed Feb 22, 2018
@blueteam1
Copy link
Collaborator

Hey,

We have not notified them yet. If you want to do that I'm sure they'd appreciate the info so the bug can be tracked (and so would we :) ).

Thanks!

@rshipp
Copy link
Author

rshipp commented Feb 23, 2018

Upstream fixed in ammario/ipisp#6

@blueteam1
Copy link
Collaborator

Wow, you're awesome. Thanks for tracking that down and taking care of it.

Cheers

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants