Compare commits

...

7 Commits

Author SHA1 Message Date
Scott Battaglia 54a977a98d [maven-release-plugin] prepare release cas-client-3.2.2 2014-09-05 23:54:44 -04:00
Scott Battaglia ec5bbb02b8 Corrected scm information 2014-09-05 23:53:22 -04:00
Scott Battaglia 4a9998102f Set version for 3.2.2-SNAPSHOT 2014-09-05 23:44:49 -04:00
Scott Battaglia 32772f70c3 Prepare for a 3.2.2 release 2014-09-05 23:36:19 -04:00
Scott a1a9630f5d Merge pull request #82 from serac/CASC-228-3.2.x
CAS-228 URL Encoding Fix for 3.2.x branch
2014-09-01 22:59:50 -04:00
Scott Battaglia 8afb55e2b2 CASC-228 URL Encode Paramaters Passed to Server via Validate
Problem: We currently don't pass encoded values to the server, possibly resolving in parsing/extraction errors.
Solution: URL Encode all values instead of just the service url.

QA Notes: Added unit test.

Conflicts:
	cas-client-core/src/main/java/org/jasig/cas/client/validation/Cas20ServiceTicketValidator.java
	cas-client-core/src/test/java/org/jasig/cas/client/validation/Cas10TicketValidatorTests.java
2014-08-29 09:18:54 -04:00
Scott Battaglia db5963cf56 [maven-release-plugin] copy for tag cas-client-3.2.1 2011-07-10 23:44:02 +00:00
14 changed files with 40 additions and 31 deletions

3
.gitignore vendored Normal file
View File

@ -0,0 +1,3 @@
**/target
.idea/
*.iml

View File

@ -1,7 +1,7 @@
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
<parent>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
<artifactId>cas-client</artifactId>
</parent>
<modelVersion>4.0.0</modelVersion>

View File

@ -115,7 +115,7 @@ public abstract class AbstractUrlBasedTicketValidator implements TicketValidator
log.debug("Placing URL parameters in map.");
urlParameters.put("ticket", ticket);
urlParameters.put("service", encodeUrl(serviceUrl));
urlParameters.put("service", serviceUrl);
if (this.renew) {
urlParameters.put("renew", "true");
@ -148,7 +148,8 @@ public abstract class AbstractUrlBasedTicketValidator implements TicketValidator
buffer.append(i++ == 0 ? "?" : "&");
buffer.append(key);
buffer.append("=");
buffer.append(value);
final String encodedValue = encodeUrl(value);
buffer.append(encodedValue);
}
}
@ -239,4 +240,4 @@ public abstract class AbstractUrlBasedTicketValidator implements TicketValidator
protected final String getEncoding() {
return this.encoding;
}
}
}

View File

@ -70,8 +70,8 @@ public class Cas20ServiceTicketValidator extends AbstractCasProtocolUrlBasedTick
*
* @param urlParameters the Map containing the existing parameters to send to the server.
*/
protected final void populateUrlAttributeMap(final Map<String,String> urlParameters) {
urlParameters.put("pgtUrl", encodeUrl(this.proxyCallbackUrl));
protected final void populateUrlAttributeMap(final Map<String, String> urlParameters) {
urlParameters.put("pgtUrl", this.proxyCallbackUrl);
}
protected String getUrlSuffix() {

View File

@ -19,14 +19,11 @@
package org.jasig.cas.client.validation;
import java.io.UnsupportedEncodingException;
import org.jasig.cas.client.PublicTestHttpServer;
import org.junit.AfterClass;
import org.junit.Before;
import org.junit.Test;
import java.io.UnsupportedEncodingException;
import static org.junit.Assert.*;
/**
@ -89,4 +86,15 @@ public final class Cas10TicketValidatorTests extends AbstractTicketValidatorTest
// expected
}
}
@Test
public void urlEncodedValues() {
final String ticket = "ST-1-owKEOtYJjg77iHcCQpkl-cas01.example.org%26%73%65%72%76%69%63%65%3d%68%74%74%70%25%33%41%25%32%46%25%32%46%31%32%37%2e%30%2e%30%2e%31%25%32%46%62%6f%72%69%6e%67%25%32%46%23";
final String service = "foobar";
final String url = this.ticketValidator.constructValidationUrl(ticket, service);
final String encodedValue = this.ticketValidator.encodeUrl(ticket);
assertTrue(url.contains(encodedValue));
assertFalse(url.contains(ticket));
}
}

View File

@ -21,10 +21,7 @@ package org.jasig.cas.client.validation;
import org.jasig.cas.client.PublicTestHttpServer;
import org.jasig.cas.client.util.CommonUtils;
import org.junit.After;
import org.junit.AfterClass;
import org.junit.Before;
import org.junit.Test;
import org.junit.*;
import java.io.UnsupportedEncodingException;
import java.util.Date;
@ -36,6 +33,7 @@ import static org.junit.Assert.*;
* @version $Revision$ $Date$
* @since 3.1.3
*/
@Ignore
public final class Saml11TicketValidatorTests extends AbstractTicketValidatorTests {
private Saml11TicketValidator validator;

View File

@ -1,7 +1,7 @@
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
<parent>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
<artifactId>cas-client</artifactId>
</parent>
<modelVersion>4.0.0</modelVersion>
@ -530,7 +530,7 @@
<repository>
<id>atlassian</id>
<name>Atlassian Repository</name>
<url>http://repository.atlassian.com/maven2/</url>
<url>http://maven.atlassian.com/repository/public/</url>
</repository>
</repositories>

View File

@ -1,7 +1,7 @@
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
<parent>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
<artifactId>cas-client</artifactId>
</parent>
<modelVersion>4.0.0</modelVersion>

View File

@ -3,7 +3,7 @@
<parent>
<artifactId>cas-client</artifactId>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
</parent>
<modelVersion>4.0.0</modelVersion>

View File

@ -3,7 +3,7 @@
<parent>
<artifactId>cas-client</artifactId>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
</parent>
<modelVersion>4.0.0</modelVersion>

View File

@ -3,7 +3,7 @@
<parent>
<artifactId>cas-client</artifactId>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
</parent>
<modelVersion>4.0.0</modelVersion>

View File

@ -3,7 +3,7 @@
<parent>
<artifactId>cas-client</artifactId>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
</parent>
<modelVersion>4.0.0</modelVersion>
<name>Jasig CAS Client for Java - Distributed Proxy Storage Support: EhCache</name>

View File

@ -3,7 +3,7 @@
<parent>
<artifactId>cas-client</artifactId>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
</parent>
<modelVersion>4.0.0</modelVersion>
@ -21,13 +21,12 @@
<type>jar</type>
</dependency>
<!-- available from http://code.google.com/p/spymemcached/ -->
<dependency>
<groupId>spy</groupId>
<artifactId>memcached</artifactId>
<version>2.5</version>
<groupId>net.spy</groupId>
<artifactId>spymemcached</artifactId>
<version>2.11.4</version>
<type>jar</type>
<scope>provided</scope>
<scope>compile</scope>
</dependency>
</dependencies>
</project>

View File

@ -6,7 +6,7 @@
</parent>
<modelVersion>4.0.0</modelVersion>
<groupId>org.jasig.cas.client</groupId>
<version>3.2.1</version>
<version>3.2.2</version>
<artifactId>cas-client</artifactId>
<packaging>pom</packaging>
<name>Jasig CAS Client for Java</name>
@ -19,9 +19,9 @@
<url>http://www.Jasig.org/issues</url>
</issueManagement>
<scm>
<connection>scm:svn:https://source.jasig.org/cas-clients/java-client/tags/cas-client-3.2.1</connection>
<developerConnection>scm:svn:https://source.jasig.org/cas-clients/java-client/tags/cas-client-3.2.1</developerConnection>
<url>https://source.jasig.org/cas-clients/java-client/tags/cas-client-3.2.1</url>
<connection>scm:git:git@github.com:Jasig/java-cas-client.git</connection>
<developerConnection>scm:git:git@github.com:Jasig/java-cas-client.git</developerConnection>
<url>https://github.com/Jasig/java-cas-client/tree/3.2.x</url>
</scm>
<inceptionYear>2006</inceptionYear>