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

NPE when trying to validate WC URI #569

Open
kamilargent opened this issue Jan 6, 2023 · 0 comments
Open

NPE when trying to validate WC URI #569

kamilargent opened this issue Jan 6, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@kamilargent
Copy link

Describe the bug
A NPE occurs when trying to validate specific WC URI

SDK Version

  • Client: Android BOM
  • Version 1.3.0

To Reproduce
Steps to reproduce the behavior:

  1. Try to validate wc:7f6e504bfad60b485450578e05678ed3e8e8c4751d3c6160be17160d63ec90f9 URI
  2. Await result

Expected behavior
It's shown as invalid URI instead NPE happens

Additional context
Uri#userInfo and Uri#query can be null but in the SDK are treated as non-null.

pairUri.userInfo must not be null
java.lang.NullPointerException: pairUri.userInfo must not be null
	at com.walletconnect.android.internal.Validator.validateWCUri$sdk_debug(Validator.kt:30)
	at com.walletconnect.android.internal.ValidatorTest.validate WC uri test optional data fields(ValidatorTest.kt:66)
@kamilargent kamilargent added the bug Something isn't working label Jan 6, 2023
@github-project-automation github-project-automation bot moved this to Backlog in Kotlin SDK Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

1 participant