Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the 'New View' sidebar link #6703

Merged
merged 6 commits into from
Jul 18, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
67 changes: 0 additions & 67 deletions core/src/main/java/jenkins/model/NewViewLink.java

This file was deleted.

2 changes: 0 additions & 2 deletions core/src/main/resources/jenkins/model/Messages.properties
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,6 @@ JenkinsLocationConfiguration.does_not_look_like_an_email_address=Does not look l
Mailer.Localhost.Error=Please set a valid host name, instead of localhost
Mailer.NotHttp.Error=The URL is invalid, please ensure you are using http:// or https:// with a valid domain.

NewViewLink.NewView=New View

PatternProjectNamingStrategy.DisplayName=Pattern
PatternProjectNamingStrategy.NamePatternRequired=Name Pattern is required
PatternProjectNamingStrategy.NamePatternInvalidSyntax=regular expression''s syntax is invalid.
Expand Down
2 changes: 0 additions & 2 deletions core/src/main/resources/jenkins/model/Messages_de.properties
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,6 @@ DefaultProjectNamingStrategy.DisplayName=keine Einschränkung
Mailer.Address.Not.Configured=Adresse nicht konfiguriert <nobody@nowhere>
Mailer.Localhost.Error=Bitte verwenden Sie einen konkreten Hostnamen anstelle von <code> localhost </code>.

NewViewLink.NewView=Ansicht anlegen

PatternProjectNamingStrategy.DisplayName=Muster
PatternProjectNamingStrategy.NamePatternRequired=Der Reguläre Ausdruck darf nicht leer sein.
PatternProjectNamingStrategy.NamePatternInvalidSyntax=Der Reguläre Ausdruck ist ungültig.
Expand Down
2 changes: 0 additions & 2 deletions core/src/main/resources/jenkins/model/Messages_fr.properties
Original file line number Diff line number Diff line change
Expand Up @@ -52,5 +52,3 @@ ParameterizedJobMixIn.build_now=Lancer un build
BlockedBecauseOfBuildInProgress.shortDescription=Le build #{0} est déjà en cours {1}
BlockedBecauseOfBuildInProgress.ETA=\ (fin prévue à : {0})
BuildDiscarderProperty.displayName=Supprimer les anciens builds

NewViewLink.NewView=Créer une Vue
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ Mailer.Address.Not.Configured=Indirizzo non ancora configurato \
Mailer.Localhost.Error=Impostare un nome host valido anziché localhost
Mailer.NotHttp.Error=L''URL non è valido, assicurarsi di utilizzare http:// o \
https:// con un dominio valido.
NewViewLink.NewView=Nuova vista
ParameterizedJobMixIn.build_now=Compila ora
ParameterizedJobMixIn.build_with_parameters=Compila con parametri
PatternProjectNamingStrategy.DisplayName=Pattern
Expand Down
2 changes: 0 additions & 2 deletions core/src/main/resources/jenkins/model/Messages_ja.properties
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@ CLI.safe-shutdown.shortDescription=\

DefaultProjectNamingStrategy.DisplayName=デフォルト

NewViewLink.NewView=新規ビュー

PatternProjectNamingStrategy.DisplayName=パターン
PatternProjectNamingStrategy.NamePatternRequired=パターンは必須です。
PatternProjectNamingStrategy.NamePatternInvalidSyntax=正規表現に誤りがあります。
Expand Down
2 changes: 0 additions & 2 deletions core/src/main/resources/jenkins/model/Messages_ko.properties
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ Hudson.ViewAlreadyExists="{0}" 이름의 뷰가 이미 존재합니다.

CLI.restart.shortDescription=Jenkins를 안전하게 재시작

NewViewLink.NewView=새로운 뷰

ParameterizedJobMixIn.build_with_parameters=파라미터와 함께 빌드
ParameterizedJobMixIn.build_now=지금 빌드
BlockedBecauseOfBuildInProgress.shortDescription=빌드 #{0} 가 이미 진행중입니다{1}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ Mailer.NotHttp.Error=A URL é inválida, por favor se certifique que você está
válido.
Hudson.Computer.IncorrectNumberOfExecutors=Campo "Número de executores" incorreto. Ele precisa ser um número não \
negativo.
NewViewLink.NewView=Nova visão
EnforceSlaveAgentPortAdministrativeMonitor.displayName=Impor porta TCP para o agente
BuiltInNodeMigration.DisplayName=Nome do nó embutido e migração de rótulo
SimpleGlobalBuildDiscarderStrategy.displayName=Descartador de construção específico
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,4 +55,3 @@ PatternProjectNamingStrategy.NamePatternInvalidSyntax=Некорректное
ParameterizedJobMixIn.build_with_parameters=Собрать с параметрами
CLI.disable-job.shortDescription=Запретить задачу
CLI.enable-job.shortDescription=Разрешить задачу
NewViewLink.NewView=Новое представление
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ Hudson.NoJavaInPath=java, PATH içerisinde değil. <a href=''{0}/configure''>JDK
Hudson.NoName=İsim belirtilmedi
Hudson.UnsafeChar=''{0}'' güvenli olmayan bir karakter
Hudson.ViewName=Hepsi
NewViewLink.NewView=Yeni Görünüm
ParameterizedJobMixIn.build_now=Şimdi Yapılandır
BlockedBecauseOfBuildInProgress.shortDescription=Yapılandırma #{0} zaten işlemde {1}
BlockedBecauseOfBuildInProgress.ETA=\ (ETA: {0})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,6 @@ JenkinsLocationConfiguration.does_not_look_like_an_email_address=這看起來不
Mailer.Localhost.Error=請設定有效的主機名稱,不要用 localhost
Mailer.NotHttp.Error=URL 無效,請確保您使用 http\:// 或 https\:// 和有效的網域名稱。

NewViewLink.NewView=新增視景

PatternProjectNamingStrategy.DisplayName=樣式
PatternProjectNamingStrategy.NamePatternRequired=一定要輸入名稱樣式
PatternProjectNamingStrategy.NamePatternInvalidSyntax=正規表示式語法錯誤。
Expand Down
2 changes: 1 addition & 1 deletion core/src/main/resources/lib/layout/tabNewItem.jelly
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
</st:documentation>

<div class="tab">
<a href="${href}" class="addTab" aria-label="${attrs.title}">
<a href="${href}" class="addTab" aria-label="${attrs.title}" tooltip="${attrs.title}">
<l:icon src="symbol-add" />
</a>
</div>
Expand Down
70 changes: 0 additions & 70 deletions core/src/test/java/jenkins/model/NewViewLinkTest.java

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -127,8 +127,9 @@ public void doNotAcceptNonHttpBasedRootURL_fromConfigXml() {

@Test
@Issue("SECURITY-1471")
public void cannotInjectJavaScriptUsingRootUrl_inNewViewLinkAction() throws Exception {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've updated the test as the 'New View' sidebar action is gone now. The updated test instead tests the 'New View' tab bar link. I've had to create a new project at the start of the test to bypass the 'Welcome to Jenkins!' screen.

public void cannotInjectJavaScriptUsingRootUrl_inNewViewLink() throws Exception {
JenkinsRule.WebClient wc = j.createWebClient();
j.createFreeStyleProject();

settingRootURL("javascript:alert(123);//");

Expand Down