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

Backward compatible Update to support Python 3.10 that Passes all unit tests #201

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

warnes
Copy link

@warnes warnes commented Jun 25, 2024

This pull makes three minor changes to enable pyswagger to be used with Python 3.10 and the current versions of PyYaml and Flask.

These changes should be backward compatible, and pass all unit tests.

  1. Adapt to the move of collections.Mapping and collections.MutableMapping to collections.abc.Mapping and collections.abc.MutableMapping.
  2. Use PyYAML's yaml.safe_load instead of yaml.load, which now requires an additional argument, and also prevent arbitrary code execution.
  3. Update the flask tests to account for a change in how multiple values for the same header key are handled.

warnes added 7 commits June 24, 2024 22:32
…ing` under Python 3.10, where these have moved to `collections.abc.Mapping` and `collections.abc.MutableMapping`.
…t with `yaml.safe_load` which prevnets arbitrary code execution.
…i. Providing multiple values for the same header key now always results in a list of values, rather than preserving only the last one.
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.

1 participant