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.

Sign up for GitHub

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

Jump to bottom

PR - Issue 50387 - enable_tls() should label ports with ldap_port_t #3446

Closed

389-ds-bot opened this issue Sep 13, 2020 · 6 comments

Labels

merged pr

389-ds-bot commented Sep 13, 2020 •

edited

Cloned from Pagure Pull-Request: https://pagure.io/389-ds-base/pull-request/50388

Created at 2019-05-17 15:46:36 by vashirov (@vashirov)
Merged at 2019-05-20 14:57:57

Bug Description:
In some tests we use enable_tls(), but the secure port doesn't get
labeled automatically with ldap_port_t.

Fix Description:
Fix enable_tls() to label secure port.

Fixes Resolves: #3445

Reviewed by: ???

389-ds-bot added merged pr labels

389-ds-bot closed this as completed

Author

389-ds-bot commented Sep 13, 2020

Comment from mreynolds (@mreynolds389) at 2019-05-17 15:49:03

Ack

Author

389-ds-bot commented Sep 13, 2020

Comment from vashirov (@vashirov) at 2019-05-17 16:23:16

rebased onto 487899c80ae126c28aac8affb1140b9998aa9e12

Author

389-ds-bot commented Sep 13, 2020

Comment from vashirov (@vashirov) at 2019-05-20 14:50:07

rebased onto 90700854f2540605d0e3ecdae5fe2e4fe6a8b6fe

Author

389-ds-bot commented Sep 13, 2020

Comment from vashirov (@vashirov) at 2019-05-20 14:50:53

rebased onto f2c63bc

Author

389-ds-bot commented Sep 13, 2020

Comment from vashirov (@vashirov) at 2019-05-20 14:57:57

Pull-Request has been merged by vashirov

Author

389-ds-bot commented Sep 13, 2020

Patch
50388.patch

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