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

encoding: Add support for identity encoding. #1660

Closed
wants to merge 1 commit into from
Closed

encoding: Add support for identity encoding. #1660

wants to merge 1 commit into from

Conversation

tsuna
Copy link
Contributor

@tsuna tsuna commented Nov 9, 2017

This fixes #1654

@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
The Linux Foundation CLA GitHub bot

@tsuna
Copy link
Contributor Author

tsuna commented Nov 9, 2017

I signed the Linux Foundation CLA. Also working to get a Corporate CLA in place between Arista and CNCF.

@dfawley
Copy link
Member

dfawley commented Nov 10, 2017

Thanks for the contribution! My fix for this is #1664. There were a number of minor problems and inconsistencies with the existing code in grpc for applying the compressor/decompressor; I believe I've ensured everything works correctly and consistently now.

@dfawley dfawley closed this Nov 10, 2017
@lock lock bot locked as resolved and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compressor not installed for grpc-encoding "identity"
3 participants