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
WIP faster join call flow - wip #34
Conversation
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.
LGTM 👍
✅ Unit and Snapshot tests are passing
DecodingError: keyNotFound: credentials
)
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.
LGTM! Only the docs seem to have some issues. Will we tackle them as part of this PR or in another one?
} | ||
return "ERR" |
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.
Shouldn't we throw an error in this case?
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.
yes, not sure what's the best way to bubble / retry this
12f5bac
to
8a788af
Compare
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.
LGTM ✅
- Remove LatencyService_Tests - Add hlsPlaylistUrl argument - Use wsEndpoint provided by SFUResponse - Update unit tests
8a788af
to
4ccc2db
Compare
Replaces the current join call flow with the one described here: https://www.notion.so/stream-wiki/Faster-Join-Flow-64dcb736e91741cc960f245503eb9798