Re: Rest-API hanging on DELETE and OPTIONS not supported.

Ralf Schlatterbeck <[email protected]>
Newsgroups gmane.comp.bug-tracking.roundup.devel
Message-ID <[email protected]>
On Mon, Feb 11, 2019 at 07:11:53AM +0100, Thomas Arendsen Hein wrote:
> * John P. Rouillard <[email protected]> [20190202 21:37]:
> > Using the OPTIONS method returns 501 (note the swagger-ui actually
> > hangs until I ^C'ed the roundup-server). I then used their demo curl
> > command to see what was happening. This patch fixes that:
> > 
> > diff -r 5fc476d4e34c roundup/scripts/roundup_server.py
> > --- a/roundup/scripts/roundup_server.py Wed Jan 30 18:11:02 2019 +0100
> > +++ b/roundup/scripts/roundup_server.py Sat Feb 02 15:21:42 2019 -0500
> > @@ -255,7 +255,7 @@
> >                      print('EXCEPTION AT', ts)
> > 		                          traceback.print_exc()
> > 
> > -    do_GET = do_POST = do_HEAD = do_PUT = do_DELETE = do_PATCH = run_cgi
> > +    do_GET = do_POST = do_HEAD = do_PUT = do_DELETE = do_PATCH = do_OPTIONS = r
> 
> This was in the original REST patch series, but seems to have been
> lost at some point. Ralf, I'm not blaming you, but may I blame git
> on this? ;-)

No there was a conflict, the bpo branch calls a debug method there which
was a merge conflict and I removed the debug method because I didn't
want to keep it. Seems I've overlooked the OPTIONS method when merging.
I had added many of the other option though which were not implemented
on the default branch.

Ralf
-- 
Dr. Ralf Schlatterbeck                  Tel:   +43/2243/26465-16
Open Source Consulting                  www:   http://www.runtux.com
Reichergasse 131, A-3411 Weidling       email: [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.