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

Add support for RSockets with Spring #349

Merged
merged 2 commits into from
Mar 5, 2021
Merged

Add support for RSockets with Spring #349

merged 2 commits into from
Mar 5, 2021

Conversation

dsyer
Copy link
Contributor

@dsyer dsyer commented Feb 25, 2021

Also generically can support structured events with any Spring
API that works with Encoder and Decoder. There's a sample for
the RSocket case with a simple request-response echo server.

Also generically can support structured events with any Spring
API that works with Encoder and Decoder. There's a sample for
the RSocket case with a simple request-response echo server.

Signed-off-by: Dave Syer <dsyer@vmware.com>
Copy link
Member

@slinkydeveloper slinkydeveloper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add some doc about these 2 new encoder/decoder here? https://github.com/cloudevents/sdk-java/blob/master/docs/spring.md

Can you also check all the copyrights and have them like this one? https://github.com/cloudevents/sdk-java/blob/master/core/src/main/java/io/cloudevents/core/CloudEventUtils.java#L1

Signed-off-by: Dave Syer <dsyer@vmware.com>
@dsyer
Copy link
Contributor Author

dsyer commented Mar 2, 2021

I changed the copyrights to match the example (but also noticed that the other Spring module source code do not precisely match BTW). I also added a link and a mention to the docs - there isn't much detail there for any of the converters, but I'd be more than happy to add some notes independently of this PR.

@slinkydeveloper
Copy link
Member

but also noticed that the other Spring module source code do not precisely match BTW

It would be nice if you can do another PR to fix them 😄

I also added a link and a mention to the docs - there isn't much detail there for any of the converters, but I'd be more than happy to add some notes independently of this PR.

Sure, these docs are really drafted and i'll be more than happy to see them growing!

@slinkydeveloper slinkydeveloper added the enhancement New feature or request label Mar 5, 2021
@slinkydeveloper slinkydeveloper added this to the 2.1 milestone Mar 5, 2021
@slinkydeveloper slinkydeveloper merged commit 8b38273 into cloudevents:master Mar 5, 2021
@dsyer dsyer deleted the rsockets branch March 5, 2021 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants