Support strms

[whiteowl3]

basic prototype for using strms
fixes #549

[whiteowl3]

@neilsb if you would kindly confirm or deny your interest in this functionality I will (or will not) try to set up a test server for you. I understand testing remote mtls streams is probably more annoying than your average PR.

[neilsb]

I'm more than happy for it to be added. It's something I was intending to try and understand how works at the weekend there, but didn't have the time. I'll try to get to it this week. Sorry for the delay.

[whiteowl3]

i was not able to fully test these final changes on my infrastructure, just as a heads up.

[neilsb]

i was not able to fully test these final changes on my infrastructure, just as a heads up.

What is it you've been unable to test? I've been able to able to test strm hosted on localhost and external URL, and it appears to work OK.

[whiteowl3]

i was not able to fully test these final changes on my infrastructure, just as a heads up.

What is it you've been unable to test? I've been able to able to test strm hosted on localhost and external URL, and it appears to work OK.

just that. my infrastructure has all the mtls stuff, so i can test all the functionality up until the final gutting of those extra parts and bsfmt-ing.

[neilsb]

So this seems OK now. If you're happy with how the get_url() interaction works I am good to approve and merge?

[whiteowl3]

So this seems OK now. If you're happy with how the get_url() interaction works I am good to approve and merge?

Im still stuck on lines 138 and 139. Do they not need to be moved out of the if block?

[whiteowl3]

actually i think its just audiotrack selection that needs to be brought out.

[whiteowl3]

pretty sure its supposed to be like this. i haven't tested this modification at all though because im not at home. ill test this evening if you dont beat me to it.

[whiteowl3]

@neilsb ok I'm happy when you're happy.

[neilsb]

All seems to work.