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

Content-type and Accept support for MIME-type content. #37

Conversation

bwehrle
Copy link
Contributor

@bwehrle bwehrle commented Jun 13, 2019

  • Content type negotiation via Accept header when returning content
  • Mappers decodes POST body based on Content-type header
  • Added new Handler functional type that allows returning Objects. These are deserialized to the most appropriate type for the client, or Json by default.
    (Squash of Feature/media type handling for resources (do not merge) #36)

…andler functional type that allows returning Objects that are deserialized to the most appropriate type for the client, or Json by default.
@bwehrle
Copy link
Contributor Author

bwehrle commented Jun 14, 2019

As an artifact of the monolithic build process, a break in vlingo-wire caused this build to break. :-(

[ERROR] /home/travis/build/vlingo/vlingo-wire/src/main/java/io/vlingo/wire/fdx/bidirectional/SecureClientRequestResponseChannel.java:[93,18] io.vlingo.wire.fdx.bidirectional.SecureClientRequestResponseChannel.LoggerAdapter is not abstract and does not override abstract method error(java.lang.String,java.lang.Object...) in io.vlingo.actors.Logger

@VaughnVernon
Copy link
Contributor

@bwehrle I pushed the fix in. The entire platform should build without errors.

@bwehrle
Copy link
Contributor Author

bwehrle commented Jun 14, 2019

@bwehrle I pushed the fix in. The entire platform should build without errors.

That was super fast

@VaughnVernon VaughnVernon merged commit 35622cd into vlingo:master Jun 14, 2019
@bwehrle
Copy link
Contributor Author

bwehrle commented Jun 14, 2019

Thanks, @VaughnVernon!

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