Skip to content

Add logs for sfu-ws #84

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

Merged
merged 1 commit into from
Nov 2, 2024
Merged

Add logs for sfu-ws #84

merged 1 commit into from
Nov 2, 2024

Conversation

cgojin
Copy link
Member

@cgojin cgojin commented Jan 8, 2021

Add some logs for debugging sfu-ws

@cgojin cgojin closed this Jan 11, 2021
@cgojin cgojin force-pushed the add-logs-for-sfu-ws branch from aa799f9 to 794232b Compare January 11, 2021 02:43
@cgojin cgojin reopened this Jan 11, 2021
@cgojin cgojin closed this Feb 10, 2021
@cgojin cgojin deleted the add-logs-for-sfu-ws branch February 10, 2021 08:13
@Sean-Der
Copy link
Member

@cgojin sorry I have been slow on reviews :(

I think this is a great change, are you still interested in merging it?

@cgojin
Copy link
Member Author

cgojin commented Feb 18, 2021

@cgojin sorry I have been slow on reviews :(

I think this is a great change, are you still interested in merging it?

It's okay, merge it if you have time, thank you!

@cgojin cgojin restored the add-logs-for-sfu-ws branch February 18, 2021 09:41
@cgojin cgojin reopened this Feb 18, 2021
@Sean-Der Sean-Der force-pushed the add-logs-for-sfu-ws branch from cec7a4b to 094b5de Compare November 2, 2024 03:54
Copy link

codecov bot commented Nov 2, 2024

Codecov Report

Attention: Patch coverage is 0% with 40 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (7a2c07e) to head (f5d8e00).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
sfu-ws/main.go 0.00% 40 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master     #84   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files          25      25           
  Lines        2803    2825   +22     
======================================
- Misses       2803    2825   +22     
Flag Coverage Δ
go 0.00% <0.00%> (ø)
wasm 0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Add logs and use pion's logger
@Sean-Der Sean-Der force-pushed the add-logs-for-sfu-ws branch from 094b5de to f5d8e00 Compare November 2, 2024 03:56
@Sean-Der
Copy link
Member

Sean-Der commented Nov 2, 2024

Almost 4 years late, but it finally got merged. thank you @cgojin

@Sean-Der Sean-Der merged commit 06a9bfe into master Nov 2, 2024
14 of 15 checks passed
@Sean-Der Sean-Der deleted the add-logs-for-sfu-ws branch November 2, 2024 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants