Почему RestTemplate не кодирует символ «+», а кодирует все остальное? - PullRequest
Новая
       5

Почему RestTemplate не кодирует символ «+», а кодирует все остальное?

0 голосов
/ 03 июня 2019

Я обнаружил странную проблему, при которой urlencoding работает некорректно.

ОБНОВЛЕНИЕ

Существуют различия между версиями Spring MVC 4.3 и 5.1: 

// FAIL in MVC 4.x 
@Test
public void test2() {
    rt.getForObject("http://localhost/expr={expr}", String.class, "x/y");
    Assert.assertEquals("http://localhost/expr=x%2Fy", savedUri.toString());
}

// FAIL in MVC 4 or 5
@Test
public void test3() {
    rt.getForObject("http://localhost/expr={expr}", String.class, "x+y");
    Assert.assertEquals("http://localhost/expr=x%2By", savedUri.toString());
}

// ok in MVC 4.x, FAIL in MVC 5
@Test
public void test4() {
    rt.getForObject("http://localhost/expr={expr}", String.class, "x+y");
    Assert.assertEquals("http://localhost/expr=x+y", savedUri.toString());
}

Возможно, это было частью более обширной ссылки * Spring MVC, поскольку это также проявляется в совершенно ином месте здесь

Подробности вопроса

Мой вопрос лучше всего иллюстрируется следующим автономным тестом.Не пугайтесь ClientHttpRequestFactory - важной частью являются последние 2 метода испытаний. 

package com.stackoverflow.questions;

import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import java.io.StringReader;
import java.io.StringWriter;
import java.net.URI;

import org.apache.commons.io.input.ReaderInputStream;
import org.apache.commons.io.output.WriterOutputStream;
import org.junit.Assert;
import org.junit.Test;
import org.springframework.http.HttpHeaders;
import org.springframework.http.HttpMethod;
import org.springframework.http.HttpStatus;
import org.springframework.http.client.ClientHttpRequest;
import org.springframework.http.client.ClientHttpRequestFactory;
import org.springframework.http.client.ClientHttpResponse;
import org.springframework.web.client.RestTemplate;

public class RestTemplateTest {

    StringWriter stringWriter = new StringWriter();

    WriterOutputStream writerOutputStream = new WriterOutputStream(stringWriter);

    HttpHeaders headers = new HttpHeaders();

    URI savedUri;

    ClientHttpRequestFactory rf = new ClientHttpRequestFactory() {
        @Override
        public ClientHttpRequest createRequest(URI uri, HttpMethod httpMethod) throws IOException {
            savedUri = uri;
            return new ClientHttpRequest() {
                @Override
                public OutputStream getBody() throws IOException {
                    return writerOutputStream;
                }

                @Override
                public HttpHeaders getHeaders() {
                    return headers;
                }

                @Override
                public URI getURI() {
                    return uri;
                }

                @Override
                public String getMethodValue() {
                    return httpMethod.name();
                }

                @Override
                public ClientHttpResponse execute() throws IOException {

                    writerOutputStream.close();

                    return new ClientHttpResponse() {
                        @Override
                        public HttpHeaders getHeaders() {
                            return new HttpHeaders();
                        }

                        @Override
                        public InputStream getBody() throws IOException {
                            return new ReaderInputStream(new StringReader("test"));
                        }

                        @Override
                        public String getStatusText() throws IOException {
                            return "OK";
                        }

                        @Override
                        public HttpStatus getStatusCode() throws IOException {
                            return HttpStatus.OK;
                        }

                        @Override
                        public int getRawStatusCode() throws IOException {
                            return 200;
                        }

                        @Override
                        public void close() {
                        }
                    };
                }
            };
        }
    };
    RestTemplate rt = new RestTemplate(rf);

    @Test
    public void test1() {
        String resp = rt.getForObject("http://whatever", String.class);
        Assert.assertEquals("test", resp);
    }

    @Test
    public void test2() {
        rt.getForObject("http://localhost/expr={expr}", String.class, "x/y");
        Assert.assertEquals("http://localhost/expr=x%2Fy", savedUri.toString());
    }

    @Test
    public void test3() {
        rt.getForObject("http://localhost/expr={expr}", String.class, "x+y");
        Assert.assertEquals("http://localhost/expr=x%2By", savedUri.toString());
    }

}

Что происходит:

  • символ деления / правильно закодирован в %2F - test2 () проходит

  • добавочный символ + НЕ кодируется в %2B - он остается +и test3 () завершается ошибкой

Что должно произойти:

test3 () должен пройти.+ должен быть закодирован в% 2B, потому что + является специальным символом в URL и интерпретируется как пробел многими серверными веб-фреймворками.

Что здесь происходит иЕсть ли общее исправление?

1 Ответ

1 голос
/ 03 июня 2019

Существует неопределенность, следует ли кодировать + или нет.Старые RFC говорят «да», более новые говорят «возможно».

Попробуйте установить режим кодирования следующим образом: 

  DefaultUriBuilderFactory builderFactory = new DefaultUriBuilderFactory();
  builderFactory.setEncodingMode(EncodingMode.VALUES_ONLY);
  restTemplate.setUriTemplateHandler(builderFactory);

См. SPR-19394 и SPR-20750 для обсуждения.

...