Skip to content

chore: Add tick and peer id to logging#293

Merged
elementbound merged 3 commits intomainfrom
chore/log-id-tick
Oct 8, 2024
Merged

chore: Add tick and peer id to logging#293
elementbound merged 3 commits intomainfrom
chore/log-id-tick

Conversation

@elementbound
Copy link
Copy Markdown
Contributor

@elementbound elementbound commented Oct 8, 2024 •
edited
Loading

Example logs with the new format:

[DBG][@124][#1][netfox::NetworkTime] Client #384545578 is now on time!
[INF][@140][#1][netfox.extras::NetworkWeapon] [1] Accepting projectile df4k0e3r73f2 from 1
[DBG][@137][#384545578][netfox.extras::NetworkWeapon] Calling after fire hook for Bomb Projectile df4k0e3r73f2
[INF][@138][#384545578][netfox.extras::NetworkWeapon] [384545578] Accepting projectile df4k0e3r73f2 from 1
[INF][@152][#1][netfox.extras::NetworkWeapon] [1] Accepting projectile s10skmcayltj from 1
[DBG][@149][#384545578][netfox.extras::NetworkWeapon] Calling after fire hook for Bomb Projectile s10skmcayltj
[INF][@150][#384545578][netfox.extras::NetworkWeapon] [384545578] Accepting projectile s10skmcayltj from 1
[INF][@164][#1][netfox.extras::NetworkWeapon] [1] Accepting projectile 359ndlmyvif1 from 1
[DBG][@161][#384545578][netfox.extras::NetworkWeapon] Calling after fire hook for Bomb Projectile 359ndlmyvif1
[INF][@162][#384545578][netfox.extras::NetworkWeapon] [384545578] Accepting projectile 359ndlmyvif1 from 1
[INF][@179][#1][netfox.extras::NetworkWeapon] [1] Accepting projectile ceezxm4488ee from 1
[DBG][@176][#384545578][netfox.extras::NetworkWeapon] Calling after fire hook for Bomb Projectile ceezxm4488ee
[INF][@177][#384545578][netfox.extras::NetworkWeapon] [384545578] Accepting projectile ceezxm4488ee from 1

Closes #286

TheYellowArchitect
TheYellowArchitect reviewed Oct 8, 2024
View reviewed changes
@elementbound elementbound merged commit 38cc04f into main Oct 8, 2024
@elementbound elementbound deleted the chore/log-id-tick branch October 8, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@TheYellowArchitect TheYellowArchitect TheYellowArchitect left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Logger to include [NetworkTime.tick] and [Peer ID]

2 participants

@elementbound @TheYellowArchitect