jenkinsci / jenkinsci/java-client-api

Regression of pull request #73 : URL encoding method is invalid for non-standard job

Ouverte
#416 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Java
Étoiles
913
Forks
466
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Java-client-api version : 0.4.0-SNAPSHOT
Jenkins Server Version : 2.150.2

Latest version without Guava can't get xml config of a job with a space in its name:

org.apache.http.client.HttpResponseException: status code: 404, reason phrase: Not Found

at com.offbytwo.jenkins.client.validator.HttpResponseValidator.validateResponse(HttpResponseValidator.java:11)
at com.offbytwo.jenkins.client.JenkinsHttpClient.get(JenkinsHttpClient.java:170)
at com.offbytwo.jenkins.JenkinsServer.getJobXml(JenkinsServer.java:524)

Just looked at the code and since 0.4.0-SNAPSHOT with guava removal, a regression of pull request #73 occurs : job names with a space in them now contains the symbol "+" again, but there is an explicit comment in the latest code asking for the opposite
`// jenkins doesn't like the + for space, use %20 instead`

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez dans JenkinsHttpClient.java, au niveau de la construction de l’URL de la requête, puis suivez JenkinsServer.getJobXml. Comparez l’encodage actuel avec le comportement décrit dans pull request #73. Vérifiez que les noms de jobs contenant des espaces utilisent %20 plutôt que + et que la récupération du XML réussit pour un nom de job non standard.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
api
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.