Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
53c79e41
Commit
53c79e41
authored
Jun 02, 2020
by
Alexander Turinske
Committed by
Natalia Tepluhina
Jun 02, 2020
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Revert "Add scanner name"
This reverts commit
a2bdc67c
parent
7c065e6b
Changes
13
Show whitespace changes
Inline
Side-by-side
Showing
13 changed files
with
11 additions
and
53 deletions
+11
-53
ee/app/assets/javascripts/security_dashboard/components/security_dashboard_table.vue
...ecurity_dashboard/components/security_dashboard_table.vue
+0
-3
ee/app/assets/javascripts/security_dashboard/components/security_dashboard_table_row.vue
...ity_dashboard/components/security_dashboard_table_row.vue
+0
-11
ee/app/assets/javascripts/security_dashboard/helpers.js
ee/app/assets/javascripts/security_dashboard/helpers.js
+1
-1
ee/app/assets/javascripts/security_dashboard/store/modules/filters/constants.js
...pts/security_dashboard/store/modules/filters/constants.js
+1
-1
ee/app/assets/javascripts/security_dashboard/store/modules/filters/state.js
...scripts/security_dashboard/store/modules/filters/state.js
+1
-1
ee/app/assets/javascripts/vulnerabilities/components/vulnerability_list.vue
...scripts/vulnerabilities/components/vulnerability_list.vue
+0
-9
ee/app/assets/javascripts/vulnerabilities/graphql/vulnerability.fragment.graphql
...ts/vulnerabilities/graphql/vulnerability.fragment.graphql
+0
-1
ee/changelogs/unreleased/add-scanner-name-to-row.yml
ee/changelogs/unreleased/add-scanner-name-to-row.yml
+0
-5
ee/changelogs/unreleased/revert-a2bdc67c.yml
ee/changelogs/unreleased/revert-a2bdc67c.yml
+5
-0
ee/spec/frontend/security_dashboard/components/security_dashboard_table_row_spec.js
...dashboard/components/security_dashboard_table_row_spec.js
+0
-8
ee/spec/frontend/vulnerabilities/mock_data.js
ee/spec/frontend/vulnerabilities/mock_data.js
+0
-2
ee/spec/frontend/vulnerabilities/vulnerability_list_spec.js
ee/spec/frontend/vulnerabilities/vulnerability_list_spec.js
+0
-5
locale/gitlab.pot
locale/gitlab.pot
+3
-6
No files found.
ee/app/assets/javascripts/security_dashboard/components/security_dashboard_table.vue
View file @
53c79e41
...
@@ -77,9 +77,6 @@ export default {
...
@@ -77,9 +77,6 @@ export default {
<div
class=
"table-section flex-grow-1"
role=
"rowheader"
>
<div
class=
"table-section flex-grow-1"
role=
"rowheader"
>
{{
s__
(
'
Reports|Vulnerability
'
)
}}
{{
s__
(
'
Reports|Vulnerability
'
)
}}
</div>
</div>
<div
class=
"table-section section-15"
role=
"rowheader"
>
{{
s__
(
'
Reports|Scanner
'
)
}}
</div>
<div
class=
"table-section section-20"
role=
"rowheader"
></div>
<div
class=
"table-section section-20"
role=
"rowheader"
></div>
</div>
</div>
...
...
ee/app/assets/javascripts/security_dashboard/components/security_dashboard_table_row.vue
View file @
53c79e41
...
@@ -43,10 +43,6 @@ export default {
...
@@ -43,10 +43,6 @@ export default {
}
}
return
location
&&
(
location
.
image
||
location
.
file
||
location
.
path
);
return
location
&&
(
location
.
image
||
location
.
file
||
location
.
path
);
},
},
vulnerabilityScanner
()
{
const
{
scanner
}
=
this
.
vulnerability
;
return
scanner
?.
name
;
},
isDismissed
()
{
isDismissed
()
{
return
Boolean
(
this
.
vulnerability
.
dismissal_feedback
);
return
Boolean
(
this
.
vulnerability
.
dismissal_feedback
);
},
},
...
@@ -139,13 +135,6 @@ export default {
...
@@ -139,13 +135,6 @@ export default {
</div>
</div>
</div>
</div>
<div
class=
"table-section section-15"
>
<div
class=
"table-mobile-header"
role=
"rowheader"
>
{{ s__('Reports|Scanner') }}
</div>
<div
class=
"table-mobile-content"
>
{{ vulnerabilityScanner }}
</div>
</div>
<div
class=
"table-section section-20"
>
<div
class=
"table-section section-20"
>
<div
class=
"table-mobile-header"
role=
"rowheader"
>
{{ s__('Reports|Actions') }}
</div>
<div
class=
"table-mobile-header"
role=
"rowheader"
>
{{ s__('Reports|Actions') }}
</div>
<div
class=
"table-mobile-content action-buttons d-flex justify-content-end"
>
<div
class=
"table-mobile-content action-buttons d-flex justify-content-end"
>
...
...
ee/app/assets/javascripts/security_dashboard/helpers.js
View file @
53c79e41
...
@@ -27,7 +27,7 @@ export const initFirstClassVulnerabilityFilters = projects => {
...
@@ -27,7 +27,7 @@ export const initFirstClassVulnerabilityFilters = projects => {
selection
:
new
Set
([
ALL
]),
selection
:
new
Set
([
ALL
]),
},
},
{
{
name
:
s__
(
'
Reports|Scanner
'
),
name
:
s__
(
'
SecurityReports|Report type
'
),
id
:
'
reportType
'
,
id
:
'
reportType
'
,
options
:
[
BASE_FILTERS
.
report_type
,
...
parseOptions
(
REPORT_TYPES
)],
options
:
[
BASE_FILTERS
.
report_type
,
...
parseOptions
(
REPORT_TYPES
)],
selection
:
new
Set
([
ALL
]),
selection
:
new
Set
([
ALL
]),
...
...
ee/app/assets/javascripts/security_dashboard/store/modules/filters/constants.js
View file @
53c79e41
...
@@ -8,7 +8,7 @@ export const BASE_FILTERS = {
...
@@ -8,7 +8,7 @@ export const BASE_FILTERS = {
id
:
ALL
,
id
:
ALL
,
},
},
report_type
:
{
report_type
:
{
name
:
s__
(
'
ciReport|All
scanner
types
'
),
name
:
s__
(
'
ciReport|All
report
types
'
),
id
:
ALL
,
id
:
ALL
,
},
},
project_id
:
{
project_id
:
{
...
...
ee/app/assets/javascripts/security_dashboard/store/modules/filters/state.js
View file @
53c79e41
...
@@ -14,7 +14,7 @@ export default () => ({
...
@@ -14,7 +14,7 @@ export default () => ({
selection
:
new
Set
([
BASE_FILTERS
.
severity
.
id
]),
selection
:
new
Set
([
BASE_FILTERS
.
severity
.
id
]),
},
},
{
{
name
:
s__
(
'
SecurityReports|
Scanner
type
'
),
name
:
s__
(
'
SecurityReports|
Report
type
'
),
id
:
'
report_type
'
,
id
:
'
report_type
'
,
options
:
[
BASE_FILTERS
.
report_type
,
...
optionsObjectToArray
(
REPORT_TYPES
)],
options
:
[
BASE_FILTERS
.
report_type
,
...
optionsObjectToArray
(
REPORT_TYPES
)],
hidden
:
false
,
hidden
:
false
,
...
...
ee/app/assets/javascripts/vulnerabilities/components/vulnerability_list.vue
View file @
53c79e41
...
@@ -93,11 +93,6 @@ export default {
...
@@ -93,11 +93,6 @@ export default {
label
:
__
(
'
Description
'
),
label
:
__
(
'
Description
'
),
thClass
:
commonThClass
,
thClass
:
commonThClass
,
},
},
{
key
:
'
reportType
'
,
label
:
s__
(
'
Reports|Scanner
'
),
thClass
:
`
${
commonThClass
}
`
,
},
];
];
},
},
},
},
...
@@ -206,10 +201,6 @@ export default {
...
@@ -206,10 +201,6 @@ export default {
<remediated-badge
v-if=
"item.resolved_on_default_branch"
class=
"ml-2"
/>
<remediated-badge
v-if=
"item.resolved_on_default_branch"
class=
"ml-2"
/>
</
template
>
</
template
>
<
template
#cell(reportType)=
"{ item }"
>
<span
class=
"text-capitalize js-reportType"
>
{{
item
.
reportType
}}
</span>
</
template
>
<
template
#table-busy
>
<
template
#table-busy
>
<gl-skeleton-loading
<gl-skeleton-loading
v-for=
"n in $options.VULNERABILITIES_PER_PAGE"
v-for=
"n in $options.VULNERABILITIES_PER_PAGE"
...
...
ee/app/assets/javascripts/vulnerabilities/graphql/vulnerability.fragment.graphql
View file @
53c79e41
...
@@ -4,7 +4,6 @@ fragment Vulnerability on Vulnerability {
...
@@ -4,7 +4,6 @@ fragment Vulnerability on Vulnerability {
state
state
severity
severity
vulnerabilityPath
vulnerabilityPath
reportType
location
{
location
{
...
on
VulnerabilityLocationContainerScanning
{
...
on
VulnerabilityLocationContainerScanning
{
image
image
...
...
ee/changelogs/unreleased/add-scanner-name-to-row.yml
deleted
100644 → 0
View file @
7c065e6b
---
title
:
Add scanner name to vulnerability row
merge_request
:
31775
author
:
type
:
changed
ee/changelogs/unreleased/revert-a2bdc67c.yml
0 → 100644
View file @
53c79e41
---
title
:
Revert Add scanner name
merge_request
:
33442
author
:
type
:
removed
ee/spec/frontend/security_dashboard/components/security_dashboard_table_row_spec.js
View file @
53c79e41
...
@@ -77,14 +77,6 @@ describe('Security Dashboard Table Row', () => {
...
@@ -77,14 +77,6 @@ describe('Security Dashboard Table Row', () => {
).
toContain
(
wrapper
.
props
().
vulnerability
.
severity
);
).
toContain
(
wrapper
.
props
().
vulnerability
.
severity
);
});
});
it
(
'
should render the scanner name
'
,
()
=>
{
expect
(
findContent
(
2
)
.
text
()
.
toLowerCase
(),
).
toContain
(
wrapper
.
props
().
vulnerability
.
scanner
.
name
.
toLowerCase
());
});
describe
(
'
the project name
'
,
()
=>
{
describe
(
'
the project name
'
,
()
=>
{
it
(
'
should render the name
'
,
()
=>
{
it
(
'
should render the name
'
,
()
=>
{
expect
(
findContent
(
1
).
text
()).
toContain
(
wrapper
.
props
().
vulnerability
.
name
);
expect
(
findContent
(
1
).
text
()).
toContain
(
wrapper
.
props
().
vulnerability
.
name
);
...
...
ee/spec/frontend/vulnerabilities/mock_data.js
View file @
53c79e41
...
@@ -4,7 +4,6 @@ export const generateVulnerabilities = () => [
...
@@ -4,7 +4,6 @@ export const generateVulnerabilities = () => [
title
:
'
Vulnerability 1
'
,
title
:
'
Vulnerability 1
'
,
severity
:
'
critical
'
,
severity
:
'
critical
'
,
state
:
'
dismissed
'
,
state
:
'
dismissed
'
,
reportType
:
'
sast
'
,
location
:
{
location
:
{
image
:
image
:
'
registry.gitlab.com/groulot/container-scanning-test/master:5f21de6956aee99ddb68ae49498662d9872f50ff
'
,
'
registry.gitlab.com/groulot/container-scanning-test/master:5f21de6956aee99ddb68ae49498662d9872f50ff
'
,
...
@@ -18,7 +17,6 @@ export const generateVulnerabilities = () => [
...
@@ -18,7 +17,6 @@ export const generateVulnerabilities = () => [
title
:
'
Vulnerability 2
'
,
title
:
'
Vulnerability 2
'
,
severity
:
'
high
'
,
severity
:
'
high
'
,
state
:
'
opened
'
,
state
:
'
opened
'
,
reportType
:
'
sast
'
,
location
:
{
location
:
{
file
:
'
src/main/java/com/gitlab/security_products/tests/App.java
'
,
file
:
'
src/main/java/com/gitlab/security_products/tests/App.java
'
,
},
},
...
...
ee/spec/frontend/vulnerabilities/vulnerability_list_spec.js
View file @
53c79e41
...
@@ -63,11 +63,6 @@ describe('Vulnerability list component', () => {
...
@@ -63,11 +63,6 @@ describe('Vulnerability list component', () => {
expect
(
cell
.
text
().
toLowerCase
()).
toBe
(
newVulnerabilities
[
0
].
severity
);
expect
(
cell
.
text
().
toLowerCase
()).
toBe
(
newVulnerabilities
[
0
].
severity
);
});
});
it
(
'
should correctly render the scanner type
'
,
()
=>
{
const
cell
=
findCell
(
'
reportType
'
);
expect
(
cell
.
text
().
toLowerCase
()).
toBe
(
newVulnerabilities
[
0
].
reportType
);
});
it
(
'
should correctly render the description
'
,
()
=>
{
it
(
'
should correctly render the description
'
,
()
=>
{
const
cell
=
findCell
(
'
description
'
);
const
cell
=
findCell
(
'
description
'
);
...
...
locale/gitlab.pot
View file @
53c79e41
...
@@ -18322,9 +18322,6 @@ msgstr ""
...
@@ -18322,9 +18322,6 @@ msgstr ""
msgid "Reports|Metrics reports failed loading results"
msgid "Reports|Metrics reports failed loading results"
msgstr ""
msgstr ""
msgid "Reports|Scanner"
msgstr ""
msgid "Reports|Severity"
msgid "Reports|Severity"
msgstr ""
msgstr ""
...
@@ -19278,10 +19275,10 @@ msgstr ""
...
@@ -19278,10 +19275,10 @@ msgstr ""
msgid "SecurityReports|Remove project from dashboard"
msgid "SecurityReports|Remove project from dashboard"
msgstr ""
msgstr ""
msgid "SecurityReports|Re
turn to dashboard
"
msgid "SecurityReports|Re
port type
"
msgstr ""
msgstr ""
msgid "SecurityReports|
Scanner type
"
msgid "SecurityReports|
Return to dashboard
"
msgstr ""
msgstr ""
msgid "SecurityReports|Security Dashboard"
msgid "SecurityReports|Security Dashboard"
...
@@ -25887,7 +25884,7 @@ msgstr ""
...
@@ -25887,7 +25884,7 @@ msgstr ""
msgid "ciReport|All projects"
msgid "ciReport|All projects"
msgstr ""
msgstr ""
msgid "ciReport|All
scanner
types"
msgid "ciReport|All
report
types"
msgstr ""
msgstr ""
msgid "ciReport|All severities"
msgid "ciReport|All severities"
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment