diff --git a/tests/reproducers/signed/SavingCookies/resources/CheckCookie.html b/tests/reproducers/signed/SavingCookies/resources/CheckCookie.html new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/resources/CheckCookie.html @@ -0,0 +1,44 @@ + + +

+ + +

+ + diff --git a/tests/reproducers/signed/SavingCookies/resources/CheckCookieAndGotoClear.html b/tests/reproducers/signed/SavingCookies/resources/CheckCookieAndGotoClear.html new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/resources/CheckCookieAndGotoClear.html @@ -0,0 +1,50 @@ + + + + + +

+ + + + + +

+ + diff --git a/tests/reproducers/signed/SavingCookies/resources/ClearPersistentCookie.html b/tests/reproducers/signed/SavingCookies/resources/ClearPersistentCookie.html new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/resources/ClearPersistentCookie.html @@ -0,0 +1,49 @@ + + +

+ + + + + + + +

+ + diff --git a/tests/reproducers/signed/SavingCookies/resources/SavePersistentCookie.html b/tests/reproducers/signed/SavingCookies/resources/SavePersistentCookie.html new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/resources/SavePersistentCookie.html @@ -0,0 +1,49 @@ + + +

+ + + + + + + +

+ + diff --git a/tests/reproducers/signed/SavingCookies/resources/SavePersistentCookieAndGotoCheck.html b/tests/reproducers/signed/SavingCookies/resources/SavePersistentCookieAndGotoCheck.html new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/resources/SavePersistentCookieAndGotoCheck.html @@ -0,0 +1,50 @@ + + +

+ + + + + + + + +

+ + diff --git a/tests/reproducers/signed/SavingCookies/resources/SaveSessionCookie.html b/tests/reproducers/signed/SavingCookies/resources/SaveSessionCookie.html new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/resources/SaveSessionCookie.html @@ -0,0 +1,49 @@ + + +

+ + + + + + + +

+ + diff --git a/tests/reproducers/signed/SavingCookies/resources/SaveSessionCookieAndGotoCheck.html b/tests/reproducers/signed/SavingCookies/resources/SaveSessionCookieAndGotoCheck.html new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/resources/SaveSessionCookieAndGotoCheck.html @@ -0,0 +1,50 @@ + + +

+ + + + + + + + +

+ + diff --git a/tests/reproducers/signed/SavingCookies/srcs/CheckingCookies.java b/tests/reproducers/signed/SavingCookies/srcs/CheckingCookies.java new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/srcs/CheckingCookies.java @@ -0,0 +1,114 @@ +/* CheckingCookies.java + Confirms that a test cookie is in the cookie store + +Copyright (C) 2012 Red Hat, Inc. + +This file is part of IcedTea. + +IcedTea is free software; you can redistribute it and/or +modify it under the terms of the GNU General Public License as published by +the Free Software Foundation, version 2. + +IcedTea is distributed in the hope that it will be useful, +but WITHOUT ANY WARRANTY; without even the implied warranty of +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +General Public License for more details. + +You should have received a copy of the GNU General Public License +along with IcedTea; see the file COPYING. If not, write to +the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA +02110-1301 USA. + +Linking this library statically or dynamically with other modules is +making a combined work based on this library. Thus, the terms and +conditions of the GNU General Public License cover the whole +combination. + +As a special exception, the copyright holders of this library give you +permission to link this library with independent modules to produce an +executable, regardless of the license terms of these independent +modules, and to copy and distribute the resulting executable under +terms of your choice, provided that you also meet, for each linked +independent module, the terms and conditions of the license of that +module. An independent module is a module which is not derived from +or based on this library. If you modify this library, you may extend +this exception to your version of the library, but you are not +obligated to do so. If you do not wish to do so, delete this +exception statement from your version. + */ +import java.applet.Applet; +import java.io.IOException; +import java.net.CookieHandler; +import java.net.CookieManager; +import java.net.CookiePolicy; +import java.net.URI; +import java.net.URISyntaxException; +import java.net.URL; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + +public class CheckingCookies extends Applet { + static class Killer extends Thread { + + public int n = 2000; + + @Override + public void run() { + try { + Thread.sleep(n); + System.out.println("Applet killing itself after " + n + " ms of life"); + System.exit(0); + } catch (Exception ex) { + } + } + } + + static private void printCookieInfo(URI uri) throws IOException { + CookieHandler handler = CookieHandler.getDefault(); + Map> cookieMap = null; + + if (handler == null) { + System.out.println("Failing due to lack of CookieHandler class!"); + return; + } + System.out.println("Using CookieHandler class: " + handler.getClass().getCanonicalName()); + + cookieMap = handler.get(uri, new HashMap>()); + for (Map.Entry> entry : cookieMap.entrySet()) { + System.out.println("Iterating cookiemap with " + entry.getKey() + " => " + entry.getValue()); + if (entry.getKey().contains("Cookie")) { + for (String cookie : entry.getValue()) { + System.out.println("Found cookie: " + cookie); + } + } + } + } + + /* If a show-document param was set, go there */ + private void gotoNextDocument() { + URL baseURL = getCodeBase(); + String nextDocument = getParameter("show-document"); + if (nextDocument != null) { + try { + System.out.println("Calling showDocument(" + nextDocument + ")"); + getAppletContext().showDocument(new URL(baseURL.toString() + nextDocument)); + } catch (Exception e) { + e.printStackTrace(); + } + } + } + + @Override + public void start() { + System.out.println("Entered CheckingCookies.java"); + try { + printCookieInfo(getCodeBase().toURI()); + } catch (Exception e) { + e.printStackTrace(); + } + System.out.println("Finished CheckingCookies.java"); + + gotoNextDocument(); + } +} diff --git a/tests/reproducers/signed/SavingCookies/srcs/SavingCookies.java b/tests/reproducers/signed/SavingCookies/srcs/SavingCookies.java new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/srcs/SavingCookies.java @@ -0,0 +1,123 @@ +/* SavingCookies.java + Store cookies in the java cookie store, and go to a page that confirms they are there. + +Copyright (C) 2012 Red Hat, Inc. + +This file is part of IcedTea. + +IcedTea is free software; you can redistribute it and/or +modify it under the terms of the GNU General Public License as published by +the Free Software Foundation, version 2. + +IcedTea is distributed in the hope that it will be useful, +but WITHOUT ANY WARRANTY; without even the implied warranty of +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +General Public License for more details. + +You should have received a copy of the GNU General Public License +along with IcedTea; see the file COPYING. If not, write to +the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA +02110-1301 USA. + +Linking this library statically or dynamically with other modules is +making a combined work based on this library. Thus, the terms and +conditions of the GNU General Public License cover the whole +combination. + +As a special exception, the copyright holders of this library give you +permission to link this library with independent modules to produce an +executable, regardless of the license terms of these independent +modules, and to copy and distribute the resulting executable under +terms of your choice, provided that you also meet, for each linked +independent module, the terms and conditions of the license of that +module. An independent module is a module which is not derived from +or based on this library. If you modify this library, you may extend +this exception to your version of the library, but you are not +obligated to do so. If you do not wish to do so, delete this +exception statement from your version. + */ +import java.applet.Applet; +import java.io.IOException; +import java.net.CookieHandler; +import java.net.CookieManager; +import java.net.HttpCookie; +import java.net.URI; +import java.net.URL; +import java.text.DateFormat; +import java.text.SimpleDateFormat; +import java.util.ArrayList; +import java.util.Calendar; +import java.util.HashMap; +import java.util.List; +import java.util.Locale; +import java.util.Map; + +public class SavingCookies extends Applet { + // Set up a response header with Set-Cookie + static private Map> responseHeaders(String cookie) { + List cookies = new ArrayList(); + cookies.add(cookie); + Map> rh = new HashMap>(); + rh.put("Set-Cookie", cookies); + rh.put("Cookie", cookies); + return rh; + } + + static private void saveCookie(URI uri, String cookie) throws IOException { + CookieHandler handler = CookieHandler.getDefault(); + + if (handler == null) { + System.out.println("Failing due to lack of CookieHandler class!"); + return; + } + + System.out.println("Using CookieHandler class: " + handler.getClass().getCanonicalName()); + System.out.println("Putting " + cookie + " at " + uri.toString()); + handler.put(uri, responseHeaders(cookie)); + } + + /* If a show-document param was set, go there */ + private void gotoNextDocument() { + URL baseURL = getCodeBase(); + String nextDocument = getParameter("show-document"); + if (nextDocument != null) { + try { + System.out.println("Calling showDocument(" + nextDocument + ")"); + getAppletContext().showDocument(new URL(baseURL.toString() + nextDocument)); + } catch (Exception e) { + e.printStackTrace(); + } + } + } + + static private String formatCookie(String cookie, boolean persistent) { + if (persistent) { + // Put in the cookie date format + final int TIMEOUT = 3600; + DateFormat df = new SimpleDateFormat("EEE, dd-MMM-yyyy HH:mm:ss 'GMT'", Locale.US); + Calendar cal = Calendar.getInstance(); + cal.add(Calendar.SECOND, TIMEOUT); + cookie += "; "; + cookie += "Expires=" + df.format(cal.getTime()) + "; "; + cookie += "Max-Age=" + TIMEOUT + "; "; + } + return cookie; + } + + @Override + public void start() { + System.out.println("Entered SavingCookies.java"); + + URL baseURL = getCodeBase(); + try { + saveCookie(baseURL.toURI(), formatCookie(getParameter("cookie"), + "yes".equals(getParameter("persistent")))); + } catch (Exception e) { + e.printStackTrace(); + } + + System.out.println("Finished SavingCookies.java"); + + gotoNextDocument(); + } +} diff --git a/tests/reproducers/signed/SavingCookies/testcases/SavingCookiesTests.java b/tests/reproducers/signed/SavingCookies/testcases/SavingCookiesTests.java new file mode 100644 --- /dev/null +++ b/tests/reproducers/signed/SavingCookies/testcases/SavingCookiesTests.java @@ -0,0 +1,174 @@ +/* SavingCookieTests.java +Copyright (C) 2011 Red Hat, Inc. + +This file is part of IcedTea. + +IcedTea is free software; you can redistribute it and/or +modify it under the terms of the GNU General Public License as published by +the Free Software Foundation, version 2. + +IcedTea is distributed in the hope that it will be useful, +but WITHOUT ANY WARRANTY; without even the implied warranty of +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +General Public License for more details. + +You should have received a copy of the GNU General Public License +along with IcedTea; see the file COPYING. If not, write to +the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA +02110-1301 USA. + +Linking this library statically or dynamically with other modules is +making a combined work based on this library. Thus, the terms and +conditions of the GNU General Public License cover the whole +combination. + +As a special exception, the copyright holders of this library give you +permission to link this library with independent modules to produce an +executable, regardless of the license terms of these independent +modules, and to copy and distribute the resulting executable under +terms of your choice, provided that you also meet, for each linked +independent module, the terms and conditions of the license of that +module. An independent module is a module which is not derived from +or based on this library. If you modify this library, you may extend +this exception to your version of the library, but you are not +obligated to do so. If you do not wish to do so, delete this +exception statement from your version. + */ + +import java.util.Arrays; +import java.util.Collections; +import java.util.List; + +import net.sourceforge.jnlp.ContentReaderListener; +import net.sourceforge.jnlp.ProcessResult; +import net.sourceforge.jnlp.annotations.Bug; +import net.sourceforge.jnlp.annotations.TestInBrowsers; +import net.sourceforge.jnlp.browsertesting.BrowserTest; +import net.sourceforge.jnlp.browsertesting.Browsers; +import org.junit.Assert; + +import org.junit.Test; + +public class SavingCookiesTests extends BrowserTest { + + static final String ENTERING_CHECK = "Entered CheckingCookies"; + static final String CHECKING_COMPLETION = "Finished CheckingCookies"; + static final String SAVING_COMPLETION = "Finished SavingCookies"; + private final static List TRUSTALL = Collections.unmodifiableList(Arrays.asList(new String[] { "-Xtrustall" })); + + static class ParallelRun extends Thread { + + ParallelRun(String url, String completionString) { + this.url = url; + this.completionString = completionString; + this.completed = false; + } + + ProcessResult pr; + private String url; + private String completionString; + volatile boolean completed; + + @Override + public void run() { + try { + final ContentReaderListener stdoutListener = new ContentReaderListener() { + @Override + public void charReaded(char ch) { + } + + @Override + public synchronized void lineReaded(String s) { + if (completionString != null && s.contains(completionString)) { + completed = true; + } + } + }; + if (url.endsWith(".html")) { + pr = server.executeBrowser(url, stdoutListener, stdoutListener); + } else if (url.endsWith(".jnlp")) { + pr = server.executeJavawsHeadless(TRUSTALL, url, stdoutListener, stdoutListener); + } + } catch (Exception ex) { + throw new RuntimeException(ex); + } finally { + completed = true; + } + } + } + + final String COOKIE_SESSION_CHECK = "Found cookie: TEST=session"; + final String COOKIE_PERSISTENT_CHECK = "Found cookie: TEST=persistent"; + + @Test + @TestInBrowsers(testIn = { Browsers.one }) + public void AppletCheckCookieIsntSet() throws Exception { + final String COOKIE_SANITY_CHECK = "Found cookie: TEST="; + ProcessResult pr = server.executeBrowser("/CheckCookie.html"); + + Assert.assertFalse("stdout should NOT contain '" + COOKIE_SANITY_CHECK + "' but did.", pr.stdout.contains(COOKIE_SANITY_CHECK)); + Assert.assertTrue("stdout should contain '" + CHECKING_COMPLETION + "' but did not.", pr.stdout.contains(CHECKING_COMPLETION)); + } + + @Test + @TestInBrowsers(testIn = { Browsers.one }) + @Bug(id = "PR588") + public void AppletSessionCookieShowDoc() throws Exception { + ProcessResult pr = server.executeBrowser("/SaveSessionCookieAndGotoCheck.html"); + + Assert.assertTrue("stdout should contain '" + ENTERING_CHECK + "' but did not.", pr.stdout.contains(ENTERING_CHECK)); + Assert.assertTrue("stdout should contain '" + COOKIE_SESSION_CHECK + "' but did not.", pr.stdout.contains(COOKIE_SESSION_CHECK)); + } + + @Test + @TestInBrowsers(testIn = { Browsers.one }) + @Bug(id = "PR588") + public void AppletSessionCookieParallel() throws Exception { + ParallelRun save = new ParallelRun("/SaveSessionCookie.html", SAVING_COMPLETION); + save.start(); + while (!save.completed) { + Thread.sleep(100); + } + + ProcessResult check = server.executeBrowser("/CheckCookie.html"); + save.join(); + + Assert.assertTrue("stdout should contain '" + ENTERING_CHECK + "' but did not.", save.pr.stdout.contains(ENTERING_CHECK)); + //XXX: It is necessary to check save.pr's stdout, because it does not show up in 'check.stdout' for some reason + Assert.assertTrue("stdout should contain '" + COOKIE_SESSION_CHECK + "' but did not.", save.pr.stdout.contains(COOKIE_SESSION_CHECK)); + } + + @Test + @TestInBrowsers(testIn = { Browsers.one }) + @Bug(id = "PR588") + public void AppletSessionCookieSequential() throws Exception { + ProcessResult save = server.executeBrowser("/SaveSessionCookie.html"); + ProcessResult check = server.executeBrowser("/CheckCookie.html"); + Assert.assertTrue("stdout should contain '" + ENTERING_CHECK + "' but did not.", check.stdout.contains(ENTERING_CHECK)); + //Session cookies should NOT be intact upon browser close and re-open + Assert.assertFalse("stdout should NOT contain '" + COOKIE_SESSION_CHECK + "' but did.", check.stdout.contains(COOKIE_SESSION_CHECK)); + } + + @Test + @TestInBrowsers(testIn = { Browsers.one }) + @Bug(id = "PR588") + public void AppletPersistentCookieShowDoc() throws Exception { + ProcessResult pr = server.executeBrowser("/SavePersistentCookieAndGotoCheck.html"); + + Assert.assertTrue("stdout should contain '" + ENTERING_CHECK + "' but did not.", pr.stdout.contains(ENTERING_CHECK)); + Assert.assertTrue("stdout should contain '" + COOKIE_PERSISTENT_CHECK + "' but did not.", pr.stdout.contains(COOKIE_PERSISTENT_CHECK)); + } + + @Test + @TestInBrowsers(testIn = { Browsers.one }) + @Bug(id = "PR588") + public void AppletPersistentCookieSequential() throws Exception { + ProcessResult save = server.executeBrowser("/SavePersistentCookie.html"); + //Use show doc to clear cookie afterwards + ProcessResult check = server.executeBrowser("/CheckCookieAndGotoClear.html"); + Assert.assertTrue("stdout should contain '" + ENTERING_CHECK + "' but did not.", check.stdout.contains(ENTERING_CHECK)); + //Persistent cookies should be stored past this point + Assert.assertTrue("stdout should contain '" + COOKIE_PERSISTENT_CHECK + "' but did not.", check.stdout.contains(COOKIE_PERSISTENT_CHECK)); + } + +} \ No newline at end of file