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.

Sign up for GitHub

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

Jump to bottom

Change NZB processing from GET to POST #2

Closed
tudstudent opened this issue Jul 3, 2014 · 7 comments
Closed

Change NZB processing from GET to POST #2

tudstudent opened this issue Jul 3, 2014 · 7 comments

Comments

@tudstudent
Copy link

@Blauwbek can you reattach your PR?
@Blauwbek
Copy link
Contributor

Blauwbek commented Jul 3, 2014

What do you mean? I've been less active on github recently so you'll have to freshen my memory...

@tudstudent
Copy link
Author

You created a patch to push the nzb into sabnzbd with the post protocol instead of the get protocol.
This to avoid the memory limit in the server config for get and the browser limitation for get.

@ghost
Copy link

ghost commented Jul 10, 2014

I think you mean my commit:
https://github.com/DirtyDutch/spotweb/commit/4eafb3edf8286ce30c3b74341ca2fe9a70ae9389

Would love to add it to this fork, but there is no mediainfo branch.

@tudstudent
Copy link
Author

Correct. That is the patch I am refering to.

@muis
Copy link
Member

muis commented Jul 17, 2014

@DirtyDutch Can you create a pull request for the mediainfo branch (including your commit)?

@ghost ghost mentioned this issue Aug 6, 2014
Closed
@ckruijntjens
Copy link

Hi There wy is spotweb not up to date? No mobile template etc?

Almost never updates when i do a git pull?

Greetings chris

@Blauwbek
Copy link
Contributor

Because there is no longer an 'official' spotweb repo

@muis muis closed this as completed Jan 15, 2015
@WorkaholicGoT WorkaholicGoT mentioned this issue Feb 12, 2015
Closed
@WorkaholicGoT WorkaholicGoT mentioned this issue Mar 31, 2015
Closed
@WzL WzL mentioned this issue Apr 7, 2015
Closed
@Zeagle88 Zeagle88 mentioned this issue Apr 15, 2015
Closed
@rcrommentuijn rcrommentuijn mentioned this issue May 13, 2015
Closed
@Northguy Northguy mentioned this issue May 15, 2015
Closed
@beta990 beta990 mentioned this issue Jun 18, 2015
Closed
@jwallden jwallden mentioned this issue Jul 30, 2015
Closed
@onitreb onitreb mentioned this issue Oct 13, 2015
Closed
@Rheimans Rheimans mentioned this issue Oct 21, 2015
Closed
@MrHappy MrHappy mentioned this issue Dec 5, 2015
Closed
@mbnn mbnn mentioned this issue Mar 7, 2016
Closed
@wardboumans wardboumans mentioned this issue Apr 6, 2016
Closed
@eboman eboman mentioned this issue Apr 19, 2016
Closed
@Casburggraaf Casburggraaf mentioned this issue Jun 15, 2016
Closed
@basnagel basnagel mentioned this issue Jun 17, 2016
Closed
@ThEWiZ76 ThEWiZ76 mentioned this issue Oct 29, 2016
Closed
GeoffreyDijkstra pushed a commit that referenced this issue Nov 4, 2016
@mbirth
Merge pull request #2 from GeoffreyDijkstra/feature/composer
1160dc0 
Adding name/description/license
Sweepr added a commit that referenced this issue Dec 1, 2016
@Sweepr
Merge pull request #2 from spotweb/master
dab3cdc 
Pull
Sweepr added a commit that referenced this issue Dec 1, 2016
@Sweepr
Merge pull request #240 from Sweepr/master
9cb237d 
Merge pull request #2 from spotweb/master
@mesa57 mesa57 mentioned this issue Dec 7, 2016
Closed
@ghost ghost mentioned this issue Dec 20, 2016
Closed
@GieltjE GieltjE mentioned this issue Mar 19, 2017
Closed
@sandervandegeijn sandervandegeijn mentioned this issue Apr 22, 2017
Closed
@Mickroz Mickroz mentioned this issue May 15, 2017
Closed
@vpieper vpieper mentioned this issue Nov 8, 2017
Closed
@heula heula mentioned this issue Feb 10, 2018
Closed
@r-vd-woude r-vd-woude mentioned this issue May 3, 2018
Closed
This was referenced Jul 26, 2018
Closed
Closed
@mesa57 mesa57 mentioned this issue Aug 6, 2019
Merged
@Hydranet Hydranet mentioned this issue Sep 2, 2019
Closed
@bizziebis bizziebis mentioned this issue Oct 4, 2019
Closed
@frankvHoof93 frankvHoof93 mentioned this issue Oct 9, 2020
Closed
@Sweepr Sweepr mentioned this issue Dec 30, 2020
Closed
@sjakief sjakief mentioned this issue Jan 18, 2021
Closed
@Sweepr Sweepr mentioned this issue Jan 23, 2021
Merged
mesa57 added a commit that referenced this issue Feb 28, 2021
@mesa57
Merge pull request #2 from mesa57/analysis-BMnWrL
b3e73d2 
Apply fixes from StyleCI
@gerarddvb gerarddvb mentioned this issue May 1, 2021
Closed
@bvrielink bvrielink mentioned this issue Jul 4, 2021
Closed
@joombler joombler mentioned this issue Jan 9, 2022
Closed
@phuongpham phuongpham mentioned this issue Dec 6, 2022
Closed
@Kroesss Kroesss mentioned this issue Dec 26, 2022
Closed
@Superbitcomputing Superbitcomputing mentioned this issue Jun 15, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Mediainfo
4 participants
@tudstudent @Blauwbek @muis @ckruijntjens