Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

Conversation

rumpl
Copy link
Contributor

@rumpl rumpl commented Jun 15, 2020

What I did

  • split the interceptors code into own file
  • interceptors (unary and stream) search for the current context in the request metadata, if it's not found they will load the current context from the configuration file

Related issue

Fixes #213

@rumpl rumpl requested review from gtardif and chris-crone June 15, 2020 21:24
server/server.go Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit : supporting with an s

Copy link
Contributor

@gtardif gtardif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a typo in comment

@rumpl rumpl force-pushed the feat-context-metadata branch from 20b30b1 to a999f34 Compare June 16, 2020 07:43
@rumpl rumpl merged commit 50c68ce into docker-archive:master Jun 16, 2020
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.

Bring back context in the metadata
2 participants