Title: Allow uri and base to be taken from system/user LDAP configuration by sambrightman · Pull Request #164 · python-ldap/python-ldap · GitHub
Open Graph Title: Allow uri and base to be taken from system/user LDAP configuration by sambrightman · Pull Request #164 · python-ldap/python-ldap
X Title: Allow uri and base to be taken from system/user LDAP configuration by sambrightman · Pull Request #164 · python-ldap/python-ldap
Description: The aim is to reduce the number of parameters that must be passed to various functions to the minimal set when LDAP is correctly configured in the environment. In my case, ldap.conf contains at least the URI and base, and is deployed externally to relevant machines. Duplicating the configuration in my code would be both inconvenient and more fragile. I believe this is the case in many common setups. I think the ideal situation would actually be to re-order some of the method parameters. People using defaults from the system configuraiton probably want to only specify e.g. the filter. Putting it first (with uri/base towards the end), would allow removing the keyword in calls. However, that is a minor inconvenience versus breaking backward compatibility. This is a prototype pull request - in particular: I'm not familiar with python-ldap (or really with LDAP in general). Docs, Demos, Tests have not been reviewed for potential updates (I have the impression that testing could be achieved via environment variables and would be willing to add tests). I did not know how to treat urlfetch and am not 100% that I have caught all locations that could be defaulted.
Open Graph Description: The aim is to reduce the number of parameters that must be passed to various functions to the minimal set when LDAP is correctly configured in the environment. In my case, ldap.conf contains at lea...
X Description: The aim is to reduce the number of parameters that must be passed to various functions to the minimal set when LDAP is correctly configured in the environment. In my case, ldap.conf contains at lea...
Opengraph URL: https://github.com/python-ldap/python-ldap/pull/164
X: @github
Domain: patch-diff.githubusercontent.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:42242ec0-cf56-6077-e3ed-cef6e0b89f1e |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | AE66:2CD505:3ACDB36:5136CCD:6977DA21 |
| html-safe-nonce | 9573d16cefdc8fefcdd8f2ac6ef4734292f35a87a755ec93f512e856e4fcffea |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBRTY2OjJDRDUwNTozQUNEQjM2OjUxMzZDQ0Q6Njk3N0RBMjEiLCJ2aXNpdG9yX2lkIjoiMjQzNjQxNDg0ODYxOTc2NDI1NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | eb977cd9283b582a73986a8c267b7314f132fb45b4b4aa6bd75c12b055ecc318 |
| hovercard-subject-tag | pull_request:162363273 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python-ldap/python-ldap/pull/164/files |
| twitter:image | https://avatars.githubusercontent.com/u/536652?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/536652?s=400&v=4 |
| og:image:alt | The aim is to reduce the number of parameters that must be passed to various functions to the minimal set when LDAP is correctly configured in the environment. In my case, ldap.conf contains at lea... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 8909f95e78dc1ccc1b9acf9d5c963c2515579e48d2898151cf571437facd1d7b |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python-ldap/python-ldap git https://github.com/python-ldap/python-ldap.git |
| octolytics-dimension-user_id | 33895877 |
| octolytics-dimension-user_login | python-ldap |
| octolytics-dimension-repository_id | 111794776 |
| octolytics-dimension-repository_nwo | python-ldap/python-ldap |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 111794776 |
| octolytics-dimension-repository_network_root_nwo | python-ldap/python-ldap |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | d6b13f6e4712dc1242cf12b5e75dec8b1bf3a9ab |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width