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

Fix PKCS #11 constants test #995

Merged
merged 1 commit into from
Feb 16, 2024
Merged

Fix PKCS #11 constants test #995

merged 1 commit into from
Feb 16, 2024

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Feb 16, 2024

The PKCS #11 constants test failed on Fedora 40 since it uses NSS 3.97 that provides some new PKCS #11 constants. As a workaround the new constants need to be excluded until NSS 3.97 becomes available on all supported platforms.

The test has also been modified to reuse the jss-builder image which already has the build dependencies.

Resolves: #993

The PKCS dogtagpki#11 constants test failed on Fedora 40 since it
uses NSS 3.97 that provides some new PKCS dogtagpki#11 constants.
As a workaround the new constants need to be excluded until
NSS 3.97 becomes available on all supported platforms.

The test has also been modified to reuse the jss-builder
image which already has the build dependencies.

Resolves: dogtagpki#993
@edewata edewata requested a review from fmarco76 February 16, 2024 02:28
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Member

@fmarco76 fmarco76 left a comment

Choose a reason for hiding this comment

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

LGTM.

I have some doubt on the utility of this test in general. If we do not declare all the constants but only the one we use/recognise, should this be a problem? We can think a better approach when we will re-introduce these constants in the file.

@edewata
Copy link
Contributor Author

edewata commented Feb 16, 2024

@fmarco76 Thanks!

IIRC older JDKs used to ship Sun packages which provided these constants. Newer JDKs don't ship those packages anymore, so now we're providing these constants. I'm not sure how these constants are actually used, but we can certainly discuss that, probably with @ladycfu and @jmagne as well.

@edewata edewata merged commit da2508e into dogtagpki:master Feb 16, 2024
35 checks passed
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.

PKCS11Constants test failed on Rawhide
2 participants