Skip to content
This repository was archived by the owner on Dec 3, 2019. It is now read-only.

Add SSL support. #172

Merged
merged 1 commit into from
Mar 18, 2016
Merged

Add SSL support. #172

merged 1 commit into from
Mar 18, 2016

Conversation

alexdupre
Copy link
Contributor

SSL is disabled by default to avoid POLA violations.
It is possible to enable and control SSL behavior via url parameters:

  • sslmode=<mode> enable ssl (prefer/require/verify-ca/verify-full [recommended])
  • sslrootcert=<path.pem> specifies trusted certificates (JDK cacert if missing)

Client certificate authentication is not implemented, due to lack of
time and interest, but it should be easy to add.

SSL is disabled by default to avoid POLA violations.
It is possible to enable and control SSL behavior via url parameters:
- `sslmode=<mode>` enable ssl (prefer/require/verify-ca/verify-full [recommended])
- `sslrootcert=<path.pem>` specifies trusted certificates (JDK cacert if missing)

Client certificate authentication is not implemented, due to lack of
time and interest, but it should be easy to add.
@alexdupre alexdupre mentioned this pull request Mar 7, 2016
@alexdupre
Copy link
Contributor Author

There are a few tests (not related to SSL) that are not deterministic and randomly fail.
After fighting with Travis I was able to add a full SSL test suite and it passed in all environments.

mauricio added a commit that referenced this pull request Mar 18, 2016
@mauricio mauricio merged commit 1570906 into mauricio:master Mar 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants