Restful WS Spring boot example

[slinkydeveloper]

Part of #125

Signed-off-by: Francesco Guardiani francescoguard@gmail.com

[slinkydeveloper]

Depends on #289

[dsyer]

Some changes suggested inline.

[dsyer]

You don't need this (Spring Boot provides it automatically).

[slinkydeveloper]

Can I remove JerseyConfiguration too?

[dsyer]

I think you need to register the CloudEventsProvider?

[slinkydeveloper]

The CloudEventsProvider should be registered through the restful ws SPI: https://github.com/cloudevents/sdk-java/tree/master/http/restful-ws/src/main/resources/META-INF/services

[dsyer]

I see. Then I guess we still need it to register the MainResource at least? It's more common in a Spring Boot app to register the instance, not the class, but in this case it doesn't matter.

[slinkydeveloper]

TBH I'm not at all expert of conventions in Spring Boot, so I can't tell which solution is best 😄 ... Should I change the registration with the instances?

[dsyer]

I don't think it matters actually. This sample does it with class literals: https://github.com/spring-projects/spring-boot/blob/master/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-jersey/src/main/java/smoketest/jersey/JerseyConfig.java

[salaboy]

@slinkydeveloper @dsyer this is good! and I think that is basically doing what I need as well.

[dsyer]

@salaboy good to hear. Your PR was using Spring @RestController though, and I think we should have some support for that as well eventually. @olegz is working on a PR that has a @RestController BTW.

[salaboy]

@dsyer that's cool.. I think that if webflux is supported I am more than ok with it.. I've created the @RestController stuff to create a PR to this repo, because I was aware that not everyone was into Webflux still. I am happy to see that supported in a much cleaner way.. I believe that this is a big enabler for people to use CE.