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

update pyopenssl dep to fix #4527 #4528

Merged
merged 1 commit into from Jan 2, 2023
Merged

Conversation

reaperhulk
Copy link
Contributor

@reaperhulk reaperhulk commented Jan 2, 2023

Which issue(s) does this change fix?

#4527

Why is this change necessary?

Exceptions :)

How does it address the issue?

Updates deps to remove the issue.

What side effects does this change have?

Mandatory Checklist

PRs will only be reviewed after checklist is complete

  • Add input/output type hints to new functions/methods
  • Write design document if needed (Do I need to write a design document?)
  • Write/update unit tests
  • Write/update integration tests
  • Write/update functional tests if needed
  • make pr passes
  • make update-reproducible-reqs if dependencies were changed
  • Write documentation

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

NOTE: I ran make update-reproducible-reqs but in the process discovered that it appears to be only updating a minimal set. If you erase that file and recreate then a huge number of deps get different versions. This may be expected behavior (I don't normally use pip-compile), but it seems quite surprising.

@reaperhulk reaperhulk requested a review from a team as a code owner January 2, 2023 09:29
@github-actions github-actions bot added pr/external stage/needs-triage Automatically applied to new issues and PRs, indicating they haven't been looked at. labels Jan 2, 2023
@moelasmar moelasmar merged commit 919d32c into aws:develop Jan 2, 2023
@maikefischer
Copy link

When is this fix going to be released?

@hawflau
Copy link
Contributor

hawflau commented Jan 5, 2023

@maikefischer The fix has just been released - https://github.com/aws/aws-sam-cli/releases/tag/v1.68.0

mildaniel pushed a commit that referenced this pull request Jan 10, 2023
* support lists of source_path dicts

* add unit tests for source_path is specified as a list of dicts

* fix: update pyopenssl dependency (#4528)

* chore(deps): bump setuptools from 54.2.0 to 65.5.1 in /requirements (#4518)

Bumps [setuptools](https://github.com/pypa/setuptools) from 54.2.0 to 65.5.1.
- [Release notes](https://github.com/pypa/setuptools/releases)
- [Changelog](https://github.com/pypa/setuptools/blob/main/CHANGES.rst)
- [Commits](pypa/setuptools@v54.2.0...v65.5.1)

---
updated-dependencies:
- dependency-name: setuptools
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: bump version to 1.68.0 (#4532)

Co-authored-by: Lau <lauwing@f8ffc25e8e59.ant.amazon.com>

* CVE-2007-4559 Patch (#4499)

* Adding tarfile member sanitization to extractall()

* fix formatting issues

* Refactored extract_tarfile functions and added unit/functional tests

Co-authored-by: Mehmet Nuri Deveci <5735811+mndeveci@users.noreply.github.com>
Co-authored-by: Mohamed Elasmar <71043312+moelasmar@users.noreply.github.com>
Co-authored-by: hnnasit <84355507+hnnasit@users.noreply.github.com>
Co-authored-by: Haresh Nasit <hnnasit@amazon.com>
Co-authored-by: Wing Fung Lau <4760060+hawflau@users.noreply.github.com>

* chore: Update development guide to include information on dependency updates (#4476)

* Add in instructions on updating SAMCLI dependencies

* Update the command to be run to write to reproducible linux

* Highlight content using separate sections for requirement files

* Fix typos and reword for better readability

* Update development guide text

* Revert "CVE-2007-4559 Patch (#4499)" (#4535)

This reverts commit 40bd90a.

* chore: change license badge in readme to display GH license (#4538)

* chore: update readme with pypi license

* use gh rather than pypi

* formatting only changes made by terraform fmt command

* add intergration tests

* Re-arrange order of if statements to remove recursion

* remove print statement that was leftover from development

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Mohamed Elasmar <71043312+moelasmar@users.noreply.github.com>
Co-authored-by: Paul Kehrer <paul.l.kehrer@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Wing Fung Lau <4760060+hawflau@users.noreply.github.com>
Co-authored-by: Lau <lauwing@f8ffc25e8e59.ant.amazon.com>
Co-authored-by: TrellixVulnTeam <112716341+TrellixVulnTeam@users.noreply.github.com>
Co-authored-by: Mehmet Nuri Deveci <5735811+mndeveci@users.noreply.github.com>
Co-authored-by: hnnasit <84355507+hnnasit@users.noreply.github.com>
Co-authored-by: Haresh Nasit <hnnasit@amazon.com>
Co-authored-by: Qingchuan Ma <69653965+qingchm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/external stage/needs-triage Automatically applied to new issues and PRs, indicating they haven't been looked at.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants