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

API change - Sonarr Torznab pagesize limits #250

Open
Taloth opened this issue Feb 17, 2016 · 10 comments
Open

API change - Sonarr Torznab pagesize limits #250

Taloth opened this issue Feb 17, 2016 · 10 comments

Comments

@Taloth
Copy link

Taloth commented Feb 17, 2016

I've made a commit for Sonarr develop that will read the 'limits' caps attribute to determine the page size instead of keeping it hardcoded to the 100 newznab default.
<limits max="100" default="100"/>

You may want to add that element in your t=caps responses and adjust the value as appropriate for each tracker. That way sonarr will know how big the page is supposed to be, and fetch multiple pages. Obviously only works if jackett appropriately handles the &offset=xx&limit=xx queryparam.

A bit a continuation of our chat in #27.

Can you guys look into it, and potentially test it on the develop version of Sonarr?

Here's the commit please note that max and default should be the same value.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@ghost
Copy link

ghost commented Feb 27, 2016

Thanks for the heads up @Taloth
We have started a complete rewrite of Jackett based on .NET Core. We'll definitely be supporting this along with paging in the rewrite.

@Taloth
Copy link
Author

Taloth commented Feb 27, 2016

Awesome, good to hear.

@JigSawFr JigSawFr added the Core label Apr 2, 2016
@cybercorey
Copy link

Hey, what does running it on the .net core mean for us Linux users running mono?

@JigSawFr
Copy link
Collaborator

JigSawFr commented Apr 6, 2016

@cybercorey With .net core, mono will be no longer used.

@cybercorey
Copy link

Will we still be able to run this on linux?

@JigSawFr
Copy link
Collaborator

JigSawFr commented Apr 6, 2016

Yes of course !

@mrjackyliang
Copy link

I tried this, it doesn't work.

@bonfire62
Copy link

Any update on this?

@mrjackyliang
Copy link

The offset and limit works for me now.

@B1773rm4n
Copy link

What is happening here?

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

No branches or pull requests

7 participants