Skip to content

Conversation

@snyk-bot
Copy link
Contributor

@snyk-bot snyk-bot commented Apr 1, 2022

Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.

Changes included in this PR

  • Changes to the following files to upgrade the vulnerable dependencies to a fixed version:
    • docs/requirements.txt
⚠️ Warning
pymdown-extensions 8.0 requires Markdown, which is not installed.
mkdocs-material 6.2.2 requires markdown, which is not installed.
mkdocs-material 6.2.2 requires mkdocs-material-extensions, which is not installed.
mkdocs-material 6.2.2 requires mkdocs, which is not installed.

Vulnerabilities that will be fixed

By pinning:
Severity Priority Score (*) Issue Upgrade Breaking Change Exploit Maturity
medium severity 556/1000
Why? Recently disclosed, Has a fix available, CVSS 5.4
Cross-site Scripting (XSS)
SNYK-PYTHON-MKDOCS-2438396
mkdocs:
1.2.3 -> 1.3.0
No No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the effected dependencies could be upgraded.

Check the changes in this PR to ensure they won't cause issues with your project.


Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report

🛠 Adjust project settings

📚 Read more about Snyk's upgrade and patch logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Cross-site Scripting (XSS)

The following vulnerabilities are fixed by pinning transitive dependencies:
- https://snyk.io/vuln/SNYK-PYTHON-MKDOCS-2438396
@CLAassistant
Copy link

CLAassistant commented Apr 1, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 3 committers have signed the CLA.

❌ gkorland
❌ snyk-bot
❌ chayim
You have signed the CLA already but the status is still pending? Let us recheck it.

@chayim chayim self-requested a review April 7, 2022 11:33
@codecov
Copy link

codecov bot commented Apr 7, 2022

Codecov Report

Merging #909 (6354549) into master (2dd4ef1) will increase coverage by 0.05%.
The diff coverage is 95.86%.

@@            Coverage Diff             @@
##           master     #909      +/-   ##
==========================================
+ Coverage   81.30%   81.36%   +0.05%     
==========================================
  Files          54       54              
  Lines        8201     8210       +9     
==========================================
+ Hits         6668     6680      +12     
+ Misses       1533     1530       -3     
Impacted Files Coverage Δ
src/execution/DAG/dag_builder.c 100.00% <ø> (ø)
src/execution/DAG/dag_op.c 100.00% <ø> (ø)
src/execution/parsing/script_commands_parser.c 78.57% <ø> (-0.12%) ⬇️
.../serialization/RDB/decoder/previous/v2/decode_v2.c 46.37% <50.00%> (+0.45%) ⬆️
tests/module/LLAPI.c 75.89% <80.00%> (-0.02%) ⬇️
src/execution/background_workers.c 94.63% <94.73%> (+0.01%) ⬆️
src/redisai.c 87.92% <95.45%> (+0.12%) ⬆️
src/backends/tensorflow.c 72.43% <100.00%> (+0.06%) ⬆️
src/execution/DAG/dag.c 91.50% <100.00%> (-0.28%) ⬇️
src/execution/execution_contexts/execution_ctx.c 100.00% <100.00%> (ø)
... and 12 more

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 3f64b9a...6354549. Read the comment docs.

@gkorland gkorland merged commit a45a7ce into master Aug 15, 2022
@gkorland gkorland deleted the snyk-fix-0967dab6c7294532dde05e178b572fee branch August 15, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants