SpringCloud/spring-cloud-openfeign
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #979 from marcosbarbero/master

Browse Source 
* pull979:
  Use request.characterEncoding prior to default encoding
This commit is contained in:
Spencer Gibb
2016-06-03 13:21:06 -06:00
parent a5eb3b6012 c743ba893c
commit 1c532d9637
2 changed files with 38 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
spring-cloud-netflix-core/src/main/java/org/springframework/cloud/netflix/zuul/filters/ProxyRequestHelper.java
View File

@@ -52,6 +52,7 @@ import lombok.extern.apachecommons.CommonsLog;
/**
* @author Dave Syer
* @author Marcos Barbero
*/
@CommonsLog
public class ProxyRequestHelper {
@@ -98,8 +99,7 @@ public class ProxyRequestHelper {
String contextURI = (String) context.get("requestURI");
if (contextURI != null) {
try {
uri = UriUtils.encodePath(contextURI,
WebUtils.DEFAULT_CHARACTER_ENCODING);
uri = UriUtils.encodePath(contextURI, characterEncoding(request));
}
catch (Exception e) {
log.debug(
@@ -110,6 +110,11 @@ public class ProxyRequestHelper {
return uri;
}
private String characterEncoding(HttpServletRequest request) {
return request.getCharacterEncoding() != null ? request.getCharacterEncoding()
: WebUtils.DEFAULT_CHARACTER_ENCODING;
}
public MultiValueMap<String, String> buildZuulRequestQueryParams(
HttpServletRequest request) {
Map<String, List<String>> map = HTTPRequestUtils.getInstance().getQueryParams();
@@ -267,7 +272,8 @@ public class ProxyRequestHelper {
/* for tests */ boolean shouldDebugBody(RequestContext ctx) {
HttpServletRequest request = ctx.getRequest();
if (!this.traceRequestBody || ctx.isChunkedRequestBody() || RequestUtils.isZuulServletRequest()) {
if (!this.traceRequestBody || ctx.isChunkedRequestBody()
|| RequestUtils.isZuulServletRequest()) {
return false;
}
if (request == null || request.getContentType() == null) {

29
spring-cloud-netflix-core/src/test/java/org/springframework/cloud/netflix/zuul/filters/ProxyRequestHelperTests.java
View File

@@ -259,4 +259,33 @@ public class ProxyRequestHelperTests {
assertThat(queryString, is("?wsdl"));
}
@Test
public void buildZuulRequestURIWithUTF8() throws Exception {
String encodedURI = "/resource/esp%C3%A9cial-char";
String decodedURI = "/resource/espécial-char";
MockHttpServletRequest request = new MockHttpServletRequest("GET", encodedURI);
request.setCharacterEncoding("UTF-8");
final RequestContext context = RequestContext.getCurrentContext();
context.setRequest(request);
context.set("requestURI", decodedURI);
final String requestURI = new ProxyRequestHelper().buildZuulRequestURI(request);
assertThat(requestURI, equalTo(encodedURI));
}
@Test
public void buildZuulRequestURIWithDefaultEncoding() {
String encodedURI = "/resource/esp%E9cial-char";
String decodedURI = "/resource/espécial-char";
MockHttpServletRequest request = new MockHttpServletRequest("GET", encodedURI);
final RequestContext context = RequestContext.getCurrentContext();
context.setRequest(request);
context.set("requestURI", decodedURI);
final String requestURI = new ProxyRequestHelper().buildZuulRequestURI(request);
assertThat(requestURI, equalTo(encodedURI));
}
}