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

Include user/pass in channel URL #39

Merged
merged 1 commit into from
Dec 31, 2021
Merged

Conversation

tenfire
Copy link
Contributor

@tenfire tenfire commented Jul 31, 2019

This solved #26 for me

@jfarseneau jfarseneau self-requested a review August 4, 2019 18:02
@jfarseneau
Copy link
Owner

So this used to be the way things were done with Tvheadend, but then they changed the auth logic. I'll test further (as nothing seems to work and I'm really hating Tvheadend at the moment) but it's possible this fixes older setups and breaks newer setups. Which would be a huge pain.

I'm hoping it'll work for both however.

@Marlinc
Copy link

Marlinc commented Nov 3, 2019

This is working for me using TVHeadend 4.3-1852~gdea96e441

@ledfloyd1984
Copy link

@jfarseneau Could you please approve this PR and create a new release please? I know that this is practically a dead repo, but it would still benefit my situation greatly.

@jfarseneau
Copy link
Owner

Sure, this is indeed a dead repo, and I have no way to test it right now, but I'll take tenfire and Marlinc's word for it. Merging!

And this is very late, but thank you @tenfire for the contribution. Happy new year!

@jfarseneau jfarseneau merged commit 07a7628 into jfarseneau:master Dec 31, 2021
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.

4 participants