Skip to content
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

Set up server side SignalR log #1563

Open
andchiind opened this issue Apr 19, 2024 · 3 comments · Fixed by #1678
Open

Set up server side SignalR log #1563

andchiind opened this issue Apr 19, 2024 · 3 comments · Fixed by #1678
Assignees
Labels
backend Backend related functionality feature New feature or request stale This issue or pull request already exists

Comments

@andchiind
Copy link
Contributor

Describe the new feature you would like to see
When a message is sent from the server using signalR, the message should be logged.

Describe the solution you'd like
See: https://learn.microsoft.com/en-us/aspnet/core/signalr/diagnostics?view=aspnetcore-8.0

How will this feature affect the current Threat Model?
Should not be affected if we treat it as our other server logs.

@andchiind andchiind added the feature New feature or request label Apr 19, 2024
@oysand oysand added this to the Ready for users milestone Apr 28, 2024
@aeshub
Copy link
Contributor

aeshub commented Jul 1, 2024

This issue has automatically been marked as stale as there has been no activity for 60 days.

@aeshub aeshub added the stale This issue or pull request already exists label Jul 1, 2024
@andchiind andchiind added good first issue Good for newcomers backend Backend related functionality and removed stale This issue or pull request already exists good first issue Good for newcomers labels Jul 17, 2024
@andchiind andchiind self-assigned this Jul 18, 2024
@andchiind
Copy link
Contributor Author

This has been reverted so we no longer have specific signalr logs. I think a future solution would be to manually use the existing log functionality when we send a signalr message on the backend.

@andchiind andchiind reopened this Jul 25, 2024
@aeshub
Copy link
Contributor

aeshub commented Sep 23, 2024

This issue has automatically been marked as stale as there has been no activity for 60 days.

@aeshub aeshub added the stale This issue or pull request already exists label Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related functionality feature New feature or request stale This issue or pull request already exists
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants