Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

Event logs transaction hash is null #542

Closed
IanDorion opened this issue May 9, 2022 · 4 comments · Fixed by #528
Closed

Event logs transaction hash is null #542

IanDorion opened this issue May 9, 2022 · 4 comments · Fixed by #528
Assignees
Labels
bug Something isn't working
Milestone

Comments

@IanDorion
Copy link

Event log transaction hash

Description

After a transaction, when looking at the event, the transaction hash is null

Your environment

  • Unbuntu(WSL2) Docker container
  • latest
  • develop

Steps to reproduce

Picking up the hash using go-ethereum;

for {
		select {
		case myEventSubErr := <-depositsSub.Err():
			log.Fatal("error in deposits event subscription: ", myEventSubErr)
		case myEvent := <-deposits:
			fmt.Printf(myEvent.Raw.TxHash)
		}
	}
	// output 0x0000000000000000000000000000000000000000000000000000000000000000
	

Expected behaviour

I should receive a valid transaction hash

Actual behaviour

receive a null transaction hash

@0xAleksaOpacic
Copy link
Contributor

Hi @IanDorion ,
We have already created the fix for this issue here is the PR #528.
We expect to merge it soon.

@0xAleksaOpacic 0xAleksaOpacic self-assigned this May 9, 2022
@0xAleksaOpacic 0xAleksaOpacic added the bug fix Functionality that fixes a bug label May 9, 2022
@IanDorion
Copy link
Author

Hello @Aleksao998,
Thank you for you quick response.

@ZeljkoBenovic ZeljkoBenovic linked a pull request May 10, 2022 that will close this issue
11 tasks
@zivkovicmilos zivkovicmilos added bug Something isn't working and removed bug fix Functionality that fixes a bug labels May 10, 2022
@zivkovicmilos zivkovicmilos added this to the 0.4.0 milestone May 13, 2022
@0xAleksaOpacic
Copy link
Contributor

Hi @IanDorion
The fix is pushed into the develop branch. And we are creating release today.
Because it is breaking change we advise pulling the latest code on all active nodes.

@IanDorion
Copy link
Author

Thank you very much!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants