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

Fix #57 - Modify var and spelling in conf #58

Merged
merged 1 commit into from
Apr 22, 2016
Merged

Fix #57 - Modify var and spelling in conf #58

merged 1 commit into from
Apr 22, 2016

Conversation

mbeacom
Copy link

@mbeacom mbeacom commented Apr 21, 2016

No description provided.

@@ -101,7 +101,7 @@ stats {
# default: 0
network 0;
# the device name to stat the disk iops.
# ignore the device of /proc/diskstats if not configed.
# ignore the device of /proc/diskstats if not configured.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@winlinvip
Copy link
Member

Appreciate for your work, thanks and give lots of confidence to me. I hope to improve my English if possible~

@mbeacom
Copy link
Author

mbeacom commented Apr 22, 2016

@winlinvip I didn't mean for my PR to seem crude! I just noticed the random adjustments as I was looking through your code, while I try to add to the HLS functionality. I'm very eager to see how this project evolves.

@mbeacom mbeacom closed this Apr 22, 2016
@winlinvip winlinvip reopened this Apr 22, 2016
@winlinvip
Copy link
Member

I means it's really good PR.

@winlinvip winlinvip merged commit 13da1da into ossrs:develop Apr 22, 2016
@winlinvip
Copy link
Member

I have merge it to go-oryx and I will merge to SRS. Thanks~
😄

@winlinvip
Copy link
Member

@mbeacom Do not support HLS in go-oryx, because I find it's possible to support HLS+, which remux the RTMP to HLS on edge(while HLS is remux RTMP on origin).

@winlinvip
Copy link
Member

ossrs/srs@892591f
Thanks~ 😃

@mbeacom mbeacom deleted the bugs-57 branch April 22, 2016 02:34
@mbeacom
Copy link
Author

mbeacom commented Apr 22, 2016

@winlinvip So, you are transmuxing the RTMP to HLS on the edge? That's great.

@winlinvip
Copy link
Member

winlinvip commented Apr 22, 2016

Yep, we have done it in our private product, it's ok to transmux on the edge. And we have public the solution(without code), use 302 or variant HLS to redirect the original HLS request to a new url with id. For example, when user request http://server/app/stream.m3u8, redirect to http://server/app/stream.m3u8?id=xxx. And each ts in m3u8 will be suffixed with the query with the id. So we can create a logic stream for HLS, and we can serve it by the same http handle, which transmux the RTMP to HLS.

The HLS+ demo: https://notehub.org/s5qiw

@winlinvip
Copy link
Member

winlinvip commented Apr 22, 2016

For exmaple, for the first request of HLS:

winlin:srs-plus winlin$ curl http://ossrs.net:8083/show/livestream.m3u8
Redirect to http://ossrs.net:8083/show/livestream.m3u8?shp_uuid=eba98df08622f770e0dcfa9000afcb45&shp_ts=1461293515678&shp_cid=133048
&shp_pid=2465&shp_sip0=182.92.80.26&shp_sip1=182.92.80.26&domain=ossrs.net

We will 302 the request to a new url, then:

winlin:srs-plus winlin$ curl 'http://ossrs.net:8083/show/livestream.m3u8?shp_uuid=eba98df08622f770e0dcfa9000afcb45&shp_ts=1461293515678&shp_cid=133048
&shp_pid=2465&shp_sip0=182.92.80.26&shp_sip1=182.92.80.26&domain=ossrs.net'
#EXTM3U
#EXT-X-VERSION:3
#EXT-X-ALLOW-CACHE:YES
#EXT-X-MEDIA-SEQUENCE:0
#EXT-X-TARGETDURATION:2
#EXT-X-DISCONTINUITY
#EXTINF:1.527, no desc
livestream-0.ts?shp_uuid=eba98df08622f770e0dcfa9000afcb45&shp_ts=1461293515678&shp_cid=133048
&shp_pid=2465&shp_sip0=182.92.80.26&shp_sip1=182.92.80.26&domain=ossrs.net
&shp_seqno=0

When request the m3u8 with id, all ts suffixed with id, and the ts is generated dynamically for this logical connection with small duration(1.5s).

Then the second piece:

winlin:srs-plus winlin$ curl 'http://ossrs.net:8083/show/livestream.m3u8?shp_uuid=eba98df08622f770e0dcfa9000afcb45&shp_ts=1461293515678&shp_cid=133048
&shp_pid=2465&shp_sip0=182.92.80.26&shp_sip1=182.92.80.26&domain=ossrs.net'
#EXTM3U
#EXT-X-VERSION:3
#EXT-X-ALLOW-CACHE:YES
#EXT-X-MEDIA-SEQUENCE:0
#EXT-X-TARGETDURATION:2
#EXT-X-DISCONTINUITY
#EXTINF:1.527, no desc
livestream-0.ts?shp_uuid=eba98df08622f770e0dcfa9000afcb45&shp_ts=1461293515678&shp_cid=133048
&shp_pid=2465&shp_sip0=182.92.80.26&shp_sip1=182.92.80.26&domain=ossrs.net
&shp_seqno=0
#EXTINF:1.530, no desc
livestream-1.ts?shp_uuid=eba98df08622f770e0dcfa9000afcb45&shp_ts=1461293515678&shp_cid=133048
&shp_pid=2465&shp_sip0=182.92.80.26&shp_sip1=182.92.80.26&domain=ossrs.net
&shp_seqno=1

HLS+ provides low latency and the same domain for RTMP and HLS+, it's very useful.

@mbeacom
Copy link
Author

mbeacom commented Apr 22, 2016

@winlinvip That is FANTASTIC! I just went through all of the documentation that you linked to and provided. Google's translator has come a long way!

Is there anything I can do to help contribute to the project? I don't want to start writing out something that doesn't align with your vision for go-oryx.

Thanks!

@winlinvip
Copy link
Member

@mbeacom I think you can help to correct the English wiki of SRS, which writen by my broken English... You can contact with me by Skype winterserver, and you will know the direction of SRS/oryx. What's about this advise?

@mbeacom
Copy link
Author

mbeacom commented Apr 25, 2016

@winlinvip That sounds good! Let me know when will work best for you and we can discuss it! Are there any specific sections you'd like me to review in the meantime?

@winlinvip
Copy link
Member

Let's talk on Skype

@winlinvip
Copy link
Member

My Skype is winterserver

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants