Index: main/core/src/main/java/hudson/util/FormFieldValidator.java =================================================================== --- main/core/src/main/java/hudson/util/FormFieldValidator.java (revision 13501) +++ main/core/src/main/java/hudson/util/FormFieldValidator.java (working copy) @@ -6,6 +6,7 @@ import hudson.Util; import hudson.model.AbstractProject; import hudson.model.Hudson; +import hudson.model.TopLevelItem; import hudson.security.Permission; import hudson.security.AccessControlled; @@ -56,6 +57,25 @@ this(request, response, adminOnly?Hudson.getInstance():null, adminOnly?CHECK:null); } + /** + * @param projectName + * Name of TopLevelItem to check permission against. If null or invalid, + * then checks for admin permission instead. + * @param permission Permission to check if a valid projectName is given. + */ + protected FormFieldValidator(StaplerRequest request, StaplerResponse response, String projectName, Permission permission) { + this.request = request; + this.response = response; + TopLevelItem project = projectName != null ? Hudson.getInstance().getItem(projectName) : null; + if (project instanceof AccessControlled) { + this.subject = (AccessControlled)project; + this.permission = permission; + } else { + this.subject = Hudson.getInstance(); + this.permission = CHECK; + } + } + protected FormFieldValidator(StaplerRequest request, StaplerResponse response, Permission permission) { this(request,response,Hudson.getInstance(),permission); } Index: main/core/src/main/java/hudson/scm/SubversionSCM.java =================================================================== --- main/core/src/main/java/hudson/scm/SubversionSCM.java (revision 13501) +++ main/core/src/main/java/hudson/scm/SubversionSCM.java (working copy) @@ -13,6 +13,7 @@ import hudson.model.AbstractProject; import hudson.model.BuildListener; import hudson.model.Hudson; +import hudson.model.Item; import hudson.model.ParameterValue; import hudson.model.ParametersAction; import hudson.model.TaskListener; @@ -1258,8 +1259,8 @@ * validate the value for a remote (repository) location. */ public void doSvnRemoteLocationCheck(final StaplerRequest req, StaplerResponse rsp) throws IOException, ServletException { - // this can be used to hit any accessible URL, so limit that to admins - new FormFieldValidator(req, rsp, true) { + // this can be used to hit any accessible URL, do only basic check for non-admins + new FormFieldValidator(req, rsp, req.getParameter("project"), Item.CONFIGURE) { protected void check() throws IOException, ServletException { // syntax check first String url = Util.nullify(request.getParameter("value")); @@ -1277,8 +1278,10 @@ return; } - // test the connection - try { + // test the connection (admins only) + if (!Hudson.getInstance().hasPermission(Hudson.ADMINISTER)) { + ok(); + } else try { SVNURL repoURL = SVNURL.parseURIDecoded(url); if (checkRepositoryPath(repoURL)==SVNNodeKind.NONE) { SVNRepository repository = null; Index: main/core/src/main/java/hudson/scm/browsers/FishEyeSVN.java =================================================================== --- main/core/src/main/java/hudson/scm/browsers/FishEyeSVN.java (revision 13501) +++ main/core/src/main/java/hudson/scm/browsers/FishEyeSVN.java (working copy) @@ -2,6 +2,8 @@ import static hudson.Util.fixEmpty; import hudson.model.Descriptor; +import hudson.model.Hudson; +import hudson.model.Item; import hudson.scm.RepositoryBrowser; import hudson.scm.SubversionChangeLogSet.LogEntry; import hudson.scm.SubversionChangeLogSet.Path; @@ -116,7 +118,8 @@ * Performs on-the-fly validation of the URL. */ public void doCheck(StaplerRequest req, StaplerResponse rsp) throws IOException, ServletException { - new FormFieldValidator.URLCheck(req,rsp) { + new FormFieldValidator(req,rsp,req.getParameter("project"),Item.CONFIGURE) { + @Override protected void check() throws IOException, ServletException { String value = fixEmpty(request.getParameter("value")); if(value==null) {// nothing entered yet @@ -130,14 +133,25 @@ return; } - try { - if(findText(open(new URL(value)),"FishEye")) { - ok(); - } else { - error("This is a valid URL but it doesn't look like FishEye"); - } - } catch (IOException e) { - handleIOException(value,e); + // Connect to URL and check content only if we have admin permission + if (Hudson.getInstance().hasPermission(Hudson.ADMINISTER)) { + final String finalValue = value; + new FormFieldValidator.URLCheck(request,response) { + @Override + protected void check() throws IOException, ServletException { + try { + if(findText(open(new URL(finalValue)),"FishEye")) { + ok(); + } else { + error("This is a valid URL but it doesn't look like FishEye"); + } + } catch (IOException e) { + handleIOException(finalValue,e); + } + } + }.process(); + } else { + ok(); } } }.process(); Index: main/core/src/main/java/hudson/scm/browsers/FishEyeCVS.java =================================================================== --- main/core/src/main/java/hudson/scm/browsers/FishEyeCVS.java (revision 13501) +++ main/core/src/main/java/hudson/scm/browsers/FishEyeCVS.java (working copy) @@ -2,6 +2,8 @@ import hudson.Util; import hudson.model.Descriptor; +import hudson.model.Hudson; +import hudson.model.Item; import hudson.scm.CVSChangeLogSet; import hudson.scm.CVSChangeLogSet.File; import hudson.scm.CVSChangeLogSet.Revision; @@ -70,7 +72,7 @@ } public void doCheck(StaplerRequest req, StaplerResponse rsp) throws IOException, ServletException { - new FormFieldValidator.URLCheck(req,rsp) { + new FormFieldValidator(req,rsp,req.getParameter("project"),Item.CONFIGURE) { @Override protected void check() throws IOException, ServletException { String value = Util.fixEmpty(request.getParameter("value")); @@ -85,14 +87,25 @@ errorWithMarkup("The URL should end like .../browse/foobar/"); return; } - try { - if (findText(open(new URL(value)), "FishEye")) { - ok(); - } else { - error("This is a valid URL but it doesn't look like FishEye"); - } - } catch (IOException e) { - handleIOException(value, e); + // Connect to URL and check content only if we have admin permission + if (Hudson.getInstance().hasPermission(Hudson.ADMINISTER)) { + final String finalValue = value; + new FormFieldValidator.URLCheck(request,response) { + @Override + protected void check() throws IOException, ServletException { + try { + if (findText(open(new URL(finalValue)), "FishEye")) { + ok(); + } else { + error("This is a valid URL but it doesn't look like FishEye"); + } + } catch (IOException e) { + handleIOException(finalValue, e); + } + } + }.process(); + } else { + ok(); } } }.process(); Index: main/core/src/main/java/hudson/tasks/BuildTrigger.java =================================================================== --- main/core/src/main/java/hudson/tasks/BuildTrigger.java (revision 13501) +++ main/core/src/main/java/hudson/tasks/BuildTrigger.java (working copy) @@ -272,7 +272,7 @@ * Form validation method. */ public void doCheck( StaplerRequest req, StaplerResponse rsp ) throws IOException, ServletException { - new FormFieldValidator(req,rsp,true) { + new FormFieldValidator(req,rsp,req.getParameter("project"),Item.CONFIGURE) { protected void check() throws IOException, ServletException { String list = request.getParameter("value"); Index: main/core/src/main/java/hudson/triggers/TimerTrigger.java =================================================================== --- main/core/src/main/java/hudson/triggers/TimerTrigger.java (revision 13501) +++ main/core/src/main/java/hudson/triggers/TimerTrigger.java (working copy) @@ -59,7 +59,8 @@ * Performs syntax check. */ public void doCheck(StaplerRequest req, StaplerResponse rsp) throws IOException, ServletException { - new FormFieldValidator(req,rsp,true) { + new FormFieldValidator(req,rsp,req.getParameter("project"),Item.CONFIGURE) { + @Override protected void check() throws IOException, ServletException { try { String msg = CronTabList.create(fixNull(request.getParameter("value"))).checkSanity(); Index: main/core/src/main/resources/hudson/scm/SubversionSCM/config.jelly =================================================================== --- main/core/src/main/resources/hudson/scm/SubversionSCM/config.jelly (revision 13501) +++ main/core/src/main/resources/hudson/scm/SubversionSCM/config.jelly (working copy) @@ -4,7 +4,7 @@ + checkUrl="'${rootURL}/scm/SubversionSCM/svnRemoteLocationCheck?project=${it.name}&value='+encode(this.value)"/> - \ No newline at end of file + Index: main/core/src/main/resources/hudson/scm/browsers/FishEyeSVN/config.jelly =================================================================== --- main/core/src/main/resources/hudson/scm/browsers/FishEyeSVN/config.jelly (revision 13501) +++ main/core/src/main/resources/hudson/scm/browsers/FishEyeSVN/config.jelly (working copy) @@ -1,9 +1,9 @@ + checkUrl="'${rootURL}/repositoryBrowser/FishEyeSVN/check?project=${it.name}&value='+escape(this.value)"/> - \ No newline at end of file + Index: main/core/src/main/resources/hudson/scm/browsers/FishEyeCVS/config.jelly =================================================================== --- main/core/src/main/resources/hudson/scm/browsers/FishEyeCVS/config.jelly (revision 13501) +++ main/core/src/main/resources/hudson/scm/browsers/FishEyeCVS/config.jelly (working copy) @@ -1,6 +1,6 @@ + checkUrl="'${rootURL}/repositoryBrowser/FishEyeCVS/check?project=${it.name}&value='+escape(this.value)"/> Index: main/core/src/main/resources/hudson/scm/browsers/Sventon/config.jelly =================================================================== --- main/core/src/main/resources/hudson/scm/browsers/Sventon/config.jelly (revision 13501) +++ main/core/src/main/resources/hudson/scm/browsers/Sventon/config.jelly (working copy) @@ -1,9 +1,12 @@ + + + + - + - \ No newline at end of file + Index: main/core/src/main/resources/hudson/tasks/BuildTrigger/config.jelly =================================================================== --- main/core/src/main/resources/hudson/tasks/BuildTrigger/config.jelly (revision 13501) +++ main/core/src/main/resources/hudson/tasks/BuildTrigger/config.jelly (working copy) @@ -1,7 +1,7 @@ + checkUrl="'${rootURL}/publisher/BuildTrigger/check?project=${it.name}&value='+escape(this.value)"/> @@ -9,4 +9,4 @@ - \ No newline at end of file + Index: main/core/src/main/resources/hudson/triggers/TimerTrigger/config.jelly =================================================================== --- main/core/src/main/resources/hudson/triggers/TimerTrigger/config.jelly (revision 13501) +++ main/core/src/main/resources/hudson/triggers/TimerTrigger/config.jelly (working copy) @@ -1,5 +1,5 @@ - + - \ No newline at end of file + Index: main/core/src/main/resources/hudson/triggers/SCMTrigger/config.jelly =================================================================== --- main/core/src/main/resources/hudson/triggers/SCMTrigger/config.jelly (revision 13501) +++ main/core/src/main/resources/hudson/triggers/SCMTrigger/config.jelly (working copy) @@ -1,5 +1,5 @@ - + - \ No newline at end of file + Index: main/core/src/main/resources/lib/hudson/project/config-upstream-pseudo-trigger.jelly =================================================================== --- main/core/src/main/resources/lib/hudson/project/config-upstream-pseudo-trigger.jelly (revision 13501) +++ main/core/src/main/resources/lib/hudson/project/config-upstream-pseudo-trigger.jelly (working copy) @@ -13,8 +13,8 @@ - \ No newline at end of file +