Project

General

Profile

Bug #1574

imagecache does not support https located icons

Added by Carlo Landmeter almost 12 years ago. Updated almost 12 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
General
Target version:
-
Start date:
2013-01-28
Due date:
% Done:

100%

Estimated time:
Found in version:
current master branch
Affected Versions:

Description

When setting the icon location to github https, icons cannot be cached by tvh.


Files

t.patch (489 Bytes) t.patch ssl cert ignore Adam Sutton, 2013-01-30 15:12

History

#1

Updated by Adam Sutton almost 12 years ago

  • Status changed from New to Need feedback

Interesting, I've just checked this and I cannot replicate. I have tried downloading images from gravatar and github using https and both were fine.

The only thing that didn't work for me is from one of my own servers that uses self-signed certs and certificate validation is enabled by default.

Can you provide an example URL that you are finding does not work.

Adam

#2

Updated by Carlo Landmeter almost 12 years ago

Does tvh curl try to validate the cert? If so i probably need ca-certificates installed (which i have not).

#3

Updated by Adam Sutton almost 12 years ago

Yes, cert validation is enabled by default. We could disable it, I'm not sure its a massive security issue for this particular feature. But I'd rather not unless we need to (would probably make it user configurable option if required).

Adam

#4

Updated by Carlo Landmeter almost 12 years ago

Could a configure switch be optional? else ill have to add it to deps in our repository.

#5

Updated by Adam Sutton almost 12 years ago

It's possible, though might as well make it run-time configurable. Try attached patch first just to check it works.

Adam

#6

Updated by Adam Sutton almost 12 years ago

  • Status changed from Need feedback to Fixed
  • % Done changed from 0 to 100

Applied in changeset commit:53a9d6a4baa3eb9ee7415e46ee6b8612d962a343.

#7

Updated by Carlo Landmeter almost 12 years ago

Thanks for the fix.

Last week I tried your patch, but run into issues when I run XBMC v12 with the updated pvr addon. When I turn on imagecaching then it doesn't seem to work anymore.
It worked with the older pvr addon, but it seems the one that should have support for it (which i read in the pvr changelog) it doesn't.

I tried it both on OpenELEC and Windows version. Should i create a separate bug for it?

Also available in: Atom PDF