ExtendedIssueService.java

- in removeLabel(), URL encode the label before appending to the URL when calling DELETE
- reported in jenkinsci#26
- thanks to @longwave for diagnosing the issue!

Author: aaronjwhiteside

src/main/java/org/jenkinsci/plugins/pipeline/github/client/ExtendedIssueService.java

@@ -9,6 +9,7 @@
 import org.eclipse.egit.github.core.service.IssueService;
 
 import java.io.IOException;
+import java.net.URLEncoder;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -143,14 +144,18 @@ public List<Label> addLabels(final IRepositoryIdProvider repository,
     public void removeLabel(final IRepositoryIdProvider repository,
                             final int issueNumber,
                             final String label) throws IOException {
-        String repoId = this.getId(repository);
-        StringBuilder uri = new StringBuilder("/repos");
+        final String repoId = this.getId(repository);
+        final StringBuilder uri = new StringBuilder("/repos");
         uri.append('/').append(repoId);
         uri.append("/issues");
         uri.append('/').append(issueNumber);
         uri.append("/labels");
         uri.append('/');
-        uri.append(label);
+
+        final String encodedLabel = URLEncoder.encode(label, "UTF-8")
+                .replace("+", "%20")
+                .replace(".", "%2E");
+        uri.append(encodedLabel);
         getClient().delete(uri.toString());
     }