Skip to content
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

Client certificate support in clj-apache-http #7

Open
wants to merge 51 commits into
base: master
Choose a base branch
from

Conversation

diamondap
Copy link

Hello,

I forked the clj-apache-http project because I needed to add support for HTTPS connections that use x509 client certificates.

This code is working. If you want to pull it back in to clj-apache-http, please do. Since my experience with clojure is limited, you may want to take some time to review the code. One issue that I know exists is that you have to run lein compile on NaiveTrustManager.clj and PermissiveHostnameVerifier.clj before you can use them. That's not really standard practice.

Anyway, if this is helpful, please feel free to add it to your project.

Thanks.

Andrew Diamond
Hotelicopter.com

A. Diamond and others added 30 commits May 10, 2011 17:07
A. Diamond and others added 21 commits June 24, 2011 16:35
the SSL context was initialized for the async client.
…. Callbacks only deal with the CountdownLatch if it exists.
…ure 1.3 use.

Changes:
- Replaced use of clojure.contrib.json to clojure.data.json in new data.json module
- Replaced use of clojure.contrib.string/chomp with clojure.string/trim-newline
- Replaced use of clojure.contrib.io/slurp* with clojure.core/slurp
- Replaced use of clojure.contrib.io/reader wtih clojure.java.io/reader
- Replaced use of clojure.contrib.java-utils/file with clojure.java.io/file
- Replaced use of clojure.contrib.base64 with Base64 in Apache commons-codec (already used transitively)
Remove dependency on clojure-contrib to make this friendlier for Clojure 1.3 use.
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.

2 participants