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.

Sign up for GitHub

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

Jump to bottom

Type mismatch while running gossamer #2625

Closed
EclesioMeloJunior opened this issue Jun 28, 2022 · 0 comments · Fixed by #2626
Closed

Type mismatch while running gossamer #2625

EclesioMeloJunior opened this issue Jun 28, 2022 · 0 comments · Fixed by #2626
Assignees
@EclesioMeloJunior

Comments

@EclesioMeloJunior
Copy link
Member

Describe the bug

  • Notice the panic while running gossamer in a cross-client dev network:
panic: value is not uint32 or common.Hash. got 150 (uint)

goroutine 59988 [running]:
github.com/ChainSafe/gossamer/lib/common/variadic.MustNewUint32OrHash(...)
	github.com/ChainSafe/gossamer/lib/common/variadic/uint32OrHash.go:44
github.com/ChainSafe/gossamer/dot/sync.workerToRequests(0xc000e15540)
	github.com/ChainSafe/gossamer/dot/sync/chain_sync.go:1038 +0x51b
github.com/ChainSafe/gossamer/dot/sync.(*chainSync).dispatchWorker(0xc001278700, 0xc000e15540)
	github.com/ChainSafe/gossamer/dot/sync/chain_sync.go:659 +0x247
created by github.com/ChainSafe/gossamer/dot/sync.(*chainSync).tryDispatchWorker
	github.com/ChainSafe/gossamer/dot/sync/chain_sync.go:625 +0xc5
  • Currently, we only support the types: int and int32 but we should ensure that we support the type uint while creating the NewUint32OrHash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EclesioMeloJunior EclesioMeloJunior

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: ensure we convert the uint type ChainSafe/gossamer
1 participant
@EclesioMeloJunior