Ok sebb, but what about the target rfc we want to support ? Thanks On Thu, Mar 17, 2016 at 9:54 PM, sebb <[email protected]> wrote:
> Ideally JMeter should delegate the redirect decision to the HTTP library. > > However this may not be possible for all libraries. > > On 17 March 2016 at 20:46, Philippe Mouawad <[email protected]> > wrote: > > +1 for this change > > > > @all thoughts ? > > > > On Thu, Mar 17, 2016 at 4:15 PM, UBIK LOAD PACK Support < > > [email protected]> wrote: > > > >> Hello Team, > >> Is JMeter 3.0 supposed to support : > >> > >> - https://tools.ietf.org/html/rfc7231 > >> > >> > >> 1/ If so there may be an issue in support of Redirect: > >> > >> - https://tools.ietf.org/html/rfc7231#section-6.4 > >> > >> > >> Currently only Head is considered as Safe: > >> > >> - > >> > https://github.com/apache/jmeter/blob/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java#L1504 > >> > >> > >> But as per https://tools.ietf.org/html/rfc7231#section-4.2.1: > >> > >> GET, HEAD, OPTIONS, and TRACE methods are defined to be safe. > >> > >> > >> So if JMeter is supposed to respect rfc7231, this should be fixed. > >> > >> 2/ Besides, as per https://tools.ietf.org/html/rfc7231#section-6.4.7, > 307 > >> redirect code does not allow to change the initial method , so 307 > redirect > >> code should be managed differently > >> > >> > >> We can create a PR to fix this. > >> > >> Thanks > >> > >> Regards > >> UbikLoadPack Support Team > >> > > > > > > > > -- > > Cordialement. > > Philippe Mouawad. > -- Cordialement. Philippe Mouawad.
