Aaron Clauson

No, nothing like that is done by this library. The preferred approach is to [send silence](https://github.com/sipsorcery-org/sipsorcery/blob/2dde80a98d11dd9dfa02ff9833c5bda409127c80/src/app/Media/Sources/AudioExtrasSource.cs#L428). If you really wanted to stop sending RTP the event handler could just be...

What's actually changing in this PR? Which line has the change and is not just a refactor?

Is this the non-refactor change? ``` default://Allocate not supported - this is a STUN server only not a full TURN server throw new ApplicationException("Unsupported MessageType " + stunRequest.Header.MessageType); ``` Is...

Not supported in .NET Framework 4.6.1 (.NET Standard 2.0).

The decision is to continue supporting .NET Framework until there is a compelling reason not to.

The flag enum is a good improvement. Happy to merge. The change is highly unlikely to affect any library users. If there are further string optimisations planned it would be...

My guess would be that the `Set remote track` is a red herring. All it's doing is responding once the first RTP packet is received to connect the local RTP...

See this [comment](https://github.com/sipsorcery-org/sipsorcery/issues/1202#issuecomment-2460748226) regarding suitability of this library for implemnting an SFU.

Minor not but 3 of the reference links go to the same page and one returns a 404. Hopefully this PR was generated with an automated tool. It's a large...

> > Additionally, IMHO instrumenation and analytics should be separate from logging. They serve different purposes and combiing them creates a maintenance burden, paricularly for simple debug log messages. >...