Skip to content

Commit cd0be83

Browse files
author
pawel.koselski
committed
Fixed build warnings - removed deprecated HttpRequest calls
1 parent 54b2936 commit cd0be83

4 files changed

Lines changed: 14 additions & 14 deletions

File tree

core/src/test/java/com/onelogin/saml2/http/HttpRequestTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public class HttpRequestTest {
2424
public void testConstructorWithNoQueryParams() throws Exception {
2525
final String url = "url";
2626

27-
final HttpRequest request = new HttpRequest(url);
27+
final HttpRequest request = new HttpRequest(url, (String)null);
2828
assertThat(request.getRequestURL(), equalTo(url));
2929
assertThat(request.getParameters(), equalTo(Collections.<String, List<String>>emptyMap()));
3030
assertThat(request.getParameters("x"), equalTo(Collections.<String>emptyList()));
@@ -41,7 +41,7 @@ public void testConstructorWithQueryParams() throws Exception {
4141
final List<String> values = Arrays.asList(value1, value2);
4242
final Map<String, List<String>> parametersMap = singletonMap(name, values);
4343

44-
final HttpRequest request = new HttpRequest(url, parametersMap);
44+
final HttpRequest request = new HttpRequest(url, parametersMap, null);
4545
assertThat(request.getRequestURL(), equalTo(url));
4646
assertThat(request.getParameters(), equalTo(parametersMap));
4747
assertThat(request.getParameters(name), equalTo(values));
@@ -54,7 +54,7 @@ public void testAddParameter() throws Exception {
5454
final String name = "name";
5555
final String value = "value";
5656

57-
final HttpRequest request = new HttpRequest(url).addParameter(name, value);
57+
final HttpRequest request = new HttpRequest(url, (String)null).addParameter(name, value);
5858
assertThat(request.getRequestURL(), equalTo(url));
5959
assertThat(request.getParameters(), equalTo(singletonMap(name, singletonList(value))));
6060
assertThat(request.getParameters(name), equalTo(singletonList(value)));
@@ -70,7 +70,7 @@ public void testRemoveParameter() throws Exception {
7070
final String name = "name";
7171
final String value = "value";
7272

73-
HttpRequest request = new HttpRequest(url).addParameter(name, value);
73+
HttpRequest request = new HttpRequest(url, (String)null).addParameter(name, value);
7474
assertThat(request.getRequestURL(), equalTo(url));
7575
assertThat(request.getParameters(), equalTo(singletonMap(name, singletonList(value))));
7676
assertThat(request.getParameters(name), equalTo(singletonList(value)));
@@ -94,7 +94,7 @@ public void testGetEncodedParameter_encodesParametersNotOnQueryString() throws E
9494
final List<String> values = Arrays.asList(value1);
9595
final Map<String, List<String>> parametersMap = singletonMap(name, values);
9696

97-
final HttpRequest request = new HttpRequest(url, parametersMap).addParameter(addedName, addedValue);
97+
final HttpRequest request = new HttpRequest(url, parametersMap, null).addParameter(addedName, addedValue);
9898

9999
assertThat(request.getEncodedParameter(name), equalTo(Util.urlEncoder(value1)));
100100
assertThat(request.getEncodedParameter(addedName), equalTo(Util.urlEncoder(addedValue)));
@@ -157,7 +157,7 @@ public void testGetEncodedParameter_withDefault_usesDefaultWhenParameterMissing(
157157
final String url = "url";
158158
final String foobar = "foo/bar!";
159159

160-
final HttpRequest request = new HttpRequest(url);
160+
final HttpRequest request = new HttpRequest(url, (String)null);
161161
assertThat(request.getEncodedParameter("missing", foobar), equalTo(Util.urlEncoder(foobar)));
162162
}
163163

core/src/test/java/com/onelogin/saml2/test/authn/AuthnResponseTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2869,7 +2869,7 @@ private static HttpRequest newHttpRequest(String samlResponseEncoded) {
28692869
}
28702870

28712871
private static HttpRequest newHttpRequest(String requestURL, String samlResponseEncoded) {
2872-
return new HttpRequest(requestURL).addParameter("SAMLResponse", samlResponseEncoded);
2872+
return new HttpRequest(requestURL, (String)null).addParameter("SAMLResponse", samlResponseEncoded);
28732873
}
28742874
}
28752875

core/src/test/java/com/onelogin/saml2/test/logout/LogoutRequestTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -721,7 +721,7 @@ public void testIsInValidSign() throws Exception {
721721
String sigAlg = "http://www.w3.org/2000/09/xmldsig#rsa-sha1";
722722
String signature = "XCwCyI5cs7WhiJlB5ktSlWxSBxv+6q2xT3c8L7dLV6NQG9LHWhN7gf8qNsahSXfCzA0Ey9dp5BQ0EdRvAk2DIzKmJY6e3hvAIEp1zglHNjzkgcQmZCcrkK9Czi2Y1WkjOwR/WgUTUWsGJAVqVvlRZuS3zk3nxMrLH6f7toyvuJc=";
723723

724-
HttpRequest httpRequest = new HttpRequest(requestURL)
724+
HttpRequest httpRequest = new HttpRequest(requestURL, (String)null)
725725
.addParameter("SAMLRequest", samlRequestEncoded)
726726
.addParameter("RelayState", relayState)
727727
.addParameter("SigAlg", sigAlg)
@@ -839,6 +839,6 @@ public void testGetError() throws Exception {
839839
}
840840

841841
private static HttpRequest newHttpRequest(String requestURL, String samlRequestEncoded) {
842-
return new HttpRequest(requestURL).addParameter("SAMLRequest", samlRequestEncoded);
842+
return new HttpRequest(requestURL, (String)null).addParameter("SAMLRequest", samlRequestEncoded);
843843
}
844844
}

core/src/test/java/com/onelogin/saml2/test/logout/LogoutResponseTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public void testGetEncodedLogoutResponseSimulated() throws Exception {
4040
Saml2Settings settings = new SettingsBuilder().fromFile("config/config.min.properties").build();
4141
final String logoutResponseString = Util.getFileAsString("data/logout_responses/logout_response.xml");
4242
final String requestURL = "/";
43-
HttpRequest httpRequest = new HttpRequest(requestURL);
43+
HttpRequest httpRequest = new HttpRequest(requestURL, (String)null);
4444

4545
LogoutResponse logoutResponseBuilder = new LogoutResponse(settings, httpRequest) {
4646
@Override
@@ -136,7 +136,7 @@ public void testBuild() throws IOException, XMLEntityException, URISyntaxExcepti
136136
Saml2Settings settings = new SettingsBuilder().fromFile("config/config.min.properties").build();
137137

138138
final String requestURL = "/";
139-
HttpRequest httpRequest = new HttpRequest(requestURL);
139+
HttpRequest httpRequest = new HttpRequest(requestURL, (String)null);
140140

141141
LogoutResponse logoutResponse = new LogoutResponse(settings, httpRequest);
142142
assertFalse(logoutResponse.isValid());
@@ -256,7 +256,7 @@ public void testIsValidNoResponse() throws XMLEntityException, IOException, Erro
256256
assertFalse(logoutResponse.isValid());
257257
assertEquals("SAML Logout Response is not loaded", logoutResponse.getError());
258258

259-
httpRequest = new HttpRequest(requestURL);
259+
httpRequest = new HttpRequest(requestURL, (String)null);
260260
logoutResponse = new LogoutResponse(settings, httpRequest);
261261
assertFalse(logoutResponse.isValid());
262262
assertEquals("SAML Logout Response is not loaded", logoutResponse.getError());
@@ -499,7 +499,7 @@ public void testIsInValidSign() throws URISyntaxException, IOException, XMLEntit
499499
String sigAlg = "http://www.w3.org/2000/09/xmldsig#rsa-sha1";
500500
String signature = "vfWbbc47PkP3ejx4bjKsRX7lo9Ml1WRoE5J5owF/0mnyKHfSY6XbhO1wwjBV5vWdrUVX+xp6slHyAf4YoAsXFS0qhan6txDiZY4Oec6yE+l10iZbzvie06I4GPak4QrQ4gAyXOSzwCrRmJu4gnpeUxZ6IqKtdrKfAYRAcVfNKGA=";
501501

502-
HttpRequest httpRequest = new HttpRequest(requestURL)
502+
HttpRequest httpRequest = new HttpRequest(requestURL, (String)null)
503503
.addParameter("SAMLResponse", samlResponseEncoded)
504504
.addParameter("RelayState", relayState)
505505
.addParameter("SigAlg", sigAlg)
@@ -593,6 +593,6 @@ public void testGetError() throws URISyntaxException, IOException, XMLEntityExce
593593
}
594594

595595
private static HttpRequest newHttpRequest(String requestURL, String samlResponseEncoded) {
596-
return new HttpRequest(requestURL).addParameter("SAMLResponse", samlResponseEncoded);
596+
return new HttpRequest(requestURL, (String)null).addParameter("SAMLResponse", samlResponseEncoded);
597597
}
598598
}

0 commit comments

Comments
 (0)