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

Release v2.0.0 #127

Merged
merged 67 commits into from Nov 28, 2022
Merged

Release v2.0.0 #127

merged 67 commits into from Nov 28, 2022

Conversation

jhollowe
Copy link
Contributor

2.0.0 (2022-11-27)

  • Improvement (all): Convert testing framework to use pytest (John Hollowell)
  • Improvement (all): Remove Python 2.x support (minimum version of 3.7) (John Hollowell)
  • Improvement (all): Refactor code to Python 3 standards (John Hollowell)
  • Bugfix (all): Remove None values from request data and params (Kristian Heljas)
  • Addition (tools): Added Task tools (John Hollowell)
  • Bugfix (all): Allow specifying resource_id as 0 (John Bergvall)
  • Improvement (all): Remove ProxmoxResourceBase (John Hollowell)
  • Bugfix (all): Add platform detection before using shlex functions (Kevin Boyd)
  • Improvement (https): Added path_prefix argument which is appended after the root of the URL (before api2/) (John Hollowell)

Breaking Changes

  • ProxmoxResourceBase removed
  • proxmoxer.backends.https.AuthenticationError moved to proxmoxer.AuthenticationError
  • Removed ProxmoxHTTPTicketAuth and its arguments auth_token and csrf_token
  • keyword arguments to backends order changed (should not affect users specifying arguments by name)

jhollowe and others added 30 commits March 15, 2022 23:40
BREAKING CHANGES:
* argument renamed from `configlfile` to `config_file` to match `identity_file`
* syntax update which is broken on previous python versions
Fix config file not being used in openssh
Remove deprecated ProxmoxHTTPTicketAuth
fix error handling for APIs that don't give a dict in the response
Add platform detection before using shlex functions (don't use Unix shlex functions on Windows)
* Removed py2 compatibility code
* Moved as much as possible into superclasses
* Added `path_prefix` argument to support Proxmox services behind reverse proxies
  * It is assumed the proxy will remove the path prefix before forwarding to the Proxmox service
* Updated pre-commit hook versions and bandit baseline
* BREAKING: Changed ordering of arguments to Auth's (should not cause issues unless directly using Auth's)
Changes to using `python3` while getting a tempfile since recent PVE does not alias `python` to `python3`
fix port not being a str (openssh_wrapper doesn't like it as an int)
  see NetAngels/openssh-wrapper#13 for the upstream fix for this, but it seems the library is no longer maintained
jhollowe and others added 23 commits October 6, 2022 03:07
Add Python 3.11 to CI action
Remove `None` values from request params and data (Fixes #119)
Add running pre-commit in CI for lint/format check
* Moved AuthenticationError to core.py
* Standardized imports in tests

closes #105
Move AuthenticationError to main proxmox. namespace
@jhollowe jhollowe added type:enhancement ⏫ New or updated feature/functionality type:testing 🧪 New or updated tests type:maintenance 🛠️ Small but needed fixes that do not change functionality labels Nov 27, 2022
@jhollowe jhollowe self-assigned this Nov 27, 2022
@jhollowe jhollowe merged commit c82d84a into master Nov 28, 2022
@jhollowe jhollowe deleted the release/2.0.0 branch November 28, 2022 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement ⏫ New or updated feature/functionality type:maintenance 🛠️ Small but needed fixes that do not change functionality type:testing 🧪 New or updated tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants