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

Panic Error / Report Issue #57

Open
shaikhyaser opened this issue Dec 29, 2022 · 4 comments
Open

Panic Error / Report Issue #57

shaikhyaser opened this issue Dec 29, 2022 · 4 comments

Comments

@shaikhyaser
Copy link

shaikhyaser commented Dec 29, 2022

Describe the bug
Getting Panic error while running / analyzing more than 100 pages

To Reproduce
Steps to reproduce the behavior:
echo ""xxxxxxxxx.com"" | ./git-hound --dig-files --dig-commits --threads 100

Error

panic: runtime error: index out of range [1] with length 0

goroutine 1 [running]:
github.com/tillson/git-hound/internal/app.SearchGitHub({0xc000160d38, 0x13}, {0x64, {0x0, 0x0}, {{0x0, 0x0}, {0x0, 0x0}, 0x0, ...}}, ...)
        /Users/tillson/git-hound/internal/app/search.go:210 +0x1138
github.com/tillson/git-hound/internal/app.Search({0xc000160d38, 0x13}, 0x2?)
        /Users/tillson/git-hound/internal/app/search.go:66 +0x1c5
github.com/tillson/git-hound/cmd.glob..func1(0xf0fa40?, {0xc000138480, 0x0, 0x6?})
        /Users/tillson/git-hound/cmd/root.go:96 +0x6ee
github.com/spf13/cobra.(*Command).execute(0xf0fa40, {0xc00013c010, 0x6, 0x6})
        /Users/tillson/go/pkg/mod/github.com/spf13/[email protected]/command.go:860 +0x663
github.com/spf13/cobra.(*Command).ExecuteC(0xf0fa40)
        /Users/tillson/go/pkg/mod/github.com/spf13/[email protected]/command.go:974 +0x3b4
github.com/spf13/cobra.(*Command).Execute(...)
        /Users/tillson/go/pkg/mod/github.com/spf13/[email protected]/command.go:902
github.com/tillson/git-hound/cmd.Execute()
        /Users/tillson/git-hound/cmd/root.go:124 +0x25
main.main()
        /Users/tillson/git-hound/main.go:10 +0x17
@tillson
Copy link
Owner

tillson commented Dec 30, 2022

Is this on 1.6.2 (latest commit)? I tried to fix this yesterday in the new version

@shaikhyaser
Copy link
Author

I didn't notice the bug fixed in the latest commit. It's fixed.

In new commit 1.6.2, Is the report getting generated for searches over 80 pages? I tried couple of domains and it hangs.

@tillson
Copy link
Owner

tillson commented Dec 30, 2022

Let me look into that... the code has gotten very messy recently due to GitHub having two different search UIs that are presented to different users.

@shaikhyaser
Copy link
Author

Thanks @tillson

@shaikhyaser shaikhyaser changed the title Panic Error Panic Error / Report Issue Dec 30, 2022
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