-
Notifications
You must be signed in to change notification settings - Fork 716
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
Expose detailed connection info with ICEConnectionDetails #2287
Conversation
Filtered bool | ||
} | ||
|
||
type ICEConnectionDetails struct { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
combining all ICE details into a new data type
return p.TransportManager.GetICEConnectionDetails() | ||
} | ||
|
||
func (p *ParticipantImpl) HasConnected() bool { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adding a new interface method. useful to know if Participant was ever successful
cds := participant.GetICEConnectionDetails() | ||
for _, cd := range cds { | ||
if cd.Type != types.ICEConnectionTypeUnknown { | ||
meta.ConnectionType = string(cd.Type) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be good to have both publisher
and subscriber
details in analytics at some point.
@@ -1416,3 +1426,39 @@ func BroadcastDataPacketForRoom(r types.Room, source types.LocalParticipant, dp | |||
} | |||
}) | |||
} | |||
|
|||
func connectionDetailsFields(cds []*types.ICEConnectionDetails) []interface{} { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be types/ice.go
?
pkg/rtc/types/ice.go
Outdated
} | ||
} | ||
|
||
func IsCandidateEqualTo(c1, c2 *webrtc.ICECandidate) bool { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These looks like local methods. Can these be lower case isICE...
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking they are useful to others that are comparing candidates. but will make them local for now until we need it elsewhere.
No description provided.