Fix Timer leakage in TimerBasedCredentialRefresher ...#1640

Closed

MatthiasWerning wants to merge 9 commits intorabbitmq:mainfrom

MatthiasWerning:fix-timer-leakage

Commits on Jul 30, 2024

Fix Timer leakage in TimerBasedCredentialRefresher by maintaining only one registration across all connection recoveries

MatthiasWerning
authored and
lukebakken
committed
Re-add accidentally removed import of System.Diagnostics.CodeAnalysis

MatthiasWerning
authored and
lukebakken
committed
Remove unnecessary whitespaces

MatthiasWerning
authored and
lukebakken
committed
Add unit test to test callback update: Make sure old callback isn't called again after updating

MatthiasWerning
authored and
lukebakken
committed
Fix whitespace in test

MatthiasWerning
authored and
lukebakken
committed
Move actual refresh on timer elapse into non-locked scope to handle await of notification callback

MatthiasWerning
authored and
lukebakken
committed
Code review suggestions: Do not pass lock object down to timer registration; dispose old timer on elapse; general suggested refactorings

MatthiasWerning
authored and
lukebakken
committed
Formatting

MatthiasWerning
authored and
lukebakken
committed
Formatting once again

MatthiasWerning
authored and
lukebakken
committed