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

bug: getParentGlue #2

Closed
f1yegor opened this issue May 6, 2017 · 2 comments
Closed

bug: getParentGlue #2

f1yegor opened this issue May 6, 2017 · 2 comments

Comments

@f1yegor
Copy link

f1yegor commented May 6, 2017

installed now
go get github.com/42wim/dt

dt yandex.ru                                                                                                                                           NS             |IP              |LOC |ASN     |ISP        |rtt         |Serial     |DNSSEC   |ValidFrom |ValidUntil
ns2.yandex.ru. |93.158.134.1    |RU  |AS13238 |YANDEX, RU |32.958385ms |2017050606 |disabled |          |
ns2.yandex.ru. |2a02:6b8:0:1::1 |RU  |AS13238 |YANDEX, RU |error       |error      |error    |
ns1.yandex.ru. |213.180.193.1   |RU  |AS13238 |YANDEX, RU |31.533485ms |2017050606 |invalid  |a long while ago |a long while ago
ns1.yandex.ru. |2a02:6b8::1     |RU  |AS13238 |YANDEX, RU |error       |error      |error    |
panic: runtime error: index out of range

goroutine 1 [running]:
main.getParentGlue(0x7ffee887b434, 0x9, 0xd, 0xc4200c39a0, 0x2, 0x2, 0x0)
        /home/f1/media/data/projects/gorepo/src/github.com/42wim/dt/glue.go:93 +0x3f0
main.(*Glue).CheckParent(0xc42005fd30, 0x7ffee887b434, 0x9, 0xc420160240, 0xc420168000, 0x40, 0x0, 0x0, 0x0)
        /home/f1/media/data/projects/gorepo/src/github.com/42wim/dt/glue.go:16 +0x39
main.(*Glue).CreateReport(0xc42005fd30, 0x7ffee887b434, 0x9)
        /home/f1/media/data/projects/gorepo/src/github.com/42wim/dt/glue.go:38 +0xb7
main.main()
        /home/f1/media/data/projects/gorepo/src/github.com/42wim/dt/main.go:239 +0x70f
@42wim
Copy link
Owner

42wim commented May 6, 2017

Thanks for reporting!
Error handling needs improvement.
Could you do a go get -u github.com/42wim/dt and see if it's fixed for you ?

@f1yegor
Copy link
Author

f1yegor commented May 6, 2017

It's always good to check errors) Fixed. Thanks for quick fix.

@f1yegor f1yegor closed this as completed May 6, 2017
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

No branches or pull requests

2 participants