Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Also mention the net.devh:grpc-spring-boot-starter library #1213

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

Merged
marcingrzejszczak merged 2 commits into spring-cloud:master from ST-DDT:more_grpc_libraries
Feb 19, 2019
Merged

Also mention the net.devh:grpc-spring-boot-starter library #1213

marcingrzejszczak merged 2 commits into spring-cloud:master from ST-DDT:more_grpc_libraries
Feb 19, 2019

Conversation

@ST-DDT
Copy link
Contributor

@ST-DDT ST-DDT commented Feb 16, 2019

Currently you mention only one library for grpc spring-boot support, but there are currently two main libraries.

LogNet/grpc-spring-boot-starter only supports the server side, whereas yidongnan/grpc-spring-boot-starter supports the server and client side, as well as Spring-Security and other features.

Please also mention the second library.

Disclosure: I'm one of the core maintainers of yidongnan/grpc-spring-boot-starter.

Copy link

@ST-DDT Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

Copy link

@ST-DDT Thank you for signing the Contributor License Agreement!

Copy link
Member

Which does sleuth integrate with in this repository?

Copy link
Contributor Author

ST-DDT commented Feb 16, 2019

This library contains support classes for the lognet library. See TraceGrpcAutoConfiguration
and some other support classes that are needed because they are not present in that library.

yidongnan/grpc-spring-boot-starter supports this library out of the box: See TraceClientAutoConfiguration . No further support classes are needed. If you want, we could move the sleuth related client + server configuration here as well.

Copy link
Contributor Author

ST-DDT commented Feb 16, 2019
edited
Loading

Copy link
Member

We don't really document external projects. We may link to it, but not document it

Copy link
Contributor Author

ST-DDT commented Feb 17, 2019
edited
Loading

Changed the PR to only contain the link.

PS: Why is the build failing? I only changed the docs.

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

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

2.1.1.RELEASE

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /