Skip to content

Comments

SSL_CTX_is_server() was added#29635

Closed
igus68 wants to merge 1 commit intoopenssl:masterfrom
igus68:fix-1620
Closed

SSL_CTX_is_server() was added#29635
igus68 wants to merge 1 commit intoopenssl:masterfrom
igus68:fix-1620

Conversation

@igus68
Copy link
Contributor

@igus68 igus68 commented Jan 14, 2026

Fixes #1620

Checklist
  • documentation is added or updated
  • tests are added or updated

paulidale
paulidale previously approved these changes Jan 14, 2026
@paulidale paulidale added branch: master Applies to master branch approval: review pending This pull request needs review by a committer triaged: feature The issue/pr requests/adds a feature labels Jan 14, 2026
Copy link
Member

@mattcaswell mattcaswell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. One minor nit.

@mattcaswell mattcaswell requested a review from paulidale January 15, 2026 10:34
Copy link
Member

@beldmit beldmit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@beldmit beldmit added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Jan 15, 2026
Copy link
Contributor

@paulidale paulidale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't think the NIT fix was required but am happy with it.

@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Jan 16, 2026
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@t8m t8m added the tests: present The PR has suitable tests present label Jan 16, 2026
@t8m
Copy link
Member

t8m commented Jan 16, 2026

Merged to the master branch. Thank you @igus68 for fixing one of the oldest open issues!

@t8m t8m closed this Jan 16, 2026
openssl-machine pushed a commit that referenced this pull request Jan 16, 2026
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Dmitry Belyavskiy <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tim Hudson <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Fri Jan 16 13:19:25 2026
(Merged from #29635)
esyr pushed a commit to esyr/openssl that referenced this pull request Jan 19, 2026
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Dmitry Belyavskiy <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tim Hudson <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Fri Jan 16 13:19:25 2026
(Merged from openssl#29635)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch severity: ABI change This pull request contains ABI changes tests: present The PR has suitable tests present triaged: feature The issue/pr requests/adds a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add SSL_CTX_is_server function

7 participants