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

CogRPC: Consider using binary metadata for CCA instead of text-based Authorization metadata #61

Open
gnarea opened this issue Apr 27, 2020 · 0 comments
Labels
enhancement New feature or request spec-cogrpc

Comments

@gnarea
Copy link
Member

gnarea commented Apr 27, 2020

The CCA is currently specified in the Authorization header/metadata because it belongs there from a semantic perspective, but we're having to serialise the CCA in base64 so we can use it in that text-based metadata.

By contrast, if we take a more pragmatic, performance-focused view, we should be using a binary metadata value, which would in turn force us to use a custom field.

This is a minor consideration in the large scheme of things but I thought it'd still be good to track it somewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request spec-cogrpc
Projects
None yet
Development

No branches or pull requests

1 participant