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

NFO partsing problem for IMDb link #793

Closed
Omertron opened this issue Mar 15, 2015 · 4 comments
Closed

NFO partsing problem for IMDb link #793

Omertron opened this issue Mar 15, 2015 · 4 comments

Comments

@Omertron
Copy link
Member

Original issue 794 created by Omertron on 2009-05-16T21:52:26.000Z:

This is strange...
I just started a new run of YAMJ on a new set of movies.
The NFO file contains at the end of the file:

SOURCE..........: The Boondock Saints 1999 1080p Blu-ray
IMDB............: http://www.imdb.com/title/tt0144117

Post Information
ENCODER.........: xxx
RELEASE DATE....: 02/27/2008

The output in the DOS box showed:
Updating data for: Boondock Saints 1999 UR
Failed retreiving imdb rating for movie : tt0144117Post
java.io.FileNotFoundException: http://www.imdb.com/title/tt0144117/Post/
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native
Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(Unknown
Source)
at
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(Unknown Source)
at java.lang.reflect.Constructor.newInstance(Unknown Source)
at sun.net.www.protocol.http.HttpURLConnection$6.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at
sun.net.www.protocol.http.HttpURLConnection.getChainedException(Unknown Source)
at
sun.net.www.protocol.http.HttpURLConnection.getInputStream(Unknown Source)
at com.moviejukebox.tools.WebBrowser.request(WebBrowser.java:85)
at com.moviejukebox.tools.WebBrowser.request(WebBrowser.java:53)
at
com.moviejukebox.plugin.ImdbPlugin.updateImdbMediaInfo(ImdbPlugin.java:258)
at com.moviejukebox.plugin.ImdbPlugin.scan(ImdbPlugin.java:62)
at
com.moviejukebox.plugin.DatabasePluginController.scan(DatabasePluginController.java:43)
at com.moviejukebox.MovieJukebox.updateMovieData(MovieJukebox.java:704)
at com.moviejukebox.MovieJukebox.generateLibrary(MovieJukebox.java:419)
at com.moviejukebox.MovieJukebox.main(MovieJukebox.java:232)
Caused by: java.io.FileNotFoundException:
http://www.imdb.com/title/tt0144117/Post/
at
sun.net.www.protocol.http.HttpURLConnection.getInputStream(Unknown Source)
at
sun.net.www.protocol.http.HttpURLConnection.getHeaderFields(Unknown Source)
at com.moviejukebox.tools.WebBrowser.readHeader(WebBrowser.java:162)
at com.moviejukebox.tools.WebBrowser.request(WebBrowser.java:83)
... 7 more

As you can see it's trying to find tt0144117Post
which seems to be two lines merged together as if because there was no
trailing '/' in the URL link for IMDb, it merged the lines.

Adding a trailing '/' fixed the problem.

@Omertron
Copy link
Member Author

Comment #1 originally posted by Omertron on 2009-05-17T06:13:25.000Z:

So the IMDB URL should be terminated with a "/" or EOL.

@Omertron
Copy link
Member Author

Comment #2 originally posted by Omertron on 2009-05-28T05:17:09.000Z:

I've been away for a while...

Yes, it seems that the trailing '/' is required.

@Omertron
Copy link
Member Author

Comment #3 originally posted by Omertron on 2009-06-09T22:33:47.000Z:

One of the issues is that its not always EOL and that there is trailing ASCII art or
alot of spaces or a /n (regex) that wrap the line. The URL should be parsed in a way
that it trims any characters after a break in the string and then auto append the "/"
if its really needed as I suspect its probably not to make the web call.

@Omertron
Copy link
Member Author

Comment #4 originally posted by Omertron on 2010-02-09T20:18:17.000Z:

Hi,
fixed in r1355.

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

No branches or pull requests

1 participant