search

wyl1232792 / ice4j

Automatically exported from code.google.com/p/ice4j
0 stars 0 forks source link

HttpDemuxFilter missing PATCH request method #29

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. Send a packet via http containing a 'PATCH' request method (Jitsi Video 
Bridge, JVB, uses this verb for the REST API).  This is used for signaling in 
JVB, here is an example JSON body:  http://pastebin.com/CmLstLcP

2. HttpDemuxFilter will not accept() the packet

What is the expected output? What do you see instead?
'PATCH' method should be accepted as a valid HTTP verb/request method.

What version of the product are you using? On what operating system?
This issue is still present in current ice4j head, r507.

Additional info:
Without this change, Jitsi Video Bridge's rest API does not work to establish 
streaming media, as it relies on PATCH.

Original issue reported on code.google.com by innerve...@gmail.com on 30 Jun 2015 at 10:36

GoogleCodeExporter commented 8 years ago 
Attaching a patch file with the fix.  This will allow the Jitsi Video Bridge 
REST API to succeed again during signaling PATCH calls.

Original comment by innerve...@gmail.com on 1 Jul 2015 at 12:10

Attachments:

GoogleCodeExporter commented 8 years ago 
I didn't notice this patch and added "PATCH" in a separate commit.

Original comment by bo...@sip-communicator.org on 13 Jul 2015 at 9:43