Page MenuHomePhabricator

[keyserver] fix production check while uploading keys to Identity
ClosedPublic

Authored by kamil on Oct 16 2023, 11:38 AM.
Tags
None
Referenced Files
F2170695: D9505.id32044.diff
Tue, Jul 2, 3:18 PM
Unknown Object (File)
Mon, Jul 1, 12:54 AM
Unknown Object (File)
Wed, Jun 26, 3:58 PM
Unknown Object (File)
Tue, Jun 25, 5:25 AM
Unknown Object (File)
Tue, Jun 25, 5:25 AM
Unknown Object (File)
Tue, Jun 25, 5:24 AM
Unknown Object (File)
Tue, Jun 25, 5:16 AM
Unknown Object (File)
Sun, Jun 23, 5:12 AM
Subscribers

Details

Summary

Fixing regression introduced in D9427.

This shouldn't affect anything, the only risk is that migration could make a silent fail on prod because the condition returns false always - but this is not what we want.

I discovered on my local setup that process.env.NODE_ENV === 'production' will not work because this env value is never set to production (this might be the fault of some missing .env files on my setup), but based on the code we only set NODE_ENV for dev, so changing this for safer solution.

Test Plan
  1. Terminate Identity
  2. Test migration on prod (fail) / dev (only warning)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable