-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#305 add syncronisation with gitlab #309
Conversation
#305 add syncronisation with gitlab
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #309 +/- ##
===========================================
- Coverage 86.59% 68.53% -18.06%
===========================================
Files 328 265 -63
Lines 4140 3350 -790
Branches 352 283 -69
===========================================
- Hits 3585 2296 -1289
- Misses 440 970 +530
+ Partials 115 84 -31 ☔ View full report in Codecov by Sentry. |
if (!string.IsNullOrEmpty(userId)) | ||
{ | ||
await Groups.AddToGroupAsync(Context.ConnectionId, userId); | ||
_logger.LogInformation($"======================================================================================User {userId} connected with connection ID: {Context.ConnectionId}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
if (!string.IsNullOrEmpty(userId)) | ||
{ | ||
await Groups.RemoveFromGroupAsync(Context.ConnectionId, userId); | ||
_logger.LogInformation($"User {userId} disconnected with connection ID: {Context.ConnectionId}"); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks Low
#305 update build sh
Test merge to mirror gitlab repo structure. |
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Synchronisation with gitlab