-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Strip URL parameters (#25) #121
base: master
Are you sure you want to change the base?
Conversation
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 think this would make more sense to move this check to L440/441:
return ext.AsSpan().LeftPart('?').ToString();
From what I can see this change shouldn't break anything.
@bchretien Shadowghost suggest to move line to L440. Hope it will be ok to merge. Best regards |
@bchretien can you take a look at the suggested changes so the pr can be merged ? |
I tried to build and use your version. But multiple issues:
It's probably the fault of orange/softathome bad software but if anybody has any idea on how to find a better alternative.. Here's logs, (Moana works, mufasa doesn't):
|
Hi,
This is a PR containing a patch listed in #25 to make the DLNA plugin work with the Orange TV Decoder. From my understanding, it strips URL parameters. I'm not sure of the impact for the rest of the DLNA clients as they may rely on the URL parameters. Maybe a per-profile option to strip these parameters would be more suitable?