Page MenuHomePhabricator

[Identity] Change default port to 50054 to avoid port collision
ClosedPublic

Authored by jon on Mar 9 2023, 3:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 12, 12:01 AM
Unknown Object (File)
Sun, May 12, 12:00 AM
Unknown Object (File)
Sat, May 11, 1:58 PM
Unknown Object (File)
Sat, May 11, 1:58 PM
Unknown Object (File)
Fri, May 10, 7:22 PM
Unknown Object (File)
Sun, May 5, 3:31 AM
Unknown Object (File)
Fri, May 3, 8:27 AM
Unknown Object (File)
Mon, Apr 29, 7:22 AM
Subscribers

Details

Summary

Avoid future port testing or deployment conflicts
by defaulting port something other than default grpc port.

https://linear.app/comm/issue/ENG-3289

Test Plan

Run identity service, keyserver, and native yarn dev

Create new user. Verify identity service is responding.

Diff Detail

Repository
rCOMM Comm
Branch
jonringer/identity-default-port
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Mar 9 2023, 4:00 PM
Harbormaster failed remote builds in B17229: Diff 23597!
services/identity/src/main.rs
61

Just added this because it's pretty jarring to get 0 output initially.

This revision is now accepted and ready to land.Mar 9 2023, 10:55 PM
varun added inline comments.
services/identity/src/main.rs
61

nit: gRPC

jon marked an inline comment as done.

grpc -> gRPC

services/identity/src/main.rs
61

fair