Skip to content

Fix encrypted properties tests to work with WSS4J 4.0.2 - #3458

Open
coheigea wants to merge 1 commit into
mainfrom
coheigea/jasypt
Open

Fix encrypted properties tests to work with WSS4J 4.0.2#3458
coheigea wants to merge 1 commit into
mainfrom
coheigea/jasypt

Conversation

@coheigea

@coheigea coheigea commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

The Jasypt algorithm changed, for now we'll just use the old encrypted values in the CXF tests until we pick up WSS4J 4.0.2 properly.

((java.io.Closeable)port).close();
bus.shutdown(true);
try {
if (!JavaUtils.isFIPSEnabled()) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@coheigea may I suggest please to have two independent test runs in Maven, non-FIPs as defaut:

<plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-surefire-plugin</artifactId>
                <executions>
                    <execution>
                        <id>default-test</id>
                        <configuration>
                            <systemPropertyVariables>
<org.apache.wss4j.crypto.jasypt.useLegacyDefaultAlgorithm>true</org.apache.wss4j.crypto.jasypt.useLegacyDefaultAlgorithm>
                            </systemPropertyVariables>
                        </configuration>
                    </execution>
</plugin>

And FIPS one:

<plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-surefire-plugin</artifactId>

                    <execution>
                        <id>fips-test</id>
                        <goals>
                            <goal>test</goal>
                        </goals>
                        <configuration>
                            <systemPropertyVariables>
                                <org.apache.wss4j.crypto.jasypt.useLegacyDefaultAlgorithm>false</org.apache.wss4j.crypto.jasypt.useLegacyDefaultAlgorithm>
                            </systemPropertyVariables>
                        </configuration>
                    </execution>
                </executions>
            </plugin>

More than happy to help with that, it is very difficult to eliminate side effects of system properties in tests, thank you.

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.

2 participants