Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#7514 closed defect (fixed)

Jamu script using wrong address for image download

Reported by: mythtv@… Owned by: Anduin Withers
Priority: minor Milestone: 0.22
Component: Plugin - MythVideo Version: unknown
Severity: medium Keywords: Jamu
Cc: Ticket locked: yes

Description

For the last few days, Jamu has been failing with the following error:

! Warning: Download IOError on URL for Filename(/home/mythtv/.mythtv/MythVideo/Fanart/Strictly Come Dancing_fanart.jpg) Orginal URL(http://images.thetvdb.com/banners/fanart/original/83127-1.jpg) IOError urllib.quote URL(http://images.thetvdb.com/banners/fanart/original/83127-1.jpg)

This is occuring for all shows it trys to download. It seems the script should be using www.thetvdb.com instead of images.thetvdb.com as specified in this forum post:

http://forums.thetvdb.com/viewtopic.php?f=4&t=2161

Changing the script manually to this address has fixed it

Attachments (1)

jamu_do_not_use_mirrors.patch (1.8 KB) - added by r.d.vaughan@… 10 years ago.
Remove forcing the TVDB URL to the mirrors web site

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by r.d.vaughan@…

Remove forcing the TVDB URL to the mirrors web site

comment:1 Changed 10 years ago by r.d.vaughan@…

As recommended in the TVDB forum removed the forcing of TVDB graphics downloads to use the mirror site. See patch above.

comment:2 Changed 10 years ago by robertm

(In [22761]) Refs #7514, undo URL replacement code in Jamu-- TVDB have removed their mirror for now-- this may require another change if they reintroduce it/start URL forwading again, but this will work for now.

comment:3 Changed 10 years ago by robertm

(In [22763]) Refs #7514, undo URL replacement code in Jamu-- TVDB have removed their mirror for now-- this may require another change if they reintroduce it/start URL forwading again, but this will work for now. (Backport to .22-fixes)

comment:4 Changed 10 years ago by robertm

Resolution: fixed
Status: newclosed

Fixed in r22763.

comment:5 Changed 10 years ago by Kode

The mirror hasn't been removed, you just can't hard code the path as it stops load balancing, by using the path www.thetvdb.com it allows the thetvdb to transparently load balance between mirrors as needs be.

comment:6 Changed 10 years ago by robertm

Ticket locked: set

The mirror is not being kept up to date and redirection has been turned off, leaving the mirror functionally disabled. Discussion belongs on lists, not tickets.

Note: See TracTickets for help on using tickets.