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

Fix nil pointer dereference and increase timeout #24

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

yahavi
Copy link
Member

@yahavi yahavi commented Oct 30, 2024

Fix this error:

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x18 pc=0x1007c8900]

goroutine 1 [running]:
main.runInRetryLoop(0x100915ee8, {0x1007dc55d, 0x27})
	/Users/runner/go/pkg/mod/github.com/jfrog/jfrog-testing-infra/[email protected]/main.go:261 +0x1c0
main.generateAccessToken()
	/Users/runner/go/pkg/mod/github.com/jfrog/jfrog-testing-infra/[email protected]/main.go:324 +0x88
main.setupLocalArtifactory()
	/Users/runner/go/pkg/mod/github.com/jfrog/jfrog-testing-infra/[email protected]/main.go:135 +0x3a0
main.main()
	/Users/runner/go/pkg/mod/github.com/jfrog/jfrog-testing-infra/[email protected]/main.go:56 +0x1c
Error: Process completed with exit code 2.

@yahavi yahavi requested a review from noyshabtay October 30, 2024 07:42
@yahavi yahavi self-assigned this Oct 30, 2024
Copy link

@noyshabtay noyshabtay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yahavi yahavi merged commit 28076c6 into jfrog:main Oct 30, 2024
2 checks passed
@yahavi yahavi deleted the fix-nil branch October 30, 2024 08:45
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

Successfully merging this pull request may close these issues.

2 participants