Skip to content

Add Transit JSON/MessagePack support #215

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Aug 11, 2014
Merged

Conversation

r0man
Copy link
Contributor

@r0man r0man commented Aug 8, 2014

This PR adds Transit JSON/MessagePack support. :form-params encoding
is triggered by setting the :content-type to either :transit+json or
:transit+msgpack. Response body decoding is triggered by setting :as
to :transit+json, :transit+msgpack or just :auto.

The wrap-form-params middleware has been refactored to call the
coerce-form-params multi method to enable pluggable form encoding in a
similar fashion as we have done with coerce-response-body.

This PR adds Transit JSON/MessagePack support. `:form-params` encoding
is triggered by setting the `:content-type` to either `:transit+json` or
`:transit+msgpack`. Response body decoding is triggered by setting `:as`
to `:transit+json`, `:transit+msgpack` or just `:auto`.

The `wrap-form-params` middleware has been refactored to call the
`coerce-form-params` multi method to enable pluggable form encoding in a
similar fashion as we have done with `coerce-response-body`.
@r0man r0man mentioned this pull request Aug 8, 2014
@dakrone dakrone merged commit f300229 into dakrone:master Aug 11, 2014
@dakrone
Copy link
Owner

dakrone commented Aug 11, 2014

This look great, I've merged this and I'm going to do another release.

@r0man
Copy link
Contributor Author

r0man commented Aug 11, 2014

Cool. Thanks!

On Mon, Aug 11, 2014 at 1:51 PM, Lee Hinman notifications@github.com
wrote:

This look great, I've merged this and I'm going to do another release.


Reply to this email directly or view it on GitHub
#215 (comment).

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants