author | aefimov |
Mon, 27 May 2019 13:29:11 +0100 | |
changeset 55045 | 3a8433d967ea |
parent 47216 | 71c04702a3d5 |
permissions | -rw-r--r-- |
11528
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
1 |
/* |
55045
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
2 |
* Copyright (c) 2012, 2019, Oracle and/or its affiliates. All rights reserved. |
11528
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
4 |
* |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
8 |
* |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
13 |
* accompanied this code). |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
14 |
* |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
18 |
* |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
21 |
* questions. |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
22 |
*/ |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
23 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
24 |
/* |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
25 |
* @test |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
26 |
* @bug 7129083 |
55045
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
27 |
* @library /test/lib |
11528
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
28 |
* @summary Cookiemanager does not store cookies if url is read |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
29 |
* before setting cookiemanager |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
30 |
*/ |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
31 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
32 |
import java.net.CookieHandler; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
33 |
import java.net.CookieManager; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
34 |
import java.net.CookiePolicy; |
55045
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
35 |
import java.net.InetAddress; |
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
36 |
import java.net.InetSocketAddress; |
11528
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
37 |
import java.net.ServerSocket; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
38 |
import java.net.Socket; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
39 |
import java.net.URL; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
40 |
import java.io.InputStream; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
41 |
import java.io.IOException; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
42 |
|
55045
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
43 |
import jdk.test.lib.net.URIBuilder; |
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
44 |
|
11528
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
45 |
public class CookieHttpClientTest implements Runnable { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
46 |
final ServerSocket ss; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
47 |
static final int TIMEOUT = 10 * 1000; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
48 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
49 |
static final String replyString = "HTTP/1.1 200 OK\r\n" + |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
50 |
"Set-Cookie: name=test\r\n" + |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
51 |
"Content-Length: 10\r\n\r\n" + |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
52 |
"1234567890"; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
53 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
54 |
// HTTP server, reply with Set-Cookie |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
55 |
@Override |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
56 |
public void run() { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
57 |
Socket s = null; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
58 |
try { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
59 |
s = ss.accept(); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
60 |
s.setSoTimeout(TIMEOUT); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
61 |
readOneRequest(s.getInputStream()); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
62 |
s.getOutputStream().write(replyString.getBytes()); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
63 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
64 |
readOneRequest(s.getInputStream()); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
65 |
s.getOutputStream().write(replyString.getBytes()); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
66 |
} catch (Exception e) { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
67 |
e.printStackTrace(); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
68 |
} finally { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
69 |
try { if (s != null) { s.close(); } ss.close(); } |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
70 |
catch (IOException unused) { /* gulp!burp! */ } |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
71 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
72 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
73 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
74 |
static final byte[] requestEnd = new byte[] {'\r', '\n', '\r', '\n' }; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
75 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
76 |
// Read until the end of a HTTP request |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
77 |
static void readOneRequest(InputStream is) throws IOException { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
78 |
int requestEndCount = 0, r; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
79 |
while ((r = is.read()) != -1) { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
80 |
if (r == requestEnd[requestEndCount]) { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
81 |
requestEndCount++; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
82 |
if (requestEndCount == 4) { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
83 |
break; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
84 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
85 |
} else { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
86 |
requestEndCount = 0; |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
87 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
88 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
89 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
90 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
91 |
CookieHttpClientTest() throws Exception { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
92 |
/* start the server */ |
55045
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
93 |
ss = new ServerSocket(); |
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
94 |
ss.bind(new InetSocketAddress(InetAddress.getLoopbackAddress(), 0)); |
11528
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
95 |
(new Thread(this)).start(); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
96 |
|
55045
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
97 |
URL url = URIBuilder.newBuilder() |
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
98 |
.scheme("http") |
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
99 |
.loopback() |
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
100 |
.port(ss.getLocalPort()) |
3a8433d967ea
8224035: Replace wildcard address with loopback or local host in tests - part 9
aefimov
parents:
47216
diff
changeset
|
101 |
.path("/").toURL(); |
11528
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
102 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
103 |
// Run without a CookieHandler first |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
104 |
InputStream in = url.openConnection().getInputStream(); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
105 |
while (in.read() != -1); // read response body so connection can be reused |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
106 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
107 |
// Set a CookeHandler and retest using the HttpClient from the KAC |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
108 |
CookieManager manager = new CookieManager(null, CookiePolicy.ACCEPT_ALL); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
109 |
CookieHandler.setDefault(manager); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
110 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
111 |
in = url.openConnection().getInputStream(); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
112 |
while (in.read() != -1); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
113 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
114 |
if (manager.getCookieStore().getCookies().isEmpty()) { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
115 |
throw new RuntimeException("Failed: No cookies in the cookie Handler."); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
116 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
117 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
118 |
|
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
119 |
public static void main(String args[]) throws Exception { |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
120 |
new CookieHttpClientTest(); |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
121 |
} |
638ca25aec87
7129083: CookieManager does not store cookies if url is read before setting cookie manager
chegar
parents:
diff
changeset
|
122 |
} |