Fix for rtmp streaming with CloudFront

Hi,

During testing the s2member plugin, i wanted to display a video.mp4, using CloudFront. we went over the video tutorial, but we hit the issue “Stream not found” During the rtmp transaction, the server just return us “NetStream.Failed”. After some hours of debugging (i was new to Amazon S3, CloudFront, S2member), i’ve found the error, and i want to share it.

Our website is served via an Elastic Load Balancer. When s2member generate rtmp/CloudFront policy, it include an IP restriction, using $_SERVER[‘REMOTE_ADDR’]. But the IP in this one is the IP of Elastic Load Balancer, not the IP of the client. At the end, the generated url contain an invalid restriction, and lead to the NetStream.Failed.

Here is a patch to use HTTP_X_FORWARDED_FOR if it exist, and supersede REMOTE_ADDR:

https://gist.github.com/2706125

I hope that will be resolved soon :)

Mathieu

• This topic was modified 4 years, 7 months ago by  Mathieu Virbel.
• This topic was modified 4 years, 7 months ago by  Mathieu Virbel.

Hi Mathieu.

Thanks so much for sharing that!

I’ll show it to Jason too.

Thanks! :)

Thanks for the heads up on this request for support.

While I completely the understand the desire for this, s2Member really needs to stick with a standard here. The $_SERVER["REMOTE_ADDR"] is an environment variable that servers need to be configured with. I’m aware that some servers are NOT properly configured, but there are ways to deal with this at the server/hosting level.

I posted some additional details regarding this issue here, along with a popular workaround.
http://www.primothemes.com/forums/viewtopic.php?t=6380&p=14606#p14618

As soon as you use wordpress in a multiple webserver environment using a proxy as a frontend, you’ll have the issue.

I understand why you want to stick on it (after reading your forum post), but would it be possible to make this behavior as a setting checkbox then ? Because _everytime_ that we want to upgrade s2, we also need to re-apply the patch..

I could provide a patch for a setting checkbox if you would agreed to merge it :)