bazel: transport sockets: Update grpc to 1.19.1#6308
bazel: transport sockets: Update grpc to 1.19.1#6308vadorovsky wants to merge 1 commit intoenvoyproxy:masterfrom vadorovsky:update-grpc
Conversation
|
I'd suggest adding a TODO to the ALTS change indicating that ALTS should be updated for the function calls rather than adding nullptr - see #5461 |
Risk Level: Low Testing: Unit tests Docs Changes: N/A Release Notes: N/A Signed-off-by: Michal Rostecki <[email protected]>
|
Yep, please do fix ALTS properly for this. Thanks for the contribution @mrostecki. |
|
@htuch @moderation Thanks for pointing that out! I will fix ALTS properly. |
|
This pull request has been automatically marked as stale because it has not had activity in the last 7 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
stale
bot
added
the
stale
|
This pull request has been automatically closed because it has not had activity in the last 14 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
This continues envoyproxy#6620. See also envoyproxy#6308 and envoyproxy#5461. Risk level: Low Testing: Existing tests Signed-off-by: Harvey Tuch <[email protected]>
This continues #6620. See also #6308 and #5461. Risk level: Low Testing: Existing tests Signed-off-by: Harvey Tuch <[email protected]>
…xy#6773) This continues envoyproxy#6620. See also envoyproxy#6308 and envoyproxy#5461. Risk level: Low Testing: Existing tests Signed-off-by: Harvey Tuch <[email protected]> Signed-off-by: Jeff Piazza <[email protected]>
4 participants
Signed-off-by: Michal Rostecki [email protected]
Description: Update grpc to 1.19.1
Risk Level: Low
Testing: Unit tests
Docs Changes: N/A
Release Notes: N/A