Page MenuHomePhabricator

[lib] Ignore exceptions in extractUserIDsFromPayload
ClosedPublic

Authored by ashoat on May 30 2024, 12:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 20, 4:22 PM
Unknown Object (File)
Thu, Jun 20, 4:51 AM
Unknown Object (File)
Sat, Jun 15, 9:10 PM
Unknown Object (File)
Mon, Jun 10, 5:02 AM
Unknown Object (File)
Mon, Jun 10, 5:02 AM
Unknown Object (File)
Mon, Jun 10, 4:59 AM
Unknown Object (File)
Sun, Jun 9, 8:49 PM
Unknown Object (File)
Fri, Jun 7, 7:22 PM
Subscribers

Details

Summary

When an endpoint fails, it will respond with something that doesn't match the expected output validator.

This causes the client to throw a different error that the one we expect from the endpoint failing.

This diff addresses ENG-8324 by wrapping the convertObject call in a try-catch that ignores all exceptions.

Test Plan

I tested on both native and web by attempting to set an invite link corresponding to a reserved string. Before this diff I got a generic error, but afterwards I got the expected, more precise error

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable