[alias-server] Patch bug in log.js
ClosedPublic
Actions

Authored by bytesofman on Mar 15 2023, 23:33.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC612173d4b406: [alias-server] Patch bug in log.js

Summary

T2965

Since this app is run on a server with systemctl, we do not need timestamps in logging (already provided by server). Patch bug in log.js where trying to log('someVariable', <someVariable>) would display nothing if <someVariable> === 0 or false

Test Plan

node index.js and no timestamps.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

log-fix

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 22473
Build 44571: Build Diff
Build 44570: arc lint + arc unit

Event Timeline

bytesofman created this revision.Mar 15 2023, 23:33

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 15 2023, 23:33

bytesofman requested review of this revision.Mar 15 2023, 23:33

Harbormaster completed remote builds in B22473: Diff 38605.Mar 15 2023, 23:33

emack accepted this revision.Mar 16 2023, 05:20

This revision is now accepted and ready to land.Mar 16 2023, 05:20

Closed by commit rABC612173d4b406: [alias-server] Patch bug in log.js (authored by bytesofman). · Explain WhyMar 16 2023, 11:55

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC612173d4b406: [alias-server] Patch bug in log.js.

Revision Contents
Changeset List

Path

Size

web/

alias-server/

log.js

13 lines

Diff 38605

View Options

[alias-server] Patch bug in log.jsClosedPublicActions