Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
LinuxSuRen
jenkins
提交
40dcfcb1
J
jenkins
项目概览
LinuxSuRen
/
jenkins
与 Fork 源项目一致
从无法访问的项目Fork
通知
2
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
J
jenkins
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
40dcfcb1
编写于
1月 28, 2013
作者:
J
Jesse Glick
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
[JENKINS-16368] Reverted faulty fix.
上级
ce170a60
变更
3
隐藏空白更改
内联
并排
Showing
3 changed file
with
19 addition
and
13 deletion
+19
-13
changelog.html
changelog.html
+3
-0
core/src/main/java/jenkins/model/Jenkins.java
core/src/main/java/jenkins/model/Jenkins.java
+6
-11
core/src/test/java/jenkins/model/JenkinsGetRootUrlTest.java
core/src/test/java/jenkins/model/JenkinsGetRootUrlTest.java
+10
-2
未找到文件。
changelog.html
浏览文件 @
40dcfcb1
...
...
@@ -55,6 +55,9 @@ Upcoming changes</a>
<!-- Record your changes in the trunk here. -->
<div
id=
"trunk"
style=
"display:none"
>
<!--=TRUNK-BEGIN=-->
<ul
class=
image
>
<li
class=
'major bug'
>
Reverted change in 1.500 causing serious regression in HTTPS reverse proxy setups.
(
<a
href=
"https://issues.jenkins-ci.org/browse/JENKINS-16368"
>
issue 16368
</a>
)
<li
class=
bug
>
Bogus “Build Record Root Directory” inadequately diagnosed.
(
<a
href=
"https://issues.jenkins-ci.org/browse/JENKINS-16457"
>
issue 16457
</a>
)
...
...
core/src/main/java/jenkins/model/Jenkins.java
浏览文件 @
40dcfcb1
...
...
@@ -1866,20 +1866,15 @@ public class Jenkins extends AbstractCIBase implements ModifiableTopLevelItemGro
* @see #getRootUrlFromRequest()
*/
public
String
getRootUrl
()
{
String
url
=
JenkinsLocationConfiguration
.
get
().
getUrl
();
if
(
url
!=
null
&&
!
url
.
endsWith
(
"/"
))
{
url
+=
'/'
;
if
(
url
!=
null
)
{
if
(!
url
.
endsWith
(
"/"
))
url
+=
'/'
;
return
url
;
}
StaplerRequest
req
=
Stapler
.
getCurrentRequest
();
if
(
req
==
null
)
return
url
;
if
(
url
==
null
)
return
getRootUrlFromRequest
();
// replace current protocol, if any, with request protocol
return
url
.
replaceFirst
(
"^\\w+(?=://)"
,
req
.
getScheme
());
if
(
req
!=
null
)
return
getRootUrlFromRequest
();
return
null
;
}
/**
...
...
core/src/test/java/jenkins/model/JenkinsGetRootUrlTest.java
浏览文件 @
40dcfcb1
...
...
@@ -87,12 +87,20 @@ public class JenkinsGetRootUrlTest {
@Bug
(
16368
)
@Test
public
void
i
nheritProtocolWhenDispatchingRequest
()
{
public
void
doNotI
nheritProtocolWhenDispatchingRequest
()
{
configured
(
"http://configured.host/"
);
accessing
(
"https://real.host/"
);
rootUrlIs
(
"https://configured.host/"
);
rootUrlIs
(
"http://configured.host/"
);
}
@Bug
(
16511
)
@Test
public
void
doNotInheritProtocolWhenDispatchingRequest2
()
{
configured
(
"https://ci/jenkins/"
);
accessing
(
"http://localhost:8080/"
);
rootUrlIs
(
"https://ci/jenkins/"
);
}
private
void
rootUrlIs
(
final
String
expectedRootUrl
)
{
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录