Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-20140] Merge pull request #35
  • Loading branch information
kohsuke committed Jun 8, 2014
2 parents 6d023ef + 5c2a041 commit e065b80
Show file tree
Hide file tree
Showing 3 changed files with 35 additions and 2 deletions.
2 changes: 1 addition & 1 deletion pom.xml
Expand Up @@ -3,7 +3,7 @@
<parent>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>plugin</artifactId>
<version>1.445</version>
<version>1.448</version>
</parent>

<groupId>com.coravy.hudson.plugins.github</groupId>
Expand Down
3 changes: 2 additions & 1 deletion src/main/java/com/cloudbees/jenkins/GitHubWebHook.java
Expand Up @@ -37,6 +37,7 @@
@Extension
public class GitHubWebHook implements UnprotectedRootAction {
private static final Pattern REPOSITORY_NAME_PATTERN = Pattern.compile("https?://([^/]+)/([^/]+)/([^/]+)");
public static final String URLNAME = "github-webhook";

public String getIconFileName() {
return null;
Expand All @@ -47,7 +48,7 @@ public String getDisplayName() {
}

public String getUrlName() {
return "github-webhook";
return URLNAME;
}

/**
Expand Down
@@ -0,0 +1,32 @@
package com.cloudbees.jenkins;

import hudson.Extension;
import hudson.security.csrf.CrumbExclusion;

import javax.servlet.FilterChain;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

import java.io.IOException;
import java.util.logging.Logger;

@Extension
public class GitHubWebHookCrumbExclusion extends CrumbExclusion {

private static final Logger LOGGER = Logger.getLogger("com.cloudbees.jenkins.GitHubWebHookCrumbExclusion");

@Override
public boolean process(HttpServletRequest req, HttpServletResponse resp, FilterChain chain) throws IOException, ServletException {
String pathInfo = req.getPathInfo();
if (pathInfo != null && pathInfo.equals(getExclusionPath())) {
chain.doFilter(req, resp);
return true;
}
return false;
}

public String getExclusionPath() {
return "/" + GitHubWebHook.URLNAME + "/";
}
}

0 comments on commit e065b80

Please sign in to comment.