Skip to content

ssl session cache, exportable flag#16322

Closed
icing wants to merge 3 commits intocurl:masterfrom
icing:ssls-exportable-flag
Closed

ssl session cache, exportable flag#16322
icing wants to merge 3 commits intocurl:masterfrom
icing:ssls-exportable-flag

Conversation

@icing
Copy link
Copy Markdown
Contributor

@icing icing commented Feb 13, 2025

Add flag exportable to SSL session cache peers, set to TRUE when sessions for this peer can be exported. This evalualtes if the peer uses confidential information (like srp username/password), a client certificate OR if the "ssl_peer_key" contains relative paths.

When SSL is configured with paths for relevant components, like CA trust anchors, an attempt is made to make this path absolute. When that does not work or the infrstructure is not available, the peer key is marked as local.

Exporting sessions based on relative paths may lead to confusion when later imported in another execution context.

Give peers and `exportable` flag, set TRUE when sessions for
this peer should not be exported. This evalualtes if the peer
uses confidential information (like srp username/password),
a client certificate OR if the "ssl_peer_key" contains relative
paths.

When SSL is configured with paths for relevant components, like
CA trust anchors, an attempt is made to make this path absolute.
When that does not work or the infrstructure is not available, the
peer key is marked as *local*.

Exporting sessions based on relative paths may lead to confusion
when later imported in another execution context.
@github-actions github-actions bot added the TLS label Feb 13, 2025
@bagder
Copy link
Copy Markdown
Member

bagder commented Feb 13, 2025

Give peers and exportable flag, set TRUE when sessions for this peer should not be exported

Really? Was this a typo? It sounds... odd.

@icing
Copy link
Copy Markdown
Contributor Author

icing commented Feb 13, 2025

Give peers and exportable flag, set TRUE when sessions for this peer should not be exported

Really? Was this a typo? It sounds... odd.

Typo for proof of humanity.

@github-actions github-actions bot added the tests label Feb 13, 2025
@bagder bagder closed this in 43012cb Feb 17, 2025
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Give peers and `exportable` flag, set TRUE when sessions for this peer
should not be exported. This evalualtes if the peer uses confidential
information (like srp username/password), a client certificate OR if the
"ssl_peer_key" contains relative paths.

When SSL is configured with paths for relevant components, like CA trust
anchors, an attempt is made to make this path absolute. When that does
not work or the infrstructure is not available, the peer key is marked
as *local*.

Exporting sessions based on relative paths may lead to confusion when
later imported in another execution context.

Closes curl#16322
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants