-
Bug
-
Resolution: Fixed
-
Minor
-
Jenkins v1.565.2 on Windows Server 2003, analysis-core-plugin v1.55. Firefox 30, IE 11
One of our developers noticed that our PC-Lint warnings trend graph only showed builds from the last 30 days and that configuring the graph to show more had no effect.
I noticed while debugging this that the cookie being generated to save the trend graph configuration had the wrong path set so that the project dashboard wasn't loading it. The path was being set to the URL of the trend graph's configuration page (ex. http://jenkins/project with spaces/warnings0/config). If I manually edited the cookie's path to point to the project dashboard (ex. http://jenkins/project with spaces/) it worked. The same issue exists if a project dashboard is opened from a view with spaces in the name (ex. http://jenkins/view/View with spaces/Project_with_underscores).
Near as I can tell the source of the issue is in CookieHandler.java:
/** * Sends a cookie with the specified value. * * @param requestAncestors * the ancestors of the request * @param value * the cookie value * @return the created cookie */ public Cookie create(final List<Ancestor> requestAncestors, final String value) { Cookie cookie = new Cookie(name, value); Ancestor ancestor = requestAncestors.get(requestAncestors.size() - ANCESTOR_PATH_PREFIX); cookie.setPath(ancestor.getUrl()); cookie.setMaxAge(ONE_YEAR); return cookie; }
I think this line of code is the culprit which returning the wrong URL for the cookie's path.
Ancestor ancestor = requestAncestors.get(requestAncestors.size() - ANCESTOR_PATH_PREFIX);
- depends on
-
JENKINS-32377 Upgrade Jetty to fix bugs in cookies
- Resolved
- is duplicated by
-
JENKINS-23063 White spaces in a Project name of a job makes the Test Result Trend fail
- Resolved
-
JENKINS-29269 Changing the trend graph for job doesn't work
- Resolved