Skip to content

Update dependencies and fix Windows compatibility on Python 3.12 #183

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

joseavegaa
Copy link
Contributor

This is follow-up of the now closed PR #174. This new PR tries to update the PKCS#11 specification to the latest published version and brings compatibility to Python 3.12 on Windows.

The specific changes of this PR are:

The package has been tested on new virtual environments using Python 3.12 on Windows 11 with the asepkcs.dll (Athena Smartcard System ASE Card).

"Topic :: Software Development :: Libraries :: Python Modules",
"Topic :: System :: Hardware :: Hardware Drivers",
"Topic :: System :: Hardware :: Hardware Drivers :: Smartcard",
"Topic :: System :: Hardware :: Hardware Drivers :: Token"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only minor comment: please leave the indentation intact. Hard to say what you changed here?

Comment on lines -2 to +7
* /Distributed under the terms of the OASIS IPR Policy,
* [http://www.oasis-open.org/policies-guidelines/ipr], AS-IS, WITHOUT ANY
* IMPLIED OR EXPRESS WARRANTY; there is no warranty of MERCHANTABILITY, FITNESS FOR A
* PARTICULAR PURPOSE or NONINFRINGEMENT of the rights of others.
*/

/* Latest version of the specification:
* http://docs.oasis-open.org/pkcs11/pkcs11-base/v2.40/pkcs11-base-v2.40.html
/*
* PKCS #11 Specification Version 3.1
* OASIS Standard
* 23 July 2023
* Copyright (c) OASIS Open 2023. All Rights Reserved.
* Source: https://docs.oasis-open.org/pkcs11/pkcs11-spec/v3.1/os/include/pkcs11-v3.1/
* Latest stage of narrative specification: https://docs.oasis-open.org/pkcs11/pkcs11-spec/v3.1/pkcs11-spec-v3.1.html
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Distribution terms are no longer noted here. I think it's still good to keep them here, right?

@mathiasertl
Copy link
Collaborator

To be transparent about the maintenance situation: I don't have any access to HSMs. I'm not sure about @kislyuk?

This means that any changes cannot currently be tested with real HSMs.

@is-alnilam
Copy link

@mathiasertl I can't promise anything ongoing, but I might be able to run the tests on Linux against AWS CloudHSMv2, if that would be of any use? Someone would have to note down exactly how to run the tests though! (I'd rather not try to work it out, get it wrong, and invalidate the results.)

@XobrekX
Copy link

XobrekX commented Dec 5, 2024

Any update on this pull request? I'd really love to see 0.8.0 being released and finally being able to use python 3.12 on my windows box.

@haagensen
Copy link

I guess this means it won't work with Python 3.13 also?

@mathiasertl
Copy link
Collaborator

I guess so.

@mathiasertl
Copy link
Collaborator

This PR has minor changes requested, if they are implemented I can merge this.

Note however that I don't think I can upload releases to PyPI at the moment. @kislyuk do you have upload permissions by now?

@kislyuk
Copy link
Member

kislyuk commented Dec 9, 2024

I do not. @danni can you please invite me to the python-pkcs11 package on PyPI?

@danni
Copy link
Collaborator

danni commented Dec 10, 2024

Yes I can do that. Same username?

@kislyuk
Copy link
Member

kislyuk commented Dec 10, 2024 via email

@mathiasertl
Copy link
Collaborator

My username is mati: https://pypi.org/user/mati/

@wrichardsonafmil
Copy link

Any updates on when this might get merged in?

@gilweis
Copy link

gilweis commented Apr 14, 2025

Tested under 3.13 and it's ok.
Please upload the releases to PyPI 🙏

@joseavegaa
Copy link
Contributor Author

Hi! Are there any updates on this PR and if it will get merge? It adds support for the latest versions of Python, which is something that I really need.

@kislyuk
Copy link
Member

kislyuk commented May 2, 2025

@mathiasertl have your blocking comments been addressed?

@@ -4,26 +4,31 @@ build-backend = "setuptools.build_meta"

[project]
name = "python-pkcs11"
version = "0.7.0"
version = "0.8.0"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not change the version manually, it will be changed automatically by the release process.

Copy link
Member

@kislyuk kislyuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes to setup.py need to be reverted before this PR can be merged.

@xues-d
Copy link

xues-d commented May 8, 2025

@mathiasertl - You can test via an HSM simulator. I typically use the Utimaco simulator which you can get for free you just have to make an account on their site. They have a traditional one called Cryptoserver (latest version is 6.1) and a post quantum one. The simulators work on Windows or Linux (with Linux you don't need to install anything, simply have JDK, copy files, and run script - supports both OpenJDK and Oracle Java framework. - https://utimaco.com/data-protection/simulators-and-trials. I would like this update as well, have a tool working in Linux but cannot make an easy deployable version to Windows...

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.

Cannot install python-pkcs11 using python 3.12 New realese with support for python >= 3.10 on Windows
10 participants