Changeset 9935 in josm for trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java

Timestamp:

2016-03-05T21:36:33+01:00 (9 years ago)

Author:

simon04

Message:

see #12584 - Attempt to obtain the session cookie via CookieManager

File:

• trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java

@@ -7 +7 @@
 import java.io.IOException;
 import java.lang.reflect.Field;
+import java.net.CookieHandler;
 import java.net.HttpURLConnection;
+import java.net.URISyntaxException;
 import java.net.URL;
 import java.nio.charset.StandardCharsets;
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.Iterator;
@@ -199 +202 @@
     }
 
-    protected SessionId extractOsmSession() {
-        List<String> setCookies = connection.getResponse().getHeaderFields().get("Set-Cookie");
+    protected SessionId extractOsmSession() throws IOException, URISyntaxException {
+        // response headers might not contain the cookie, see #12584
+        final List<String> setCookies = CookieHandler.getDefault()
+                .get(connection.getURL().toURI(), Collections.<String, List<String>>emptyMap())
+                .get("Cookie");
         if (setCookies == null) {
             Main.warn("No 'Set-Cookie' in response header!");
@@ -267 +273 @@
                         tr("OSM website did not return a session cookie in response to ''{0}'',", url.toString()));
             return sessionId;
-        } catch (IOException e) {
+        } catch (IOException | URISyntaxException e) {
             throw new OsmOAuthAuthorizationException(e);
         } finally {