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
append app token to outgoing calls, if it is provided #7404
Conversation
Signed-off-by: Ryan Kelly <gatedlogic@live.com>
@vermillionsword see the minor linter error |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@vermillionsword please fix DCO. |
Signed-off-by: Ryan Kelly <gatedlogic@live.com>
Fixed |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7404 +/- ##
==========================================
- Coverage 62.39% 62.36% -0.04%
==========================================
Files 240 240
Lines 22156 22159 +3
==========================================
- Hits 13825 13820 -5
- Misses 7173 7179 +6
- Partials 1158 1160 +2 ☔ View full report in Codecov by Sentry. |
* append app token to outgoing calls, if it is provided Signed-off-by: Ryan Kelly <gatedlogic@live.com> * use declare and assign syntax Signed-off-by: Ryan Kelly <gatedlogic@live.com> --------- Signed-off-by: Ryan Kelly <gatedlogic@live.com> Co-authored-by: Ryan Kelly <gatedlogic@live.com>
* append app token to outgoing calls, if it is provided Signed-off-by: Ryan Kelly <gatedlogic@live.com> * use declare and assign syntax Signed-off-by: Ryan Kelly <gatedlogic@live.com> --------- Signed-off-by: Ryan Kelly <gatedlogic@live.com> Co-authored-by: Ryan Kelly <gatedlogic@live.com>
)" This reverts commit b1215d5.
…dapr#7404)"" This reverts commit 12406b9.
* append app token to outgoing calls, if it is provided Signed-off-by: Ryan Kelly <gatedlogic@live.com> * use declare and assign syntax Signed-off-by: Ryan Kelly <gatedlogic@live.com> --------- Signed-off-by: Ryan Kelly <gatedlogic@live.com> Co-authored-by: Ryan Kelly <gatedlogic@live.com> Signed-off-by: Elena Kolevska <elena@kolevska.com>
)" This reverts commit b1215d5. Signed-off-by: Elena Kolevska <elena@kolevska.com>
…dapr#7404)"" This reverts commit 12406b9. Signed-off-by: Elena Kolevska <elena@kolevska.com>
* append app token to outgoing calls, if it is provided Signed-off-by: Ryan Kelly <gatedlogic@live.com> * use declare and assign syntax Signed-off-by: Ryan Kelly <gatedlogic@live.com> --------- Signed-off-by: Ryan Kelly <gatedlogic@live.com> Co-authored-by: Ryan Kelly <gatedlogic@live.com>
Description
This PR appends the App API Token to calls made using the transparent proxy functionality provided in grpc_proxy.go
It will only be appended if the environment variable exists
Issue reference
#7344
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: