Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

Missing TLS certificate verification #169

Open
mcatanzaro opened this issue May 25, 2021 · 4 comments
Open

Missing TLS certificate verification #169

mcatanzaro opened this issue May 25, 2021 · 4 comments

Comments

@mcatanzaro
Copy link

Hi, libdylena's device.c does not appear to be enabling TLS certificate verification on the SoupSessionAsync object it creates. You have to do this manually when using the deprecated SoupSessionAsync.

phako added a commit to phako/dleyna-server that referenced this issue Aug 31, 2021
@phako
Copy link

phako commented Aug 31, 2021

I suppose the linked fix above should suffice?

@mcatanzaro
Copy link
Author

I suppose the linked fix above should suffice?

That's the best way to fix this issue, yes, assuming you've read the porting guide and are OK with the differences.

@mcatanzaro
Copy link
Author

Note there is an identical issue in dleyna-renderer: intel/dleyna-renderer#171

@phako
Copy link

phako commented Aug 31, 2021

Yes that should not matter here. I have fixed the renderer issue as well, the commit should be referenced in that ticket.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants