Skip to content

Add test for OpenJdkSelfSignedCertGenerator - #15619

Merged
normanmaurer merged 1 commit into
4.2from
self_signed_test
Sep 4, 2025
Merged

normanmaurer merged 1 commit into
4.2from
self_signed_test

Conversation

@normanmaurer

Copy link
Copy Markdown
Member

Motivation:

We recently had a problem where we run into issues when we ported some changes to 4.2 that made the OpenJdkSelfSignedCertGenerator not work at all anymore. We should test that it works on the expected JDK versions

Modifications:

Add unit test

Result:

Better coverage

Motivation:

We recently had a problem where we run into issues when we ported some changes to 4.2 that made the OpenJdkSelfSignedCertGenerator not work at all anymore. We should test that it works on the expected JDK versions

Modifications:

Add unit test

Result:

Better coverage
@normanmaurer

Copy link
Copy Markdown
Member Author

/cc @violetagg ... I will pull this in, port to 4.1 and then reapply the changes for JDK24.

@normanmaurer normanmaurer added this to the 4.2.6.Final milestone Sep 4, 2025
@normanmaurer

Copy link
Copy Markdown
Member Author

Related to #15613

@normanmaurer
normanmaurer merged commit f7f5b48 into 4.2 Sep 4, 2025
18 checks passed
@normanmaurer
normanmaurer deleted the self_signed_test branch September 4, 2025 08:19
normanmaurer added a commit that referenced this pull request Sep 4, 2025
Motivation:

We recently had a problem where we run into issues when we ported some
changes to 4.2 that made the OpenJdkSelfSignedCertGenerator not work at
all anymore. We should test that it works on the expected JDK versions

Modifications:

Add unit test

Result:

Better coverage
@violetagg

Copy link
Copy Markdown
Member

Thanks!

normanmaurer added a commit that referenced this pull request Sep 4, 2025
Motivation:

We recently had a problem where we run into issues when we ported some
changes to 4.2 that made the OpenJdkSelfSignedCertGenerator not work at
all anymore. We should test that it works on the expected JDK versions

Modifications:

Add unit test

Result:

Better coverage
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