[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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 38619

View Options

[alias-server] Patch bug in log.jsClosedPublicActions