search

mehrdadrad / mylg

Network Diagnostic Tool
http://mylg.io
MIT License
2.7k stars 232 forks source link

Panic Error #78

Open bacherol opened 7 years ago

bacherol commented 7 years ago

Hello.

Anyone with this error? Just logged in and i have error.

================================================= 

            _ __ _  _| |  / __|
           | '  \ || | |_| (_ |
           |_|_|_\_, |____\___|
                  |__/          

             My Looking Glass
       Free Network Diagnostic Tool
              http://mylg.io

================== myLG v0.2.6 ================== local> local> local> panic: runtime error: index out of range

goroutine 7 [running]: panic(0x905420, 0xc42000c0d0) /usr/local/go/src/runtime/panic.go:500 +0x1a1 github.com/mehrdadrad/mylg/ns.fetchNSHosts(0x0, 0x0, 0x0) /home/vagrant/golang/src/github.com/mehrdadrad/mylg/ns/ns.go:357 +0x98d github.com/mehrdadrad/mylg/ns.(*Request).Init(0xc42004a770) /home/vagrant/golang/src/github.com/mehrdadrad/mylg/ns/ns.go:99 +0x7f created by main.init.1 /home/vagrant/golang/src/github.com/mehrdadrad/mylg/mylg.go:104 +0x125 axtelecom@laboratorio:~/pacotes$

Regards, Leandro de Lima Camargo

mehrdadrad commented 7 years ago

@llcamargo i fixed it today, if you get latest code and compile you shouldn't get this error. please confirm. I'll try to make packages (mac/ubuntu ....)

bacherol commented 7 years ago

I using MAC but i don't know how to compile. I'll search how to. Thanks.

bacherol commented 7 years ago

@mehrdadrad I just ran the command: brew install homebrew/dupes/libpcap And works again! Is it right?

Thanks.

mehrdadrad commented 7 years ago

@llcamargo that's not related. the root cause of the issue related to a CSV file that it downloads from a 3rd party website and there is a uncompleted line that myLG (not latest code) doesn't validate and causes crash. the latest code fixed also I asked the 3rd party website if they can fix it.

mehrdadrad commented 7 years ago

@llcamargo I'm seeing the issue fixed at CSV file (3rd party website). so it should be worked w/ old version as well.

FHRNet commented 7 years ago

I'm seeing those exact errors quite frequently, even though the CSV file can be opened in browser with no problems.

mehrdadrad commented 7 years ago

@FHRNet what's the myLG version?

FHRNet commented 7 years ago

v0.2.6 on Mac. Happens on Linux too.

It’s currently working. But as I said, sometimes it works, sometimes it throws that error.

On May 18, 2017, at 6:52 PM, Mehrdad Arshad Rad notifications@github.com wrote:

@FHRNet https://github.com/fhrnet what's the myLG version?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mehrdadrad/mylg/issues/78#issuecomment-302469298, or mute the thread https://github.com/notifications/unsubscribe-auth/AGMs7Y3_5X4dnFuDXH7ixKEUT-hJEN3kks5r7HdagaJpZM4Mi427.

mehrdadrad commented 7 years ago

@FHRNet I see but no idea why a field that myLG expects to have is not exists for few lines at CSV file, it fixed in v0.2.7 to skip those lines but the Mac package is not available for latest version you have to compile it w/ the latest version code from github.

FHRNet commented 7 years ago

Here's the CSV from the time it didn't work, maybe it'll help: https://expirebox.com/download/bf55d5136a1d82a50e6dac8112471f9f.html

mehrdadrad commented 7 years ago

@FHRNet thank for ur help but already it fixed in the new version. try to compile from latest code

luckman212 commented 7 years ago

I am getting the errors too. Any instructions for compiling for macOS?

edit: nevermind, I found it in the README: Just need to have go installed. Then:

go get github.com/mehrdadrad/mylg
cd $GOPATH/src/github.com/mehrdadrad/mylg/
go build mylg.go