Skip to content
New issue

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

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

Already on GitHub? # to your account

Improve error handling for invalid detailspage links #1986

Merged
merged 8 commits into from
Feb 24, 2020

Conversation

sarahd93
Copy link
Contributor

Improve error messages for invalid detailspage links. Show extra information for invalid CVE links.

Checklist:

gsa/src/web/entity/page.js Outdated Show resolved Hide resolved
gsa/src/web/entity/page.js Outdated Show resolved Hide resolved
gsa/src/web/entity/page.js Outdated Show resolved Hide resolved
Copy link
Contributor

@bjoernricks bjoernricks left a comment

Choose a reason for hiding this comment

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

Looks good. Just add the suggestions and we are done :)

gsa/src/web/entity/page.js Outdated Show resolved Hide resolved
sarahd93 and others added 4 commits February 24, 2020 14:04
Co-Authored-By: cfi-gb <34644702+cfi-gb@users.noreply.github.com>
Co-Authored-By: cfi-gb <34644702+cfi-gb@users.noreply.github.com>
Co-Authored-By: cfi-gb <34644702+cfi-gb@users.noreply.github.com>
Co-Authored-By: Björn Ricks <bjoern.ricks@gmail.com>
@sarahd93 sarahd93 marked this pull request as ready for review February 24, 2020 13:47
@sarahd93 sarahd93 requested a review from a team February 24, 2020 13:47
@sarahd93 sarahd93 force-pushed the invalid_secinfo_links_gsa9 branch from c2895d0 to 8a467fb Compare February 24, 2020 14:07
@codecov
Copy link

codecov bot commented Feb 24, 2020

Codecov Report

Merging #1986 into gsa-9.0 will decrease coverage by <.01%.
The diff coverage is 20%.

Impacted file tree graph

@@             Coverage Diff             @@
##           gsa-9.0    #1986      +/-   ##
===========================================
- Coverage    49.97%   49.96%   -0.01%     
===========================================
  Files         1067     1067              
  Lines        24969    24976       +7     
  Branches      7035     7064      +29     
===========================================
+ Hits         12479    12480       +1     
- Misses       11358    11360       +2     
- Partials      1132     1136       +4
Impacted Files Coverage Δ
gsa/src/web/pages/nvts/detailspage.js 22% <0%> (-0.45%) ⬇️
gsa/src/web/entity/page.js 74% <28.57%> (-7.82%) ⬇️

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 467f33e...8a467fb. Read the comment docs.

@sarahd93 sarahd93 requested a review from bjoernricks February 24, 2020 14:17
@sarahd93 sarahd93 merged commit 1ffb554 into greenbone:gsa-9.0 Feb 24, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants