openthc / ccrs

Tracking for the WSLCB CCRS
https://lcb.wa.gov/ccrs
MIT License
4 stars 0 forks source link

Error Response CSVs Missing Full Date Part #44

Open djbusby opened 1 year ago

djbusby commented 1 year ago

The documentation states that the CSVs will have a datetime format matching Ymd\TGisv (from PHP DateTime->format(). However sometimes we see a response that is only Ymd\TGis which is unexpected.

Expect: Inventory_XX_20221101T143621952.csv Actual: Inventory_XX_20221101T143615.csv

Most files include the microseconds; but sometimes it seems it gets dropped?

djbusby commented 1 year ago

Also noticed on upload Area_XX_20221101T151854519.csv the error response file was Area_XX_20221101T151927.csv -- missing the microseconds.

djbusby commented 1 year ago

LCB Incident Report: IR65290

djbusby commented 1 year ago

ALso, rleated this one is missing the leading zero on the month Manifest_AF0E72B77C_01GPES556DMBZ1YQRCQWG3QZ4F_2023111T822904.csv

So, it makes parsing the time information wrong; cause when the leading zero is missing, is it missing from the month or the day? Same on the time for hour/minute second -- like when the milliseconds are missing.

I mean, time formatting is trivial and folks have been telling your agency about ISO8601 and RFC3339 for at least five years.

djbusby commented 3 months ago

Related to https://github.com/openthc/ccrs/issues/69