Skip to content

[Spring] Add a generator for spring cloud feign clients#3294

Merged
wing328 merged 6 commits intoswagger-api:masterfrom
cbornet:spring_cloud
Jul 7, 2016
Merged

[Spring] Add a generator for spring cloud feign clients#3294
wing328 merged 6 commits intoswagger-api:masterfrom
cbornet:spring_cloud

Conversation

@cbornet
Copy link
Copy Markdown
Contributor

@cbornet cbornet commented Jul 5, 2016

First version of a generator for spring-cloud-feign clients in a spring-boot application.
The generator will generate auto-configured beans for authentication (with full OAuth2 support using spring-security/spring-cloud-security).

Some things will be added later:

  • README with explanation on how to use and customize (especially how to use Ribbon/Eureka/Hystrix, ...)
  • Petstore sample tests

@wing328 wing328 added this to the v2.2.0 milestone Jul 5, 2016
@cbornet
Copy link
Copy Markdown
Contributor Author

cbornet commented Jul 5, 2016

@wing328 I see some issues while writing the tests so please wait before merging.

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 5, 2016

@cbornet please take your time. I will wait for update from you.

@cbornet
Copy link
Copy Markdown
Contributor Author

cbornet commented Jul 6, 2016

Tests are now passing except the file upload because it's not supported by spring-cloud-feign (See spring-cloud/spring-cloud-netflix#1164) so it is ignored for now.
@wing328 can be merged now ! I'll do the README later.

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 6, 2016

@cbornet thanks!! I'll merge shortly (after the python test enhancement PR)

@cbornet cbornet closed this Jul 6, 2016
@cbornet cbornet reopened this Jul 6, 2016
@cbornet cbornet closed this Jul 6, 2016
@cbornet cbornet reopened this Jul 6, 2016
@wing328 wing328 merged commit 9830e82 into swagger-api:master Jul 7, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 7, 2016

@cbornet thanks for the contribution to add the Spring Cloud Feign API client.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants