-
Notifications
You must be signed in to change notification settings - Fork 254
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
Fix CI build #333
Fix CI build #333
Commits on Nov 14, 2019
-
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
Configuration menu - View commit details
-
Copy full SHA for c3f0e4c - Browse repository at this point
Copy the full SHA c3f0e4cView commit details -
Configuration menu - View commit details
-
Copy full SHA for fdbe61e - Browse repository at this point
Copy the full SHA fdbe61eView commit details -
Configuration menu - View commit details
-
Copy full SHA for ddd8dce - Browse repository at this point
Copy the full SHA ddd8dceView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3caf85b - Browse repository at this point
Copy the full SHA 3caf85bView commit details -
change test DSN and authenticate with the right user
anonymous access is not enabled in this setup, so every test needs to perform authentication first
Configuration menu - View commit details
-
Copy full SHA for 683d660 - Browse repository at this point
Copy the full SHA 683d660View commit details -
adjust hostname to the new certificate for TLS tests
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
Configuration menu - View commit details
-
Copy full SHA for fb4c76d - Browse repository at this point
Copy the full SHA fb4c76dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 94c2ba9 - Browse repository at this point
Copy the full SHA 94c2ba9View commit details -
adds custom retcode.ldif to the bootstrap sequence
so that all retcode tests succeed
Configuration menu - View commit details
-
Copy full SHA for 4280d18 - Browse repository at this point
Copy the full SHA 4280d18View commit details -
Configuration menu - View commit details
-
Copy full SHA for f38c39c - Browse repository at this point
Copy the full SHA f38c39cView commit details -
let's assume folks will have to modify /etc/hosts
it's a little price to pay in the current setup and allows us to have the same tests locally and dev.
Configuration menu - View commit details
-
Copy full SHA for 1b595b2 - Browse repository at this point
Copy the full SHA 1b595b2View commit details -
we don't need vagrant specific tests anymore
all environments run the same set of tests, no env specific test
Configuration menu - View commit details
-
Copy full SHA for ba225a9 - Browse repository at this point
Copy the full SHA ba225a9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 376c42e - Browse repository at this point
Copy the full SHA 376c42eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 839318a - Browse repository at this point
Copy the full SHA 839318aView commit details -
installs docker container CA certificate
so that we are able to run a test that does not specific CACERT and so the library fallsback to system cert store
Configuration menu - View commit details
-
Copy full SHA for 9b8d80d - Browse repository at this point
Copy the full SHA 9b8d80dView commit details -
let's disable the test that uses system cert store
I'm not sure how to enable this in Travis
Configuration menu - View commit details
-
Copy full SHA for 7c85643 - Browse repository at this point
Copy the full SHA 7c85643View commit details -
with the dockerized test openldap server, none of this is needed
Configuration menu - View commit details
-
Copy full SHA for 6c1c56d - Browse repository at this point
Copy the full SHA 6c1c56dView commit details -
Configuration menu - View commit details
-
Copy full SHA for f67e741 - Browse repository at this point
Copy the full SHA f67e741View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5c4644e - Browse repository at this point
Copy the full SHA 5c4644eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 10a44d5 - Browse repository at this point
Copy the full SHA 10a44d5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8ba87b3 - Browse repository at this point
Copy the full SHA 8ba87b3View commit details -
clarify /etc/hosts caveat, needed for local integration tests
the container uses HOSTNAME to generate the cert, and it really didn't like "localhost" as hostname. As a workaround, I had to add an arbitrary hostname. There may be other alternatives to get the host to be known, but the most obvious is modifying /etc/hosts
Configuration menu - View commit details
-
Copy full SHA for 201fdfa - Browse repository at this point
Copy the full SHA 201fdfaView commit details -
Configuration menu - View commit details
-
Copy full SHA for dc6d75d - Browse repository at this point
Copy the full SHA dc6d75dView commit details -
Configuration menu - View commit details
-
Copy full SHA for fee04d8 - Browse repository at this point
Copy the full SHA fee04d8View commit details -
Configuration menu - View commit details
-
Copy full SHA for fef7cdb - Browse repository at this point
Copy the full SHA fef7cdbView commit details -
Configuration menu - View commit details
-
Copy full SHA for cf4e9bd - Browse repository at this point
Copy the full SHA cf4e9bdView commit details
Commits on Nov 15, 2019
-
Co-Authored-By: Matt Todd <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 35bcc92 - Browse repository at this point
Copy the full SHA 35bcc92View commit details -
Configuration menu - View commit details
-
Copy full SHA for b90ee7e - Browse repository at this point
Copy the full SHA b90ee7eView commit details -
Configuration menu - View commit details
-
Copy full SHA for a9228e2 - Browse repository at this point
Copy the full SHA a9228e2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 14dfc2a - Browse repository at this point
Copy the full SHA 14dfc2aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 693b210 - Browse repository at this point
Copy the full SHA 693b210View commit details