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

Bug fix for read-only containers #107

Merged
merged 1 commit into from
Jun 14, 2022
Merged

Bug fix for read-only containers #107

merged 1 commit into from
Jun 14, 2022

Conversation

pivotal-david-osullivan
Copy link
Contributor

Summary

Fixes #106

Use Cases

A temp directory is created when additional certificates are supplied (e.g. via binding) for addition to the JVM truststore. Read-only containers with no extra certificates added via bindings should start successfully.

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
  • I have added an integration test, if necessary.
  • I have reviewed the styleguide for guidance on my code quality.
  • I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@pivotal-david-osullivan pivotal-david-osullivan added semver:patch A change requiring a patch version bump type:bug A general bug labels Jun 10, 2022
@pivotal-david-osullivan pivotal-david-osullivan requested a review from a team as a code owner June 10, 2022 20:56
@dmikusa dmikusa merged commit 771acb4 into main Jun 14, 2022
@dmikusa dmikusa deleted the readonly-bug branch June 14, 2022 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch A change requiring a patch version bump type:bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Loading CA certificates fails since #103 was merged
3 participants