Skip to content

Conversation

@therealcisse
Copy link

Description

This PR adds support for specifying a separate password for key pairs within the keystore, which is useful when the keystore password and the private key password differ.

Changes

  • Added keyPairsPassword parameter to SSL configurations
  • Updated ClientSSLConverter and ServerSSLDecoder to handle separate passwords for keystore and key pairs
  • Maintains backward compatibility by falling back to keystore password if key pairs password is not specified
  • Added configuration support for keyPairsPassword in both client and server SSL configs

Motivation

In some SSL/TLS configurations, the keystore password (used to access the keystore) can be different from the password protecting the private keys within the keystore. This is a common security practice. Previously, zio-http assumed both passwords were the same, which limited compatibility with certain keystore configurations.

Backward Compatibility

This change is fully backward compatible. If keyPairsPassword is not specified, it will fall back to using the keyManagerPassword as before.

- Add keyPairsPassword parameter to SSL configurations
- Allow different passwords for keystore and key pairs
- Update ClientSSLConverter and ServerSSLDecoder to handle separate passwords
- Maintain backward compatibility by falling back to keystore password if key pairs password not specified
- Add configuration support for keyPairsPassword in both client and server SSL configs
@netlify
Copy link

netlify bot commented Jul 5, 2025

Deploy Preview for zio-http ready!

Name Link
🔨 Latest commit 2f13add
🔍 Latest deploy log https://app.netlify.com/projects/zio-http/deploys/6869102a473c3f0008d9dfa8
😎 Deploy Preview https://deploy-preview-3577--zio-http.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CLAassistant
Copy link

CLAassistant commented Jul 5, 2025

CLA assistant check
All committers have signed the CLA.

@987Nabil 987Nabil closed this Sep 20, 2025
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.

3 participants