Page MenuHomePhabricator

[web] remove ping component
ClosedPublic

Authored by varun on Jun 12 2024, 12:29 PM.
Tags
None
Referenced Files
F3410143: D12408.diff
Wed, Dec 4, 7:59 PM
F3407820: D12408.id.diff
Wed, Dec 4, 7:59 AM
Unknown Object (File)
Sat, Nov 16, 3:42 AM
Unknown Object (File)
Fri, Nov 15, 6:23 AM
Unknown Object (File)
Nov 1 2024, 1:29 PM
Unknown Object (File)
Oct 26 2024, 3:22 AM
Unknown Object (File)
Oct 5 2024, 6:43 AM
Unknown Object (File)
Oct 5 2024, 6:43 AM
Subscribers

Details

Summary

we are now calling the identity service from MinVersionHandler on web, so there's no need for this ping component anymore. the point was to make sure that the identity client is callable via gRPC-web on prod, which is achieved now in MinVersionHandler.

Depends on D12407

Test Plan

made sure there were no references to IdentityPing left in code

Diff Detail

Repository
rCOMM Comm
Branch
webping (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage