Skip to content

Merge 0.19.0 of net ldap #6

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 190 commits into from
Jul 8, 2024
Merged

Merge 0.19.0 of net ldap #6

merged 190 commits into from
Jul 8, 2024

Conversation

JasonHerr
Copy link

@JasonHerr JasonHerr commented Jul 5, 2024

Merge in latest changes of ruby-net-ldap gem into our fork.

  • test with ldapr rspecs
  • test on VM

jch and others added 30 commits January 11, 2016 21:42
Base64.encode64 adds \n every 60 encoded chars. This was originally an encoding mechanism for sending binary
content in e-mail, where the line length is limited. For passwords we dont want this.
cf https://stackoverflow.com/questions/2620975/strange-n-in-base64-encoded-string-in-ruby
instead of polluting devs machine, let's rely on a docker container
to spin up the service

relies on osixia/openldap:1.3.0. Customizes a few things:
- adds the seed on bootstrap
- does not enforce client certificate
- sets a hostname to avoid domain verification issues during handshake

The cert domain is also added to /etc/hosts
anonymous access is not enabled in this setup, so every test needs
to perform authentication first
I couldn't manage to get the container running with a cert
issued to a given IP, like 127.0.0.1 or localhost.
Instead, I specified a static hostname
(the container uses hostname to generate the cert)
and injected it in travis.

Unfortunately, in local development this means changing /etc/hosts,
but I feel that's a better option that having to install LDAP locally
so that all retcode tests succeed
it's a little price to pay in the current setup and allows us to
have the same tests locally and dev.
all environments run the same set of tests, no env specific test
so that we are able to run a test that does not specific CACERT
and so the library fallsback to system cert store
gwillcox-r7 and others added 20 commits December 7, 2022 08:58
…g user controls even if the paged and ber_sort flags weren't set
…_values

Net::LDAP::DN - Retain trailing spaces in RDN values in DNs
…-flexibility

Add in ability for users to specify LDAP controls when conducting searches
Document `connect_timeout` in Constructor Details
Fix openssl error when using multiple hosts
@JasonHerr JasonHerr changed the title [WIP] Merge 0.19.0 of net ldap Merge 0.19.0 of net ldap Jul 5, 2024
@JasonHerr JasonHerr merged commit 637edf4 into main Jul 8, 2024
1 of 9 checks passed
@JasonHerr JasonHerr deleted the merge_0.19.0_of_net-ldap branch July 8, 2024 17:32
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.