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

Show xref type in NVT details #2980

Merged
merged 3 commits into from
Jun 15, 2021
Merged

Conversation

swaterkamp
Copy link
Member

@swaterkamp swaterkamp commented Jun 9, 2021

What:
Show the type of xrefs in the reference section of NVTs, except url.

xrefs1
xrefs2

Why:
There seemed to be changes made in the NASL scripts that altered the content of name and type of xrefs.

NOTE: It is not 100% clear how the change of behavior happened but GSA is already getting the type in lower case only, therefore it will appear as that in the details as well..

How:
Visually check whether the type is included.

Checklist:

  • [N/A] Tests
  • CHANGELOG Entry
  • Labels for ports to other branches

@swaterkamp swaterkamp added port-to-main Use mergifiy to port PR to master port-to-stable Use mergifiy to port PR to stable port-to-21.10 labels Jun 9, 2021
@swaterkamp swaterkamp requested a review from sarahd93 June 9, 2021 12:10
@swaterkamp swaterkamp self-assigned this Jun 9, 2021
@swaterkamp swaterkamp requested a review from a team as a code owner June 9, 2021 12:10
@codecov
Copy link

codecov bot commented Jun 11, 2021

Codecov Report

Merging #2980 (cae6e8a) into gsa-20.08 (4ec604b) will increase coverage by 0.00%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           gsa-20.08    #2980   +/-   ##
==========================================
  Coverage      53.74%   53.74%           
==========================================
  Files           1072     1072           
  Lines          25909    25912    +3     
  Branches        7400     7403    +3     
==========================================
+ Hits           13924    13926    +2     
- Misses         10881    10882    +1     
  Partials        1104     1104           
Impacted Files Coverage Δ
gsa/src/web/pages/nvts/references.js 91.30% <66.66%> (-3.70%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4ec604b...cae6e8a. Read the comment docs.

@saberlynx saberlynx enabled auto-merge June 14, 2021 04:35
@bjoernricks bjoernricks merged commit 393f185 into greenbone:gsa-20.08 Jun 15, 2021
swaterkamp added a commit that referenced this pull request Jun 15, 2021
swaterkamp added a commit that referenced this pull request Jun 16, 2021
swaterkamp added a commit that referenced this pull request Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
port-to-main Use mergifiy to port PR to master port-to-stable Use mergifiy to port PR to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants