-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: error handling for HttpErrorResponse #2324
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
3c65b3b
to
d72b454
Compare
d72b454
to
65bb8aa
Compare
return html | ||
.replace(/<[^>]*>/g, "") // remove tags |
Check failure
Code scanning / CodeQL
Polynomial regular expression used on uncontrolled data High
regular expression
library input
This
regular expression
library input
return html | ||
.replace(/<[^>]*>/g, "") // remove tags |
Check failure
Code scanning / CodeQL
Incomplete multi-character sanitization High
<script
Copilot Autofix AI 8 minutes ago
Generating an autofix suggestion, this might take a while…
Proposed changes
Added handling for
HttpErrorResponse
:DApp gets this error if used with WalletConnect
Types of changes
Steps to reproduce
Screenshots
Add the screenshots of how the app used to look like and how it looks now
Before:
The only useful message comes for error
500
. So we need to strip it from tags and keep for the user:Data-polling:
Now:
Data polling:
Checklist