On Sun, 2015-12-13 at 22:28 +0100, Philippe Mouawad wrote:
> Finally, for those 2 I am not sure as per RFC6265, to use them , put code
> in org.apache.http.client.protocol.TestResponseProcessCookies.
> They worked with HC3 but it does not mean they should work with HC4, but I
> would like a confirmation:
>
I cannot get the tests to compile due to missing instance variables
cookieOrigin, cookieSpec and cookieStore. I'll take another look if you
post the complete class.
Oleg
>
> @Test
> public void testParseCookies() throws Exception {
> final HttpResponse response = new
> BasicHttpResponse(HttpVersion.HTTP_1_1, 200, "OK");
> response.addHeader(SM.SET_COOKIE, "name1=value1");
>
> final HttpClientContext context = HttpClientContext.create();
> context.setAttribute(HttpClientContext.COOKIE_ORIGIN,
> this.cookieOrigin);
> context.setAttribute(HttpClientContext.COOKIE_SPEC,
> this.cookieSpec);
> context.setAttribute(HttpClientContext.COOKIE_STORE,
> this.cookieStore);
>
> final HttpResponseInterceptor interceptor = new
> ResponseProcessCookies();
> interceptor.process(response, context);
>
> final List<Cookie> cookies = this.cookieStore.getCookies();
> List<Header> lstHdr = cookieSpec.formatCookies(cookies);
> StringBuilder sbHdr = new StringBuilder();
> for (Header header : lstHdr) {
> sbHdr.append(header.getValue());
> }
> org.junit.Assert.assertEquals("test1=1; test2=2", sbHdr.toString());
> }
> @Test
> public void testParseCookies2() throws Exception {
> final HttpResponse response = new
> BasicHttpResponse(HttpVersion.HTTP_1_1, 200, "OK");
> response.addHeader(SM.SET_COOKIE, "test1=1;secure, test2=2;secure");
>
> final HttpClientContext context = HttpClientContext.create();
> context.setAttribute(HttpClientContext.COOKIE_ORIGIN,
> this.cookieOrigin);
> context.setAttribute(HttpClientContext.COOKIE_SPEC,
> this.cookieSpec);
> context.setAttribute(HttpClientContext.COOKIE_STORE,
> this.cookieStore);
>
> final HttpResponseInterceptor interceptor = new
> ResponseProcessCookies();
> interceptor.process(response, context);
>
> final List<Cookie> cookies = this.cookieStore.getCookies();
> List<Header> lstHdr = cookieSpec.formatCookies(cookies);
> StringBuilder sbHdr = new StringBuilder();
> for (Header header : lstHdr) {
> sbHdr.append(header.getValue());
> }
> org.junit.Assert.assertEquals("test1=1; test2=2", sbHdr.toString());
> }
>
> Thanks
>
> On Sun, Dec 13, 2015 at 10:09 PM, Philippe Mouawad <
> philippe.moua...@gmail.com> wrote:
>
> > Hi,
> > Created https://issues.apache.org/jira/browse/HTTPCLIENT-1705 with a Test
> > case in it to show issue with Domain starting with ".".
> >
> >
> > Regards
> >
> > On Sun, Dec 13, 2015 at 9:50 PM, Philippe Mouawad <
> > philippe.moua...@gmail.com> wrote:
> >
> >> Hi Oleg,
> >> Created https://issues.apache.org/jira/browse/HTTPCLIENT-1705 to show
> >> issue with Cookie Header ordering.
> >>
> >> Regards
> >>
> >> On Sun, Dec 13, 2015 at 9:30 PM, Philippe Mouawad <
> >> philippe.moua...@gmail.com> wrote:
> >>
> >>> Hi,
> >>> I created https://issues.apache.org/jira/browse/HTTPCLIENT-1704 with
> >>> JUnit test case so that you understand the difference we have.
> >>>
> >>> Regards
> >>>
> >>> On Sat, Dec 5, 2015 at 1:26 PM, Oleg Kalnichevski <ol...@apache.org>
> >>> wrote:
> >>>
> >>>> On Fri, 2015-12-04 at 21:43 +0100, Philippe Mouawad wrote:
> >>>> > Thanks Oleg.
> >>>> > But look at
> >>>> >
> >>>> https://github.com/ubikloadpack/jmeter/blob/HC4_5/test/src/org/apache/jmeter/protocol/http/control/TestCookieManager.java#L368
> >>>> ,
> >>>> > it concerns IGNORE_POLICY
> >>>> >
> >>>> > I really think there is at least one bug.
> >>>> > Regards
> >>>> >
> >>>>
> >>>> Sure thing. All these issues may well be due to bugs in HC. Could you
> >>>> please though reproduce them with unit tests that do not involve JMeter
> >>>> specific code?
> >>>>
> >>>> Oleg
> >>>>
> >>>>
> >>>> > On Wed, Dec 2, 2015 at 4:00 PM, Oleg Kalnichevski <ol...@apache.org>
> >>>> wrote:
> >>>> >
> >>>> > > On Wed, 2015-12-02 at 15:51 +0100, Philippe Mouawad wrote:
> >>>> > > > Hi Oleg,
> >>>> > > > Thanks for answer.
> >>>> > > >
> >>>> > > > Find my answers inline.
> >>>> > > > Regards
> >>>> > > >
> >>>> > > > On Wed, Dec 2, 2015 at 3:36 PM, Oleg Kalnichevski <
> >>>> ol...@apache.org>
> >>>> > > wrote:
> >>>> > > >
> >>>> > > > > On Tue, 2015-12-01 at 22:09 +0100, Philippe Mouawad wrote:
> >>>> > > > > > Hello,
> >>>> > > > > > Any answer on this question ?
> >>>> > > > > > Thanks
> >>>> > > > > > Regards
> >>>> > > > > >
> >>>> > > > > > On Sun, Nov 15, 2015 at 8:39 PM, Philippe Mouawad <
> >>>> > > > > > philippe.moua...@gmail.com> wrote:
> >>>> > > > > >
> >>>> > > > > > > Hello,
> >>>> > > > > > > We're in the process of migrating JMeter to last HttpClient
> >>>> 4.5.1.
> >>>> > > > > > >
> >>>> > > > > > > We are now migrating the CookieManager to remove all
> >>>> deprecated
> >>>> > > code.
> >>>> > > > > > > To test this class, we had a JUNIT class :
> >>>> > > > > > >
> >>>> > > > > > >
> >>>> > > > >
> >>>> > >
> >>>> https://github.com/ubikloadpack/jmeter/blob/HC4_5/test/src/org/apache/jmeter/protocol/http/control/TestCookieManager.java
> >>>> > > > > > >
> >>>> > > > > > > Before migration, it was still testing HC3 version.
> >>>> > > > > > > So we first switched to HC4.23 and noticed some failures:
> >>>> > > > > > >
> >>>> > > > >
> >>>> > > > > I am confused. Are you using HC 4.2.3 or HC 4.5.1?
> >>>> > > >
> >>>> > > >
> >>>> > > > As I wrote, first we switched our JUnit tests to use HC4
> >>>> Implementation
> >>>> > > of
> >>>> > > > CookieHandler and used 4.2.3, we got some failures compared to
> >>>> HC3.
> >>>> > > >
> >>>> > > > Then we switched from HC 4.2.3 to HC 4.5.1 and got other failures
> >>>> all
> >>>> > > > detailed in initial mail.
> >>>> > > >
> >>>> > > >
> >>>> > > > What policy does
> >>>> > > > > CookeManager use internally?
> >>>> > > > >
> >>>> > > > It depends on tests, you can see in the code that depending on
> >>>> test
> >>>> > > method
> >>>> > > > we set different Policy to test them.
> >>>> > > >
> >>>> > >
> >>>> > > You ought to be using RFC6265 policy either in strict or relaxed
> >>>> mode
> >>>> > > and nothing else. All other policies have been marked as obsolete
> >>>> and
> >>>> > > have already been removed in 5.0 (trunk).
> >>>> > >
> >>>> > > Oleg
> >>>> > >
> >>>> > >
> >>>> > >
> >>>> ---------------------------------------------------------------------
> >>>> > > To unsubscribe, e-mail: httpclient-users-unsubscr...@hc.apache.org
> >>>> > > For additional commands, e-mail:
> >>>> httpclient-users-h...@hc.apache.org
> >>>> > >
> >>>> > >
> >>>> >
> >>>> >
> >>>>
> >>>>
> >>>>
> >>>> ---------------------------------------------------------------------
> >>>> To unsubscribe, e-mail: httpclient-users-unsubscr...@hc.apache.org
> >>>> For additional commands, e-mail: httpclient-users-h...@hc.apache.org
> >>>>
> >>>>
> >>>
> >>>
> >>> --
> >>> Cordialement.
> >>> Philippe Mouawad.
> >>>
> >>>
> >>>
> >>
> >>
> >> --
> >> Cordialement.
> >> Philippe Mouawad.
> >>
> >>
> >>
> >
> >
> > --
> > Cordialement.
> > Philippe Mouawad.
> >
> >
> >
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: httpclient-users-unsubscr...@hc.apache.org
For additional commands, e-mail: httpclient-users-h...@hc.apache.org
