UriUtils ignores allowReservedCharacters=false when the input has no percent triplet

Abierto Apto para principiantes
  • ¿Algo no está bien?
#3,625 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@hutiefang76 ya está trabajando en esto.

Desde el 19/10/2026.

  • #3627 de @hutiefang76 — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
java
Área
backend, testing

Línea de trabajo

Lee UriUtils y su conjunto de pruebas existente UriUtilsTest; el informe señala la rama sin tripletes de encodeInternal y el indicador codificado de forma fija que se pasa a encodeChunk. Ejecuta las pruebas indicadas en el informe y añade pruebas de regresión para las sobrecargas booleanas, incluida una entrada no ASCII. El trabajo estará terminado cuando false codifique los caracteres reservados sin procesar, se conserven los tripletes %XX existentes y las sobrecargas con true y sin indicador mantengan su comportamiento; el issue también menciona el changelog.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

UriUtils.encode("a&b=c", false) returns a&b=c, whereas the corresponding call with an existing triplet, UriUtils.encode("a&b=c%20", false), encodes the raw reserved characters. The no-triplet branch in encodeInternal passes a hard-coded true to encodeChunk instead of the argument.

Expected: a%26b%3Dc when the flag is false; existing %XX triplets should still be preserved. The true-flag and flagless overloads keep their current behavior. This concerns the existing boolean contract, separate from #2002's discussion of double encoding.

Reproduced on current master with UriUtilsTest: 32 tests, four failures, no errors or skips. The failing cases include both boolean overloads and non-ASCII input. Main sources compile with JDK 8 and tests run with actual JDK 25 using the project's dev profile.

I will pass the flag through in that branch and add these regressions. The scope is UriUtils, its existing test suite, and the changelog; no URI-template expansion or query collection changes.

Reproduction and report prepared and posted by an AI assistant for @hutiefang76.

Lenguaje dominante
Java
Estrellas
9.8k
Forks
2k
Merge medio
1 h 23 min
PR fusionados (30 d)
31

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de OpenFeign/feign

Todos los issues de OpenFeign/feign

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.