Skip to content
This repository has been archived by the owner on Feb 4, 2025. It is now read-only.

Update viper dependency to solve CVE-2021-3121 #244

Merged
merged 1 commit into from
May 10, 2021

Conversation

epapbak
Copy link
Collaborator

@epapbak epapbak commented May 10, 2021

Description

Security fix

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing steps

build and run UTs

Checklist

  • make before_commit passes
  • updated documentation wherever necessary
  • added or modified tests if necessary
  • updated schemas and validators in insights-data-schemas in case of input/output change

@codecov-commenter
Copy link

Codecov Report

Merging #244 (b2108c5) into master (213c5bc) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage   72.27%   72.27%           
=======================================
  Files           9        9           
  Lines         386      386           
=======================================
  Hits          279      279           
  Misses         88       88           
  Partials       19       19           

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 213c5bc...b2108c5. Read the comment docs.

Copy link
Collaborator

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

LGTM

@epapbak epapbak merged commit bafe4ba into RedHatInsights:master May 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants