Skip to content

network: add socket ip version accessor#11514

Merged
ggreenway merged 1 commit into
envoyproxy:masterfrom
florincoras:sock_addr
Jun 11, 2020
Merged

network: add socket ip version accessor#11514
ggreenway merged 1 commit into
envoyproxy:masterfrom
florincoras:sock_addr

Conversation

@florincoras
Copy link
Copy Markdown
Member

Signed-off-by: Florin Coras fcoras@cisco.com

Risk Level: Medium
Testing: unit tests
Docs Changes: n/a
Release Notes: n/a

@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to api/.
CC @envoyproxy/api-watchers: FYI only for changes made to api/.

🐱

Caused by: #11514 was opened by florincoras.

see: more, trace.

@florincoras
Copy link
Copy Markdown
Member Author

@mattklein123 just a heads up, this PR tries to solve, amongst others, this issue with retrieving a socket's IP version we've discussed some weeks back.

Signed-off-by: Florin Coras <fcoras@cisco.com>
@yanavlasov yanavlasov self-assigned this Jun 9, 2020
@htuch htuch removed the api label Jun 9, 2020
@ggreenway ggreenway merged commit e34d12f into envoyproxy:master Jun 11, 2020
@florincoras florincoras deleted the sock_addr branch June 11, 2020 17:37
arthuryan-k pushed a commit to arthuryan-k/envoy that referenced this pull request Jun 15, 2020
Signed-off-by: Florin Coras <fcoras@cisco.com>
Signed-off-by: Arthur Yan <arthuryan@google.com>
yashwant121 pushed a commit to yashwant121/envoy that referenced this pull request Jun 24, 2020
Signed-off-by: Florin Coras <fcoras@cisco.com>
Signed-off-by: yashwant121 <yadavyashwant36@gmail.com>
songhu pushed a commit to songhu/envoy that referenced this pull request Jun 25, 2020
Signed-off-by: Florin Coras <fcoras@cisco.com>
yashwant121 pushed a commit to yashwant121/envoy that referenced this pull request Jul 24, 2020
Signed-off-by: Florin Coras <fcoras@cisco.com>
Signed-off-by: yashwant121 <yadavyashwant36@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants