Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add general use RTMP server #85
Add general use RTMP server #85
Changes from 5 commits
5c0b665
fe10680
02f1f9d
2495cc6
6c56ca0
982bc0d
a821766
6262d90
7888b2c
2eaead3
0f4a5cd
6747ddd
348f525
1d55c5e
af6a05e
4e6358e
855c71d
6c6a4b4
e9df383
baa1014
3c3b351
6a68408
27a5d20
cf98cfd
3518d97
bc80001
e288276
aa77ff4
406e061
dd28fcd
692c664
385bce1
a5a521d
8a53668
7752fd8
b833ff9
2620570
6e15040
bd74a5a
83adaf3
ecb4c3f
9dd8317
5d11bfd
8eee65c
cdc36d7
bdbd699
d976959
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Maybe we could have a very simple GenServer that would start a pipeline each time it gets the client handler?
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 believe this one would require adding an option to subscribe to "any"
app
and anystream_key
. It could be quite useful to have such an option, but perhaps let' add it in a separate PR, WDYT?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.
@varsill Ok, please create an issue, put it in TODO and assign @bartkrak ;) We should also have documentation for the message with
client_ref
, and the way to get the app and the stream key from there. Maybe we should allow awaiting for any app and stream key too.