Skip to content

Conversation

@Meir017
Copy link
Contributor

@Meir017 Meir017 commented Jul 24, 2025

related #2899

Introduces dedicated LoggerMessage attributes for PhysicalConnection operations to improve logging performance and consistency.

Replaces string interpolation and manual log calls with strongly-typed logging methods that provide better compile-time safety and runtime performance through source generation.

Covers connection lifecycle events including endpoint validation, connection attempts, TLS configuration, socket operations, and connection establishment.

Meir017 added 2 commits July 24, 2025 16:27
…alConnection & PhysicalBridge

Replaces string interpolation with structured logging

Migrates logging calls from string interpolation to structured logging using LoggerMessage attributes. This improves performance by avoiding string allocation and enables better log analysis through structured data.

Adds new logging extension methods for PhysicalBridge operations including connection establishment, failures, dead socket detection, and resurrection attempts.
Introduces dedicated LoggerMessage attributes for PhysicalConnection operations to improve logging performance and consistency.

Replaces string interpolation and manual log calls with strongly-typed logging methods that provide better compile-time safety and runtime performance through source generation.

Covers connection lifecycle events including endpoint validation, connection attempts, TLS configuration, socket operations, and connection establishment.
Copy link
Collaborator

@mgravell mgravell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you're on a roll

@mgravell mgravell merged commit b3aeb61 into StackExchange:main Jul 24, 2025
6 checks passed
@Meir017 Meir017 deleted the chore/logger-message-attribute branch July 24, 2025 14:09
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.

2 participants