-
Notifications
You must be signed in to change notification settings - Fork 197
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
Cxf-soap tests fail in FIPS environment (fix for JVM) #5485
Cxf-soap tests fail in FIPS environment (fix for JVM) #5485
Conversation
Native failure has to be investigated further, reported as #5486 |
acfe96b
to
343ea9a
Compare
All certficates and keystores have to be generated via openssl to work correctly in the FIPS, therefore I removed the keytool-plugin and added a description, how to generate certificates. I found several bad passwords in the callbacks and fixed them (I'm not sure why it was working in non-FIPS environment - probably the passwords were not required by the cxf in non-FIPS) . For the FIPS-enabled environment, the profile |
The
What does that do? |
Sorry I forgot to remove it on all places. It shouldn'tbe there |
343ea9a
to
c3ca73e
Compare
Removed. Thanks for noticing! |
It looks good. So we configure openssl with the |
Great work, thanks for taking care, @JiriOndrusek!
Do you happen to know what aspect of the generated keystore is not FIPS-compliant with keytool-maven-plugin? At the first sight, the plugin mojos seem to support all we need: keysize, keyalgorithm, storetype... |
I probably mislead a little bit with my comment. It is possible to run successfully the I remember seeing some not-supported algorithms for signing or similar, when I was trying to use keystore only. I don't remember the detail but will save them during investigation of other FIPS issues. |
TBH I'm using *.cnf mainly because the default values functionality (so user is not forced to fill all - e.g. CN which is "referenced" from the tests.) I'll look into an option with no *.cnf files in the future to be sure whether it is mandatory. |
There are 2 open issues:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for answer @JiriOndrusek . That's look good start and investigation can be done in further prs. Related to .cnf
file, it seems good point to have default value provided, beyond no strong opinion on this.
This sounds like an answer: https://stackoverflow.com/a/50398011
|
Thanks for the explanation. I can confirm the behavior, as I tried JKS keystores at first, but they are not accepted by FIPS-enabled system. I had to move to PKCS12 |
integration-test-groups/cxf-soap/cxf-soap-ws-security-server/README.adoc
Outdated
Show resolved
Hide resolved
integration-test-groups/cxf-soap/cxf-soap-ws-security-server/README.adoc
Outdated
Show resolved
Hide resolved
integration-test-groups/cxf-soap/cxf-soap-ws-security-server/README.adoc
Outdated
Show resolved
Hide resolved
85f7a71
to
c4974c2
Compare
c4974c2
to
7ec670d
Compare
This PR is using BCFIPS, which is not the right call anymore. Once the following change is merged into cxf and leveraged by the CQ, this PR should be reworked. |
Superseeded by #5980 |
fixes #5482