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

Removed special casing of HTTP headers and request options #58

Closed
wants to merge 4 commits into from

Conversation

matthewgertner
Copy link

Not suggesting this is ready to be merged yet, but I'm interested in your opinion if you have a chance to take a look. Am I on the right track? For now I removed the broken test since it doesn't really make sense anymore, but of course more tests should be added. Also I should probably add a with-all-pages macro so the user doesn't need to modify *all-pages* directly.

@Raynes
Copy link
Owner

Raynes commented Dec 19, 2014

I'm not following this pull request. We're deleting a bunch of code. How does that work? What are we doing?

@matthewgertner
Copy link
Author

The idea is to use the middleware capabilities in newer versions of clj-http rather than special casing HTTP headers. I also added a Tentacles middleware function for all-pages so that doesn't need special-casing anymore either. You can see an example of using clj-http middleware in #57 (comment). IMO this is more elegant but I'm interested to know what you think.

@Raynes
Copy link
Owner

Raynes commented Jan 11, 2016

Closing because it has been a year and I don't think a ton of value would be added by trying to resolve the merge conflicts. Sorry, I've been out of the clj game for quite a while and am just catching up.

@Raynes Raynes closed this Jan 11, 2016
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.

None yet

2 participants