jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Philippe Mouawad <philippe.moua...@gmail.com>
Subject Re: Management of Redirect and rfc7231
Date Thu, 17 Mar 2016 21:43:57 GMT
Ok sebb, but what about the target rfc we want to support ?
Thanks

On Thu, Mar 17, 2016 at 9:54 PM, sebb <sebbaz@gmail.com> 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 <philippe.mouawad@gmail.com>
> wrote:
> > +1 for this change
> >
> > @all thoughts ?
> >
> > On Thu, Mar 17, 2016 at 4:15 PM, UBIK LOAD PACK Support <
> > support@ubikloadpack.com> 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.

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message