Page MenuHomePhabricator

[services] Tunnelbroker - Removing of emptiness check for notification token
ClosedPublic

Authored by max on Nov 4 2022, 9:10 AM.
Tags
None
Referenced Files
F2190739: D5535.id18085.diff
Thu, Jul 4, 12:14 PM
Unknown Object (File)
Tue, Jul 2, 12:13 PM
Unknown Object (File)
Tue, Jul 2, 8:50 AM
Unknown Object (File)
Mon, Jul 1, 3:52 AM
Unknown Object (File)
Sun, Jun 30, 4:02 AM
Unknown Object (File)
Tue, Jun 25, 11:55 PM
Unknown Object (File)
Tue, Jun 25, 11:55 PM
Unknown Object (File)
Tue, Jun 25, 11:55 PM

Details

Summary

In this diff, we are removing the database item internal check for the device's notification token field.
We are fixing the bug where the database error occurs in the case when the device token in the database became empty and the server will return a database error when retrieving the token in the next client connection.

The field can be empty and the database check shouldn't fail on this. For more context please refer to the D5536.

Linear task: ENG-2060

Test Plan

This is a legacy C++ codebase change, the expected successful result is the successful building of the app and passing the CI checks.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

max held this revision as a draft.
max edited the test plan for this revision. (Show Details)
max added a reviewer: tomek.
max edited the summary of this revision. (Show Details)
max published this revision for review.Nov 4 2022, 9:57 AM
This revision is now accepted and ready to land.Nov 7 2022, 1:54 AM

Rebasing on master changes and merging.