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

Streaming interceptor #16

Merged
merged 8 commits into from
May 14, 2019
Merged

Streaming interceptor #16

merged 8 commits into from
May 14, 2019

Conversation

platinummonkey
Copy link
Owner

Expands upon #15 to implement both RecMsgv/SendMsg

cc @systay

systay and others added 4 commits April 5, 2019 10:42
Added grpc stream server interceptor
new option type for streaming implementations to add different limiters (or the same) to send/recv

wraps all streaming calls
@coveralls
Copy link

Pull Request Test Coverage Report for Build 111

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.277%

Totals Coverage Status
Change from base Build 104: 0.0%
Covered Lines: 1532
Relevant Lines: 1697

💛 - Coveralls

@coveralls
Copy link

coveralls commented May 10, 2019

Pull Request Test Coverage Report for Build 117

  • 2 of 4 (50.0%) changed or added relevant lines in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage remained the same at 90.277%

Changes Missing Coverage Covered Lines Changed/Added Lines %
limiter/default.go 2 4 50.0%
Files with Coverage Reduction New Missed Lines %
limiter/default.go 2 75.17%
Totals Coverage Status
Change from base Build 104: 0.0%
Covered Lines: 1532
Relevant Lines: 1697

💛 - Coveralls

@platinummonkey platinummonkey merged commit 43e12f0 into master May 14, 2019
@platinummonkey platinummonkey deleted the streaming-interceptor branch May 14, 2019 22:01
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.

None yet

3 participants