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

Merge pull request #2204 from kofemann/fix-ssl-param #2223

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

carryel
Copy link
Contributor

@carryel carryel commented Feb 3, 2025

fix regression in SSLEngineConfigurator#setSSLParameters

fix regression in SSLEngineConfigurator#setSSLParameters
@carryel carryel requested a review from arjantijms February 3, 2025 05:27
@carryel
Copy link
Contributor Author

carryel commented Feb 3, 2025

This is the content that was moved from the PR of the master branch to the main branch.

@carryel
Copy link
Contributor Author

carryel commented Feb 11, 2025

This fix is ​​a patch for a very simple typo error. Anyone who can review and approve.

@carryel carryel requested review from dmatej and removed request for a team and arjantijms February 12, 2025 11:00
@carryel
Copy link
Contributor Author

carryel commented Feb 12, 2025

@dmatej could you approve this again? (Because it didn't change to a status where merging was possible)

@pzygielo
Copy link
Contributor

could you approve this again? (Because it didn't change to a status where merging was possible)

For merging to be enabled it requires approval from ee4j-grizzly-contributors.

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.

4 participants