[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-05-24 Thread nisshah1...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Nisarg Shah updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-57186  
 
 
  New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
Change By: 
 Nisarg Shah  
 
 
Comment: 
 I tried applying the proposed solution, but few tests failed. So this solution won't work [~nap].  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-issues/JIRA.198939.1556205175000.11553.1558708440809%40Atlassian.JIRA.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-05-24 Thread nisshah1...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Nisarg Shah updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-57186  
 
 
  New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
Change By: 
 Nisarg Shah  
 
 
Comment: 
 Test Failures : [INFO] Results:[INFO][ERROR] Failures:[ERROR] FilePathTest.isDescendant_regularSymlinks:960[ERROR] FilePathTest.isDescendant_worksEvenInSymbolicWorkspace:1110[ERROR] UtilTest.resolveSymlinkToFile:570[ERROR] NewViewLinkTest.getActionsHasPermission:55 expected:<[https://127.0.0.1:8080]/newView> but was:<[]/newView>[ERROR] NewViewLinkTest.getActionsNoPermission:68 expected:<[https://127.0.0.1:8080]/newView> but was:<[]/newView>[ERROR] VirtualFileTest.forFilePath_isDescendant:277->checkCommonAssertionForIsDescendant:312[ERROR] VirtualFileTest.forFilePath_listOnlyDescendants_withoutIllegal:366->checkCommonAssertionForList:392Expected: iterable over [Has name: aa, Has name: ab, Has name: _b] in any order but: No item matches: Has name: _b in [, ][ERROR] VirtualFileTest.forFile_isDescendant:261->checkCommonAssertionForIsDescendant:312[ERROR] VirtualFileTest.forFile_listOnlyDescendants_withoutIllegal:351->checkCommonAssertionForList:392Expected: iterable over [Has name: aa, Has name: ab, Has name: _b] in any order but: No item matches: Has name: _b in [, ][ERROR] Errors:[ERROR] UtilTest.testIsSymlink_ParentIsSymlink:286 » FileSystem C:\OpenSource\core\tar...[ERROR] AtomicFileWriterTest.symlinkToDirectory:80 » FileSystem C:\OpenSource\core\tar...[ERROR] PathRemoverTest.testForceRemoveFile_ParentIsSymbolicLink:184 » FileSystem C:\O...[ERROR] PathRemoverTest.testForceRemoveFile_SymbolicLink:153 » FileSystem C:\OpenSourc...[ERROR] PathRemoverTest.testForceRemoveRecursive_ContainsSymbolicLinks:373 » FileSystem[ERROR] PathRemoverTest.testForceRemoveRecursive_ParentIsSymbolicLink:411 » FileSystem[INFO][ERROR] Tests run: 3712, Failures: 9, Errors: 6, Skipped: 28  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
  

[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-05-24 Thread nisshah1...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Nisarg Shah commented on  JENKINS-57186  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
 Test Failures :  [INFO] Results: [INFO] [ERROR] Failures: [ERROR] FilePathTest.isDescendant_regularSymlinks:960 [ERROR] FilePathTest.isDescendant_worksEvenInSymbolicWorkspace:1110 [ERROR] UtilTest.resolveSymlinkToFile:570 [ERROR] NewViewLinkTest.getActionsHasPermission:55 expected: but was:<[]/newView> [ERROR] NewViewLinkTest.getActionsNoPermission:68 expected: but was:<[]/newView> [ERROR] VirtualFileTest.forFilePath_isDescendant:277->checkCommonAssertionForIsDescendant:312 [ERROR] VirtualFileTest.forFilePath_listOnlyDescendants_withoutIllegal:366->checkCommonAssertionForList:392 Expected: iterable over [Has name: aa, Has name: ab, Has name: _b] in any order but: No item matches: Has name: _b in [, ] [ERROR] VirtualFileTest.forFile_isDescendant:261->checkCommonAssertionForIsDescendant:312 [ERROR] VirtualFileTest.forFile_listOnlyDescendants_withoutIllegal:351->checkCommonAssertionForList:392 Expected: iterable over [Has name: aa, Has name: ab, Has name: _b] in any order but: No item matches: Has name: _b in [, ] [ERROR] Errors: [ERROR] UtilTest.testIsSymlink_ParentIsSymlink:286 » FileSystem C:\OpenSource\core\tar... [ERROR] AtomicFileWriterTest.symlinkToDirectory:80 » FileSystem C:\OpenSource\core\tar... [ERROR] PathRemoverTest.testForceRemoveFile_ParentIsSymbolicLink:184 » FileSystem C:\O... [ERROR] PathRemoverTest.testForceRemoveFile_SymbolicLink:153 » FileSystem C:\OpenSourc... [ERROR] PathRemoverTest.testForceRemoveRecursive_ContainsSymbolicLinks:373 » FileSystem [ERROR] PathRemoverTest.testForceRemoveRecursive_ParentIsSymbolicLink:411 » FileSystem [INFO] [ERROR] Tests run: 3712, Failures: 9, Errors: 6, Skipped: 28  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To 

[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-05-24 Thread nisshah1...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Nisarg Shah commented on  JENKINS-57186  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
 I tried applying the proposed solution, but few tests failed. So this solution won't work Napoleon BlownApart.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-issues/JIRA.198939.1556205175000.11546.1558707600290%40Atlassian.JIRA.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-05-08 Thread nisshah1...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Nisarg Shah commented on  JENKINS-57186  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
 Napoleon BlownApart I think that the solution you proposed would work. Because making URL Relative would be a solution. Oleg Nenashev What is your view?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-issues/JIRA.198939.1556205175000.20911.1557335820192%40Atlassian.JIRA.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-04-26 Thread napoleon_blownap...@abms.net.au (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Napoleon BlownApart updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-57186  
 
 
  New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
Change By: 
 Napoleon BlownApart  
 

  
 
 
 
 

 
 The href of the "New View" link in the left panel is constructed differently from the other links and is an absolute URL rather than a relative URL. I've only noticed this issue because our Jenkins server is setup  behind  inside  a VPN  & SSH, .  So we VPN into the university  and  then SSH to our gateway machine.  Thus  we use two domain name variants to access Jenkins depending on whether the user is in the lab (thus using "jenkins.int") or outside the lab (thus using "jenkins.vpn").The consequence of this is that when an external user accesses the Jenkins server via VPN/SSH using "jenkins.vpn", the "New View" link points to "jenkins.int/newView" instead of "jenkins.vpn/newView".  This means that external users get a 404 error (Server not found) and need to manually correct the url.  (Minor/trivial bug.) In core/src/main/java/jenkins/model/NewViewLink.java, starting line 45 {code:java}@Overridepublic String getUrlName() {String urlName = Jenkins.getInstance().getRootUrl() + URL_NAME;return urlName;}{code}the class prepends _Jenkins.getInstance().getRootUrl()_ to _URL_NAME_.  If "/" is prepended instead, _urlName_ becomes a relative address, and the "New View" link is fixed.As I am not familiar with the Jenkins codebase, I'm not sure if this is the right way to fix this problem.    
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by 

[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-04-26 Thread napoleon_blownap...@abms.net.au (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Napoleon BlownApart updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-57186  
 
 
  New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
Change By: 
 Napoleon BlownApart  
 

  
 
 
 
 

 
 The href of the "New View" link in the left panel is constructed differently from the other links and is an absolute URL rather than a relative URL. I've only noticed this issue because our Jenkins server is setup behind a VPN & SSH, and we use two domain name variants to access Jenkins depending on whether the user is in the lab (thus using "jenkins.int") or outside the lab (thus using "jenkins.vpn").The consequence of this is that when an external user accesses the Jenkins server via VPN/SSH using "jenkins.vpn", the "New View" link points to "jenkins.int/newView" instead of "jenkins.vpn/newView".  This means that external users get a 404 error (Server not found) and need to manually correct the url.  (Minor/trivial bug.)  In core/src/main/java/jenkins/model/NewViewLink.java, starting line 45   {code:java}  @Overridepublic String getUrlName() {String urlName = Jenkins.getInstance().getRootUrl() + URL_NAME;return urlName;}{code}the class prepends _Jenkins.getInstance().getRootUrl()_ to _URL_NAME_.    If "/" is prepended instead, _urlName_ becomes a relative address, and the "New View" link is fixed.As I am not familiar with the Jenkins codebase, I'm not sure if this is the right way to fix this problem.    
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 
 

[JIRA] (JENKINS-57186) New View href is an absolute URL but other menu items are relative URLs

2019-04-26 Thread napoleon_blownap...@abms.net.au (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Napoleon BlownApart updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-57186  
 
 
  New View href is an absolute URL but other menu items are relative URLs   
 

  
 
 
 
 

 
Change By: 
 Napoleon BlownApart  
 
 
Summary: 
 New View href  does not include "${rootURL}/"  is an absolute URL but other menu items are relative URLs  
 

  
 
 
 
 

 
 The href of the "New View" link in the left panel is constructed differently from the other links  and is an absolute URL rather than a relative URL .   The sidepanel2.jelly file seems to be missing ${rootURL}/.   I've only noticed this issue because our Jenkins server is setup behind a VPN & SSH, and we use two domain name variants to access Jenkins depending on whether the user is in the lab (thus using "jenkins.int") or outside the lab (thus using "jenkins.vpn").The consequence of this is that when an external user accesses the Jenkins server via VPN/SSH  using "jenkins.vpn" , the "New View" link points to "jenkins.int/newView" instead of "jenkins.vpn/newView".  This means that external users get a 404 error (Server not found) and need to manually correct the url.  (Minor/trivial bug.)     
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





--