Node-Media-Server icon indicating copy to clipboard operation
Node-Media-Server copied to clipboard

Added checks with promises

Open dev-steverob opened this issue 5 years ago • 8 comments
trafficstars

I have added an alternative to the Event Emit. This is a promise based workflow to allow for promise based request inside the callbacks and then rejecting the stream based on the callbacks return value. I have tried to keep the coding the same style as the current Event Emit. I have added examples to the Readme also.

dev-steverob avatar Feb 04 '20 11:02 dev-steverob

Please do not change the current code style. No spaces between function names and parentheses.

illuspas avatar Feb 20 '20 03:02 illuspas

Sorry it was my IDE that auto lints I will try to stop it and re submit

dev-steverob avatar Feb 20 '20 19:02 dev-steverob

Hello @dev-steverob, I think your PR is very interesting, could you reformat this and resubmit?

chrisspiegl avatar Apr 08 '20 16:04 chrisspiegl

Hello @dev-steverob, I think your PR is very interesting, could you reformat this and resubmit?

Once I get some time yes I will, I have extended my fork quite a bit more since by adding HLS secure keys to protect front end watching so I will have to split out the code again.

dev-steverob avatar Apr 09 '20 05:04 dev-steverob

Hello @dev-steverob, I think your PR is very interesting, could you reformat this and resubmit?

Once I get some time yes I will, I have extended my fork quite a bit more since by adding HLS secure keys to protect front end watching so I will have to split out the code again.

Hey @dev-steverob, this pr has a lot of benefits to us too (asynchronous validation of stream keys). If you are too busy I would be more than happy to clean this pr up for you. :)

mattmatters avatar Jul 20 '20 21:07 mattmatters

Hello @dev-steverob, I think your PR is very interesting, could you reformat this and resubmit?

Once I get some time yes I will, I have extended my fork quite a bit more since by adding HLS secure keys to protect front end watching so I will have to split out the code again.

Hey @dev-steverob, this pr has a lot of benefits to us too (asynchronous validation of stream keys). If you are too busy I would be more than happy to clean this pr up for you. :)

My folk has moved on a bit since this PR as I needed more functionality, but your welcome to take anything I have done and put it into a PR. You can message me if you need anything from me

dev-steverob avatar Jul 21 '20 05:07 dev-steverob

Hello @dev-steverob, I think your PR is very interesting, could you reformat this and resubmit?

Once I get some time yes I will, I have extended my fork quite a bit more since by adding HLS secure keys to protect front end watching so I will have to split out the code again.

Hey @dev-steverob, this pr has a lot of benefits to us too (asynchronous validation of stream keys). If you are too busy I would be more than happy to clean this pr up for you. :)

My folk has moved on a bit since this PR as I needed more functionality, but your welcome to take anything I have done and put it into a PR. You can message me if you need anything from me

Awesome, will do!

mattmatters avatar Jul 22 '20 22:07 mattmatters

Hello guys, this would be really interesting, there is no news on that?

lonasozo avatar Oct 20 '22 14:10 lonasozo