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

feat: Retrieve request metadata from the server #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chimano
Copy link

@chimano chimano commented Mar 31, 2023

Following the same implementation as grpc-go: https://github.com/grpc/grpc-go/blob/master/Documentation/grpc-metadata.md#sending-and-receiving-metadata---server-side

Clients have the ability to add metadata to requests, but the servers cannot access it. This change allows servers to access the request's metadata from the RPC context. I believe this also resolves #42.

* Add Request Metadata

* Fix Misc Issues

* Start Sending Metadata

* Add Method to get metadata

* Change Way of Getting Metadata

* Change Way of Getting Metadata

* Small Refactor

* Fixed bugs and failing test cases

* Removed unneeded variable in test

---------

Co-authored-by: Matt James <[email protected]>
Co-authored-by: chimano <[email protected]>
@shkolnik
Copy link
Contributor

shkolnik commented Apr 6, 2023

@chimano I'll take a look at this tomorrow!

@chimano
Copy link
Author

chimano commented Apr 17, 2023

@shkolnik Do you have some time to review it this week?

@stevenharman
Copy link

👋 Hello! Thank you for all that you've put into creating and maintaining this Gem. If there's any feedback you have for improving this change, we'd love to hear it and move this one forward toward merge.

@chimano chimano closed this Aug 16, 2023
@chimano chimano reopened this Aug 16, 2023
@vh
Copy link

vh commented Jan 27, 2024

@shkolnik Could you please? Still actual.

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.

Handling custom headers on the server
5 participants