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 on new Go-Feature-Flag provider #578

Closed
dangra opened this issue Sep 6, 2024 · 2 comments · Fixed by #579
Closed

Panic on new Go-Feature-Flag provider #578

dangra opened this issue Sep 6, 2024 · 2 comments · Fixed by #579

Comments

@dangra
Copy link
Contributor

dangra commented Sep 6, 2024

GoFeatureFlag provider panics after upgrading from v0.1.33 to v0.2.0.

I don't have a small snippet I can share yet but will try to attach one asap.

This is related to #547

 panic: runtime error: invalid memory address or nil pointer dereference
 [signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x1bcf4d2]
 goroutine 82 [running]:
 github.com/open-feature/go-sdk-contrib/providers/go-feature-flag/pkg.(*Provider).startPolling.func1()
         /home/vagrant/go/pkg/mod/github.com/open-feature/go-sdk-contrib/providers/go-feature-flag@v0.2.0/pkg/provider.go:248 +0x112
 created by github.com/open-feature/go-sdk-contrib/providers/go-feature-flag/pkg.(*Provider).startPolling in goroutine 69
         /home/vagrant/go/pkg/mod/github.com/open-feature/go-sdk-contrib/providers/go-feature-flag@v0.2.0/pkg/provider.go:226 +0xd6
@dangra
Copy link
Contributor Author

dangra commented Sep 6, 2024

the only possible way it can fail on line 248 is when err is nil

image

@dangra
Copy link
Contributor Author

dangra commented Sep 6, 2024

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 a pull request may close this issue.

1 participant