Consolidated CVE fixes #2
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fix syntax error Switch to mock Fix print function Get TemporaryDirectory for Python2 We don't really care about the encoding of the file Its a script Use six to get assertRegex Create a nullcontext Fix unicode problem Just use the Python2 methods
rickprice
force-pushed
the
Consolidated_CVE_Fixes
branch
from
37442c8 to
2e75588
Compare
More unfinished changes Unfinished changes Unfinished changes All code changes should be in, time to test them Fix things Tox didn't like for Python3 Fix more errors found during testing Try to fix DDT problem with Python2
…a73681a9c4025bbeb58b81e72588ba37e00cff87R355 Fix format string problems in Python2 Try to fix some errors Fix importing of pathlib Fixes to get GitPython to work on Python2 Backport Pathlib2 to Pathlib This is probably more correct since it won't drag in tests we haven't tried before Fix errors caused by git_unpack_args More changes to try and deal with errors Fix problem with test not running
Try to merge changes from the patch, modified call to maybe_patch_caller_env More code changes More changes Fix syntax error Add blank line to remove it Remove blank line again Fix syntax error Fix syntax error Fix Python3 f string Add in venv, and sort list Change name of package to be more correct
rickprice
force-pushed
the
Consolidated_CVE_Fixes
branch
from
07ce38c to
5224461
Compare
Fixups: Code fixups after squashing Fixups: Fix test-requirements
rickprice
force-pushed
the
Consolidated_CVE_Fixes
branch
from
8d78e9c to
d1e140f
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These are the fixes for:
CVE-2022-24439
CVE-2023-40267
CVE-2023-40590