Broken episode thumbnail crashes MediaPortal (1 Viewer)

RoChess

Extension Developer
  • Premium Supporter
  • March 10, 2006
    4,434
    1,897
    Probably some conflict with the TheTVdB data, so you will have to analyze each language, find out error, update content, and then between a few seconds and a few hours the API ZIP files will be updated for MP-TVSeries to get fixed.
     

    emphatic

    Design Group
  • Team MediaPortal
  • August 25, 2006
    3,717
    1,178
    Alingsås
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #12
    Probably some conflict with the TheTVdB data, so you will have to analyze each language, find out error, update content, and then between a few seconds and a few hours the API ZIP files will be updated for MP-TVSeries to get fixed.

    While this is true, isn't it pretty bad to have this kind of vulnerability inside a plugin?
    Emph
     

    JimCatMP

    Documentation Group
  • Team MediaPortal
  • April 1, 2010
    652
    283
    Leeds
    Home Country
    United Kingdom United Kingdom
    Solved

    The jpg downloaded is corrupt, remove it and your fine [..../thumbs/MPTVSeriesBanners/American Gods/Episodes/1x5.jpg ].

    Image appearing on TheTVDB page looks good but click on it and it's clearly broken.

    I note issue already reported and a corrected image uploaded but that is not what is being delivered from cache at the moment.

    See http://thetvdb.com/banners/episodes/253573//6082597.jpg> to get image to save it yourself, see <[Bad Thumbnail] American Gods (253573) 1x05 - TheTVDB.com> for forum reference.

    Cheers - JCMP
     

    milhouse

    Portal Pro
    November 9, 2006
    363
    54
    50
    Raleigh, NC U.S.
    Home Country
    United States of America United States of America
    In case you are curious, 12 Monkeys S03E09 seems to have the exact same issue.

    Glad to see this thread - weird issue, thought it was just me. Glad y'all have figured it out for me!

    Milhouse
     

    RoChess

    Extension Developer
  • Premium Supporter
  • March 10, 2006
    4,434
    1,897
    While this is true, isn't it pretty bad to have this kind of vulnerability inside a plugin?
    Emph
    Programming = 1 line to do something
    Error-free = 1000 lines of code to do the same thing, but account for every possible condition that can go wrong and properly handle it
    Reality = 1001 lines of code next day, because you overlooked something that was never thought possible
    1002 lines of code after that
    1003 lines of code after that
    ...

    you get the point ;)
     

    emphatic

    Design Group
  • Team MediaPortal
  • August 25, 2006
    3,717
    1,178
    Alingsås
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #18
    Sure, but for a user that has no idea what's wrong will think MediaPortal sucks and uninstall it the 3rd or 4th time this happens. Some sort of safety net for when media is out of control would just be good practice IMHO.

    Emph
     

    RoChess

    Extension Developer
  • Premium Supporter
  • March 10, 2006
    4,434
    1,897
    Yup, not saying you're wrong, just that it is not easy.

    Not sure if @ltfearme is still active on development, so somebody might need to look at the GitHub code and submit a patch, or assist in managing the project.
     

    JimCatMP

    Documentation Group
  • Team MediaPortal
  • April 1, 2010
    652
    283
    Leeds
    Home Country
    United Kingdom United Kingdom
    Hi all

    Assumption is this is in the plugin, but I'm not so sure - I'd assume the plugin call's standard core routines to handle image display - the DB stuff handles the path [all of which is good] and the import, but hands actual display off...... Adding image 'verification' to the plugin may be significant overhead which plugin dev may not consider appropriate - given the SRC should [will be?] correct - as RoChess says, tis far from a simple:eek:

    Idiot proof is impossible, there are just so many of them and they are beyond inventive:oops: - cretin resistant is the best you can hope for [and MPTVSeries is very resistant generally:love:

    Cheers - JCMP.
     

    Users who are viewing this thread

    Top Bottom