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

Integration tests for RestClient require periodic regeneration of certificates #27335

Closed
Ladicek opened this issue Aug 17, 2022 · 1 comment · Fixed by #27360
Closed

Integration tests for RestClient require periodic regeneration of certificates #27335

Ladicek opened this issue Aug 17, 2022 · 1 comment · Fixed by #27360
Assignees
Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements
Milestone

Comments

@Ladicek
Copy link
Contributor

Ladicek commented Aug 17, 2022

Description

See e.g. #27320

Implementation ideas

Use the Keytool Maven plugin to generate the necessary certificates during each build, avoiding the requirement to store the certificates in the repo and hence also the need to regenerate them. See e.g. https://github.com/rhoar-qe/thorntail-test-suite/blob/master/protocols/https-2way/pom.xml

@Ladicek Ladicek added the area/housekeeping Issue type for generalized tasks not related to bugs or enhancements label Aug 17, 2022
@Ladicek Ladicek self-assigned this Aug 17, 2022
@Ladicek
Copy link
Contributor Author

Ladicek commented Aug 18, 2022

It turns out that the tests use badssl.com and so requires its certificates. Hence, the Keytool Maven plugin is useless. Fortunately, there's a Maven plugin that can obtain certificates from external sites and store them in a keystore format: https://github.com/automatictester/truststore-maven-plugin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants