Skip to content

[LDAP] Add TIMEOUT Option to LDAP Connection Options #28853

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

Merged
merged 1 commit into from
Dec 1, 2018

Conversation

lmatte7
Copy link
Contributor

@lmatte7 lmatte7 commented Oct 13, 2018

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

This PR adds a TIMEOUT (Full name LDAP_OPT_TIMEOUT) option to the ConnectionOptions.php class in the LDAP Component. This option is not documented in the PHP Docs for the LDAP component but is required to set a timeout for certain server configurations. One use case for this option is if the LDAP server has a whitelist the client is not approved to access.

@javiereguiluz
Copy link
Member

@lmatte7 thanks for contributing this (and congrats on your first Symfony contribution!)

In order to update the Symfony Docs about LDAP, we'd need to know which changes are required to use this feature:

Thanks a lot!

@lmatte7
Copy link
Contributor Author

lmatte7 commented Nov 7, 2018

@javiereguiluz thanks for the comment, I'm excited about being able to contribute!

Looking at the current docs I don't believe this option will need to be added specifically to any page. This page links out to show all of the available connection options, so as long as the option will be listed here it aligns with the current documentation.

As for a default option, I don't see one while looking through the PHP documentation or source code. I don't believe this option has a value until it is manually set by the code, and this aligns with my testing of this issue as well.

@javiereguiluz
Copy link
Member

@lmatte7 great! Thanks a lot for checking the docs. This is much appreciated!

@nicolas-grekas nicolas-grekas modified the milestones: next, 4.2 Dec 1, 2018
@nicolas-grekas nicolas-grekas changed the base branch from master to 4.2 December 1, 2018 09:07
@nicolas-grekas
Copy link
Member

Thank you @lmatte7.

@nicolas-grekas nicolas-grekas merged commit 2d88b89 into symfony:4.2 Dec 1, 2018
nicolas-grekas added a commit that referenced this pull request Dec 1, 2018
…te7)

This PR was submitted for the master branch but it was squashed and merged into the 4.2 branch instead (closes #28853).

Discussion
----------

[LDAP] Add TIMEOUT Option to LDAP Connection Options

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? |no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

This PR adds a TIMEOUT (Full name `LDAP_OPT_TIMEOUT`) option to the `ConnectionOptions.php` class in the LDAP Component. This option is not documented   in the PHP Docs for the LDAP component but is required to set a timeout for certain server configurations. One use case for this option is if the LDAP server has a whitelist the client is not approved to access.

Commits
-------

2d88b89 [LDAP] Add TIMEOUT Option to LDAP Connection Options
@fabpot fabpot mentioned this pull request Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants