jenkinsci / jenkinsci/java-client-api

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

オープン
#416 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
913
フォーク
466
PR マージ指標
30日以内にマージされた PR はありません

説明

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`

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

JenkinsHttpClient.java のリクエスト URL 構築箇所から始め、次に JenkinsServer.getJobXml を追ってください。現在のエンコーディングを pull request #73 に記載された動作と比較してください。スペースを含むジョブ名が + ではなく %20 を使用すること、および標準ではないジョブ名で XML の取得が成功することを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
api
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
48/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。