[SERVER-26744] Change formatting of Timestamp in logs to be proper javascript syntax Created: 24/Oct/16  Updated: 06/Dec/22  Resolved: 31/Aug/20

Status: Closed
Project: Core Server
Component/s: Testing Infrastructure
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Major - P3
Reporter: Judah Schvimer Assignee: Backlog - Security Team
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
Assigned Teams:
Server Security
Participants:

 Description   

Timestamps are currently formatted like in this optime:

 lastOpCommitted: { ts: Timestamp 1477084219000|1, t: 1 }

This makes it so you can't just copy and paste commands from logs to use for bug repros and in adhoc tests and have to go through and rewrite the command into

lastOpCommitted: { ts: Timestamp(1477084219,1), t: 1 }

in this case. Considering the latter syntax is no less readable, I think it could be useful to log Timestamps in a way that javascript does not complain about.



 Comments   
Comment by Ian Whalen (Inactive) [ 31/Aug/20 ]

we believe that this is gone away with the completion of structured logging, but please respond and @ mention spencer if you have any more concerns, Judah.

Generated at Thu Feb 08 04:13:05 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.