marzent / IINACT

A Dalamud plugin to run the FFXIV_ACT_Plugin in an ACT-like enviroment with a heavily modified port of Overlay Plugin
https://www.iinact.com
GNU General Public License v3.0
182 stars 42 forks source link

Fights show starting at default epoch time when live logging with FFLogs Uploader (Overwolf + Original) #109

Closed lennalefay closed 3 weeks ago

lennalefay commented 3 weeks ago

Issue: Combat data for fights logs with a start date of 12/31/1969 and end date of the proper time when using the Overwolf FFLogs Uploader and live logging. Logs were showing as recording 55 years of time. (Please see below: I could not replicate this with ACT so I submitted it here, though I have also reported this error to FFLogs staff.)

Can Replicate: Yes, by live logging combat with IINACT logs from two different devices.

Notes:

I am away from my PC at the moment but can upload the logs that errored in 2-3 hours, though even creating new logs seemed to produce the same issue. Please let me know if there's anything else you need from me in order to facilitate looking into this.

jasonrm commented 3 weeks ago

Did a quick check of my logs, and while they live logged just fine, I do see this one line during init

256|2024-08-18T18:50:18.3618665-07:00|274|OverlayPlugin|ActorControlSelfExtra|1|189dc820365a61f6
260|1970-01-02T11:43:38.0000000-07:00|0|0|1|1|bd4895cba0fa2a22
42|2024-08-18T18:51:35.5230000-07:00|1320729D||0|0|0|f1db00338279e443

Which gets picked up by the FFLogs uploader and turns the duration into uh,

Screenshot 2024-08-19 at 05 20 13

Oldest report I have with that issue is from August 9. All the fights themselves have the right dates/times, but it's tricking the FF Logs uploader into thinking the overall report was started at epoch.

I'll see if the FF Logs uploader can be less silly. 😅

marzent commented 3 weeks ago

Ah this makes a bit more sense then, personally didn't see that issue happening yet. The timestamps are now exclusively fed from a network packet hook, but this also means that at least one packet needs to have been processed before the first logline is emitted, leaving a tiny window for OverlayPlugin to emit a logline with the unix epoch (but not the actual network based loglines from FFXIV_ACT_Plugin)...

marzent commented 3 weeks ago

Fixed with https://github.com/marzent/IINACT/commit/19502f474bb1f251930e726b64a685deeab87b88