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:
@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 <
[email protected]> 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 <
> [email protected]> 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 <
>> [email protected]> 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 <[email protected]>
>>> 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 <[email protected]>
>>>> 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 <
>>>> [email protected]>
>>>> > > 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 <
>>>> > > > > > [email protected]> 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: [email protected]
>>>> > > For additional commands, e-mail:
>>>> [email protected]
>>>> > >
>>>> > >
>>>> >
>>>> >
>>>>
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: [email protected]
>>>> For additional commands, e-mail: [email protected]
>>>>
>>>>
>>>
>>>
>>> --
>>> Cordialement.
>>> Philippe Mouawad.
>>>
>>>
>>>
>>
>>
>> --
>> Cordialement.
>> Philippe Mouawad.
>>
>>
>>
>
>
> --
> Cordialement.
> Philippe Mouawad.
>
>
>
--
Cordialement.
Philippe Mouawad.