Skip to content

feat: Netcode log system#3944

Draft
EmandM wants to merge 2 commits intodevelop-2.0.0from
experimental/logger
Draft

feat: Netcode log system#3944
EmandM wants to merge 2 commits intodevelop-2.0.0from
experimental/logger

Conversation

@EmandM
Copy link
Copy Markdown
Member

@EmandM EmandM commented Apr 17, 2026

Purpose of this PR

  • Adds a ContextualLogger class that can hold references to a NetworkManager and a GameObject
  • Adds a ILogContext interface to standardize how LogContexts can be written to a builder
  • Adds a Context struct that is passed to the functions on the ContextualLogger to collect information about specific callsites
  • Uses these new structs throughout NetworkManager, NetworkManagerHelper, and NetworkManagerEditor.
  • Swaps out NetworkLog to use a system wide static version of the ContextualLogger
  • The new logger will highlight the relevant GameObject in the scene view when clicking on logs!

Jira ticket

Changelog

  • Added: Context aware logging

Documentation

  • No documentation changes or additions were necessary.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

  • Manual testing done

Automated tests:

  • Covered by existing automated tests
  • Covered by new automated tests

Does the change require QA team to:

  • Review automated tests?
  • Execute manual tests?
  • Provide feedback about the PR?

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Backports

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant