Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Error: Returned error: Invalid params: invalid type: map, expected a 0x-prefixed hex string with length ... #3275

Closed
sentry-io bot opened this issue Sep 7, 2019 · 6 comments
Closed

Error: Returned error: Invalid params: invalid type: map, expected a 0x-prefixed hex string with length ... #3275

sentry-io bot opened this issue Sep 7, 2019 · 6 comments
Assignees
@mikeshultz
Labels
bug Something isn't working as intended dapp P1 Large amount of significant user impact, a meaningful feature is broken

Comments

@sentry-io
Copy link

sentry-io bot commented Sep 7, 2019

Sentry Issue: ORIGIN-DAPP-62

Error: Returned error: Invalid params: invalid type: map, expected a 0x-prefixed hex string with length of 64.
  at ErrorResponse (/app/node_modules/web3-core-helpers/src/errors.js:29:16)
  at callback (/app/node_modules/web3-core-requestmanager/src/index.js:140:36)
  at apply (/app/node_modules/web3-providers-http/src/index.js:77:13)
  at XMLHttpRequest.r (/app/node_modules/@sentry/browser/esm/helpers.js:70:23)
@mikeshultz mikeshultz added bug Something isn't working as intended dapp labels Sep 7, 2019
@mikeshultz
Copy link
Contributor

A bit of a guess, but I think something is being sent to eth_getTransactionReceipt that isn't a tx hash.

@mikeshultz
Copy link
Contributor

Raw request seen in Alchemy dashboard:

{
	"id":1172
	"jsonrpc":"2.0"
	"method":"eth_getTransactionByHash"
	"params":[
		0:{
			"message":"User denied transaction signature"
		}
	]
}

@mikeshultz
Copy link
Contributor

Also:

{
  "jsonrpc": "2.0",
  "id": 1079,
  "method": "eth_getTransactionByHash",
  "params": [
    null
  ]
}

{
  "jsonrpc": "2.0",
  "id": 1173,
  "method": "eth_getTransactionReceipt",
  "params": [
    {}
  ]
}

@mikeshultz mikeshultz mentioned this issue Sep 7, 2019
Closed
mikeshultz added a commit that referenced this issue Sep 7, 2019
@mikeshultz
validate input for receipt resolver. issue #3266 and #3275
2cb76ca
@mikeshultz mikeshultz mentioned this issue Sep 7, 2019
Merged
4 tasks
mikeshultz added a commit that referenced this issue Sep 12, 2019
@mikeshultz
Validate tx receipt id in resolver (#3277)
32b8a68 
* validate input for receipt resolver.  issue #3266 and #3275

* do a try/catch around decodeLog() to pinpoint error if it happens

* adds extra error handling to swapToToken mutation
@franckc franckc added the P1 Large amount of significant user impact, a meaningful feature is broken label Sep 13, 2019
@mikeshultz mikeshultz mentioned this issue Sep 16, 2019
Closed
@mikeshultz mikeshultz self-assigned this Sep 27, 2019
mikeshultz added a commit that referenced this issue Sep 27, 2019
@mikeshultz
adds tx hash validation to _txHelper to try and get to the bottom of #…
6d23f19 
…3275
@mikeshultz mikeshultz mentioned this issue Sep 27, 2019
Merged
4 tasks
mikeshultz added a commit that referenced this issue Sep 27, 2019
@mikeshultz
adds tx hash validation to _txHelper to try and get to the bottom of #…
bd88850 
…3275 (#3538)
@sentry-io
Copy link
Author

sentry-io bot commented Sep 30, 2019

Sentry issue: ORIGIN-DAPP-GA

This was referenced Sep 30, 2019
Open
Open
@mikeshultz
Copy link
Contributor

mikeshultz commented Sep 30, 2019 •
edited
Loading

#3574 and #3573 are the two occurrences so far of this, but there may be others.

@mikeshultz mikeshultz mentioned this issue Oct 1, 2019
Closed
mikeshultz added a commit that referenced this issue Oct 1, 2019
@mikeshultz
small logic adjustments to how weird transaction hash values are hand…
3742b08 
…led. Issues #3573 #3574 #3275
@mikeshultz mikeshultz mentioned this issue Oct 1, 2019
Merged
4 tasks
mikeshultz added a commit that referenced this issue Oct 1, 2019
@mikeshultz
Transaction hash handling (#3589)
a712c37 
* small logic adjustments to how weird transaction hash values are handled.  Issues #3573 #3574 #3275

* prevent refetch when hash is pending, and integrate @nick's feedback
@mikeshultz
Copy link
Contributor

closing this in favor of the other two issues.

@mikeshultz mikeshultz mentioned this issue Jan 6, 2020
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mikeshultz mikeshultz

Labels
bug Something isn't working as intended dapp P1 Large amount of significant user impact, a meaningful feature is broken
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@franckc @mikeshultz