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

CouchPotato not working #362

Open
eangulus opened this issue May 14, 2014 · 24 comments · May be fixed by #370 or #382
Open

CouchPotato not working #362

eangulus opened this issue May 14, 2014 · 24 comments · May be fixed by #370 or #382

Comments

@eangulus
Copy link

Add correct API Key and module never shows.

Not even sure if its a problem here or with CP, but have been talking to the CP Dev, and he says that the database has changed slightly in CP and that developers need to adjust their software to suit. Hence why I am reporting it here.

Using Latest Maraschino build as of writing this, and using CP Dev build fully upto date.

Logs:

http://pastebin.com/bkSrNAcM

@ovq
Copy link

ovq commented May 31, 2014

Same problem here. CP stopped working several Maraschino builds ago.
I am on the developer fork of CP, however, so this might be the reason in my case.

@dannyb78
Copy link

CP is working for me, I'm running latest stable maraschino.

@ovq
Copy link

ovq commented May 31, 2014

@dannyb78 which version of CP are you running? Latest stable or developer?

@dannyb78
Copy link

latest stable. thats the about:
Version 240283405e5389a77642ccfd68d619313142635e (7/5/2014 10:50:36)
Updater source, master
ID6257
all is running in a freenas box, with standard plugins pbi.

@ovq
Copy link

ovq commented May 31, 2014

Then maybe my CP developer fork screws up the integration...

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 4, 2014

It's possible one of the movies in your wanted list is missing some info Maraschino expects. CP is not very consistent with the info it replies.

@eangulus
Copy link
Author

eangulus commented Jun 4, 2014

Not likely considering I don't use CP for NFO data. I use either whatever XBMC saves or Ember media manager.

@eangulus
Copy link
Author

eangulus commented Jun 4, 2014

The problem is that the dev branch of CP has had some changes to the database, and therefore all my apps/addons will no longer work with CP until they are updated to use the new dB.

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 4, 2014

The CP modules grabs data from CP only, not sure what you mean by XBMC in this case.
And I am assuming you mean the API since we do not touch the db for that very reason. I have lately switched to the movie watchlist feature in DogNZB so I cant really comment on CP (I only used it's post processing features).

@eangulus
Copy link
Author

eangulus commented Jun 4, 2014

I don't developed CP and am only forwarding information I was told by the CP dev. Appatently there has been changes to the database which would most likely in turn mean changes to the API to suit.

And as to XBMC I thought you were talking about the NFO data

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 4, 2014

I see. It's possible there were changes as I havent had a read through CP's documentation in a long time. You can try going to: http://MARASCHINO_URL/xhr/couchpotato/ and see if you get any errors there.

@eangulus
Copy link
Author

eangulus commented Jun 4, 2014

I ge the following with that URL.

Internal Server Error

The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 4, 2014

Ok, I can see they changed a few things. I'll see if i can push a quick patch in a moment.

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 4, 2014

@eangulus I saw they have had a tone of updates in the last few days so I updated a few sections of the Maraschino code. Give it a shot and let me know if it has worked for you.

@eangulus
Copy link
Author

eangulus commented Jun 4, 2014

Still not working. Heres the logs:

http://pastebin.com/djxpdLRc

@eangulus
Copy link
Author

eangulus commented Jun 4, 2014

Noticed these lines in particular:

Jun-04 19:32:56 :: DEBUG :: CouchPotato :: EXCEPTION -- No JSON object could be decoded: line 1 column 0 (char 0)

@eangulus
Copy link
Author

eangulus commented Jun 5, 2014

Just tried something.

The CP Module seems to work when set to compact view but doesn't otherwise.

@matt7277
Copy link

matt7277 commented Jun 6, 2014

Also seems to have broken my CP Module with the latest update.... worked
fine before. running the latest CP as far as I can tell. I haven't dived
through the logs just yet though..

Matt

On Thu, Jun 5, 2014 at 6:11 AM, Shaun Williams notifications@github.com
wrote:

Just tried something.

The CP Module seems to work when set to compact view but doesn't otherwise.


Reply to this email directly or view it on GitHub
#362 (comment)
.

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 6, 2014

This is the version I am coding for: 80559d3 (6/4/2014 6:31:20 AM)
It's only a couple of days old so make sure this is the one you have. It looks like they changed a lot of things in their API, mostly tho the structure of the response.

@eangulus
Copy link
Author

eangulus commented Jun 6, 2014

I have the latest version and its not that.

I have: d252b660 (6/4/2014 4:18:41 PM)

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 6, 2014

Well this is weird. This is what I see on the master branch over at https://github.com/RuudBurger/CouchPotatoServer/tree/master

http://i.imgur.com/X8dqFR2.png

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 6, 2014

That time seems to match with my version so I am not sure whats up.

@eangulus
Copy link
Author

eangulus commented Jun 6, 2014

I know, as per written in the first comment, I already stated I am on the Dev build.

@gugahoi
Copy link
Collaborator

gugahoi commented Jun 6, 2014

Oh i misinterpreted your last comment I think. I cant really support the dev branch tho as most people are not on it so if there is a significant change there then we will have to wait till it trickles down to the master branch.

miri64 added a commit to miri64/maraschino that referenced this issue Aug 6, 2014
@miri64 miri64 linked a pull request Aug 6, 2014 that will close this issue
@imro2 imro2 linked a pull request Dec 19, 2014 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants