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

Sentry issues #727

Open
3 of 7 tasks
ferostabio opened this issue Aug 25, 2023 · 2 comments · May be fixed by #728
Open
3 of 7 tasks

Sentry issues #727

ferostabio opened this issue Aug 25, 2023 · 2 comments · May be fixed by #728
Assignees

Comments

@ferostabio
Copy link
Contributor

ferostabio commented Aug 25, 2023

  • TypeError: Cannot read properties of undefined (reading 'providerName')

  • SecurityError: Attempt to use history.replaceState() more than 100 times per 30 seconds

Could mean a loop between two pages, I believe it happened on mobile (for sure) and firefox desktop and both times between my positions and lend.

  • TypeError: Cannot set property isZerion of # which has only a getter.

Zerion is a wallet (https://zerion.io/) and whatever the reason, it seems its crashing the app.

  • TypeError Function.defineProperty(): ‘defineProperty' on proxy: trap returned falsish for property 'request'

  • ChunkLoadError

We were getting this one a lot a couple of weeks ago. Not sure why (https://rollbar.com/blog/javascript-chunk-load-error/#)

  • Get rid of borrow/lend wrong placed actions caused error on render
  • Chart d path wrong move
@ferostabio
Copy link
Contributor Author

@brozorec @hovoh/ethcall/lib/call.js is sometimes failing when doing tryAll. Sentry reports Cannot read properties of undefined (reading 'success')". When we look at the code, it does the following:

for (let i = 0; i < callCount; i++) {
    const name = calls[i].name;
    const outputs = calls[i].outputs;
    const result = response[i];
    if (!result.success) {
      callResult.push(null);
    }
    ...

If result is undefined, result.success will fail. Checked and there has been no recent updates, we should report the issue.

@NikolaiYurchenko NikolaiYurchenko linked a pull request Aug 28, 2023 that will close this issue
@NikolaiYurchenko
Copy link
Contributor

UPD on other points:

  1. TypeError Function.defineProperty(): ‘defineProperty' on proxy: trap returned falsish for property 'request'

This often occurs when we trying to change immutable properties (props, freezed, set-method-blocked, etc), morelikely in sdk

  1. Zerion - cant log in at all with this wallet by app
  2. history.replaceState() - tried on desktop safari and firefox, not getting an error yet, will try to use mobile
  3. providerName - need more info, cant even find a place where we are using it as property of an object

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants