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

Cryptic errors in some situations #487

Closed
dpordomingo opened this issue Jan 23, 2019 · 0 comments · Fixed by #488
Closed

Cryptic errors in some situations #487

dpordomingo opened this issue Jan 23, 2019 · 0 comments · Fixed by #488
Assignees
Labels
enhancement New feature or request help needed Further information is requested

Comments

@dpordomingo
Copy link
Contributor

related to #486

There are some situations that cause unhandled errors like the following one.

[2019-01-23T11:17:34.461935361+01:00] ERROR event processing failed 
app=lookoutd 
error=posting analysis failed: %!s(PANIC=runtime error: invalid memory address or nil pointer dereference) 
event-id=b44c158d09bebb40b85ce23763dd9ab179926019 
event-type=*pb.ReviewEvent 
github.pr=63 
head=refs/pull/63/head 
repo=https://github.com/dpordomingo/testing-repo.git

When it happens, it's difficult to know what's causing the error.
In this case, I found that it was related to google/go-github#540

It would be a good idea to provide more details in those logs.

@dpordomingo dpordomingo added enhancement New feature or request help needed Further information is requested labels Jan 23, 2019
@dpordomingo dpordomingo self-assigned this Jan 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help needed Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant