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

Error when attempting to view manually-added videos #169

Closed
asmartin opened this issue May 1, 2016 · 7 comments
Closed

Error when attempting to view manually-added videos #169

asmartin opened this issue May 1, 2016 · 7 comments

Comments

@asmartin
Copy link

asmartin commented May 1, 2016

Hello,

I am currently running streama v0.2.0. I manually added a show that is not on The Movie DB and then uploaded an episode to it. This worked - I appear to have been able to successfully upload the video. However, when I go to view it, I get this error:

2016-04-30 19:24:08,938 [http-bio-80-exec-11] ERROR errors.GrailsExceptionResolver  - FileNotFoundException occurred when processing request: [GET] /theMovieDb/countNewEpisodesForSeason
https://api.themoviedb.org/3/tv/null/season/2007?api_key=bf3c0b3232cbfa73da393379c15d2688. Stacktrace follows:
java.io.FileNotFoundException: https://api.themoviedb.org/3/tv/null/season/2007?api_key=bf3c0b3232cbfa73da393379c15d2688
        at streama.TheMovieDbController.listNewEpisodesForSeason(TheMovieDbController.groovy:63)
        at streama.TheMovieDbController.countNewEpisodesForSeason(TheMovieDbController.groovy:81)
        at grails.plugin.cache.web.filter.PageFragmentCachingFilter.doFilter(PageFragmentCachingFilter.java:198)
        at grails.plugin.cache.web.filter.AbstractFilter.doFilter(AbstractFilter.java:63)
        at grails.plugin.springsecurity.web.filter.GrailsAnonymousAuthenticationFilter.doFilter(GrailsAnonymousAuthenticationFilter.java:53)
        at grails.plugin.springsecurity.web.authentication.RequestHolderAuthenticationFilter.doFilter(RequestHolderAuthenticationFilter.java:49)
        at grails.plugin.springsecurity.web.authentication.logout.MutableLogoutFilter.doFilter(MutableLogoutFilter.java:82)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
        at java.lang.Thread.run(Thread.java:745)

Is this a bug? My understanding is that since this is a manually-added video, no query to The Movie DB should be necessary (since it won't be found there). Please let me know what I can do to help test and fix this, thanks!!

@asmartin
Copy link
Author

asmartin commented Jul 4, 2016

Any updates on this?

@asmartin
Copy link
Author

asmartin commented Jul 4, 2016

No problem, thanks for the quick reply!
On Jul 4, 2016 1:39 PM, "Giorgi Kikolashvili" notifications@github.com
wrote:

Sorry, hadn't time to investigate it
On Jul 4, 2016 22:23, "Andrew Martin" notifications@github.com wrote:

Any updates on this?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#169 (comment),
or mute the thread
<
https://github.com/notifications/unsubscribe/ACdXhHVaOAkSYHgT3LFhjorHRqmj5z7lks5qSU-ygaJpZM4IThKW

.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#169 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ADuYu1Uk6dzdMfucbx5SOEv5V2fNB2Rhks5qSVNxgaJpZM4IThKW
.

@gkiko
Copy link
Collaborator

gkiko commented Jul 4, 2016

I thought it was another issue 😆
Will take a look anyways

@dularion
Copy link
Member

dularion commented Jul 4, 2016

Oh yeah, you're absolutely right, no query to themoviedb should be made. In all honesty, I never intended for that feature to be used when no moviedb api key is added at all. I'll try to take a look at it soon, or have you done anything here already @Gikko?

@asmartin
Copy link
Author

asmartin commented Jul 4, 2016

Most of what I've added to the library is on TheMovieDB, but I have a few
things that aren't (which I want to manually add)
On Jul 4, 2016 2:17 PM, "Antonia Hulha" notifications@github.com wrote:

Oh yeah, you're absolutely right, no query to themoviedb should be made.
In all honesty, I never intended for that feature to be used when no
moviedb api key is added at all. I'll try to take a look at it soon, or
have you done anything here already @Gikko?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#169 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ADuYu7VbGWzjraScJv-29zKKUNhaok9Wks5qSVxggaJpZM4IThKW
.

@dularion
Copy link
Member

dularion commented Jul 4, 2016

@dularion dularion closed this as completed Jul 4, 2016
dularion pushed a commit that referenced this issue Jul 4, 2016
@asmartin
Copy link
Author

asmartin commented Jul 5, 2016

Confirmed, this is now fixed. Thanks!

dularion pushed a commit that referenced this issue Jan 14, 2017
* origin/master:
  Update README.md
  Update README.md
  Update README.md
  Update README.md
  Update sample_application.yml
  Create a sample application.yml for v1.x deployment
  Typo
  Added Spanish translation
  Update README.md
  Update README.md
  fix Error when attempting to view manually-added videos #169
  Update README.md
  add try-catch for tvshow movieDb meta
  fixed header search
  Update README.md
  formatting
  added paypal link

# Conflicts:
#	grails-app/assets/javascripts/streama/translations/EN_us.js
#	grails-app/assets/javascripts/streama/translations/streama.translations.js
#	grails-app/views/layouts/main.gsp
#	streama.translations.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants