You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@jmeter.apache.org by bu...@apache.org on 2015/02/20 21:17:31 UTC

[Bug 57613] New: Patch proposal: Add CalDAV verbs to HTTP Sampler

https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

            Bug ID: 57613
           Summary: Patch proposal: Add CalDAV verbs to HTTP Sampler
           Product: JMeter
           Version: 2.12
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: HTTP
          Assignee: issues@jmeter.apache.org
          Reporter: richard.brigham@teamaol.com

Created attachment 32502
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=32502&action=edit
Diff adding support for REPORT and MKCALENDAR

In 2.12, the patch for 57107 added support for WebDAV verbs to the HTTP
Sampler.  This has been a great addition for WebDAV testing but the CalDAV
standard (RFC 4791) adds 2 new methods, REPORT and MKCALENDAR.  Attached is a
patch based on the 2.12 source that adds support for these. REPORT is also
useful for CardDAV testing.

My team has been using this patch successfully in our testing.  Can it be added
to jmeter?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57613] HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

--- Comment #5 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
(In reply to Felix Schumacher from comment #4)
> (In reply to Philippe Mouawad from comment #3)
> > Hi,
> > Thanks for patch.
> > Please note there was some missing part in implementation, I am surprised
> > that patch worked as it was.
> > I fixed it , could you test with upcoming nightly build ?
> > Thanks
> 
> To be fair, the missing part was part of the original patch.

Oups sorry , it was me, I misread the diff, thanks for check Felix..

@Richard thanks for patch and no more comment :-)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57613] HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om
            Summary|Patch proposal: Add CalDAV  |HTTP Sampler : Added CalDAV
                   |verbs to HTTP Sampler       |verbs (REPORT, MKCALENDAR)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57613] HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

--- Comment #2 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Date: Fri Feb 20 21:29:08 2015
New Revision: 1661229

URL: http://svn.apache.org/r1661229
Log:
Bug 57613 - HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)
Bugzilla Id: 57613

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HttpWebdav.java

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57613] HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Date: Fri Feb 20 20:57:37 2015
New Revision: 1661219

URL: http://svn.apache.org/r1661219
Log:
Bug 57613 - HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)
Bugzilla Id: 57613

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/util/HTTPConstantsInterface.java
    jmeter/trunk/xdocs/changes.xml
    jmeter/trunk/xdocs/usermanual/component_reference.xml

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57613] HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

--- Comment #4 from Felix Schumacher <fe...@internetallee.de> ---
(In reply to Philippe Mouawad from comment #3)
> Hi,
> Thanks for patch.
> Please note there was some missing part in implementation, I am surprised
> that patch worked as it was.
> I fixed it , could you test with upcoming nightly build ?
> Thanks

To be fair, the missing part was part of the original patch.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57613] HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

--- Comment #6 from Richard Brigham <ri...@teamaol.com> ---
That change worked perfectly in the nightly build. Thanks for adding it.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57613] HTTP Sampler : Added CalDAV verbs (REPORT, MKCALENDAR)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57613

--- Comment #3 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Hi,
Thanks for patch.
Please note there was some missing part in implementation, I am surprised that
patch worked as it was.
I fixed it , could you test with upcoming nightly build ?
Thanks

-- 
You are receiving this mail because:
You are the assignee for the bug.