Page MenuHomePhabricator

[keyserver] Handle null browser detection result
ClosedPublic

Authored by tomek on Oct 3 2024, 4:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 9:06 AM
Unknown Object (File)
Wed, Dec 18, 9:06 AM
Unknown Object (File)
Wed, Dec 18, 9:06 AM
Unknown Object (File)
Wed, Dec 18, 9:06 AM
Unknown Object (File)
Mon, Nov 25, 5:09 AM
Unknown Object (File)
Nov 22 2024, 7:14 AM
Unknown Object (File)
Nov 19 2024, 2:40 AM
Unknown Object (File)
Nov 17 2024, 2:50 AM
Subscribers

Details

Summary

When a browser can't be detected we need to make sure that we don't crash. Handling this case in the same way as if it was a web browser probably makes the most sense.

https://linear.app/comm/issue/ENG-9477/typeerror-cannot-read-properties-of-null-reading-os-on-landing

Depends on D13587

Test Plan

Replaced detectionResult with null and checked that we're handling it as if it was a web browser.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable