It appears the second-instance event (used by deeplinks) can fire in other cases too, at least on Windows. This is resulting in one of our highest errors in Sentry since we assume they are all valid URLs which is evidently not the case.
What changed
Be resilient to invalid URLs in deeplinks and log instead of throwing.
Why
It appears the
second-instance
event (used by deeplinks) can fire in other cases too, at least on Windows. This is resulting in one of our highest errors in Sentry since we assume they are all valid URLs which is evidently not the case.What changed
Be resilient to invalid URLs in deeplinks and log instead of throwing.
Test plan
Should see the Sentry error go to 0 after this