Skip to content

Build envoy as dependency#17

Merged
lizan merged 3 commits intoistio:masterfrom
lizan:envoy_workspace
Dec 20, 2016
Merged

Build envoy as dependency#17
lizan merged 3 commits intoistio:masterfrom
lizan:envoy_workspace

Conversation

@lizan
Copy link
Copy Markdown
Contributor

@lizan lizan commented Dec 20, 2016

No description provided.

@lizan lizan self-assigned this Dec 20, 2016
@lizan lizan requested a review from chowchow316 December 20, 2016 21:34
)


def googletest_repositories(bind=True):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why we move this repository here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Envoy uses googletest for test too.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So this file contains the common dependencies of ESP and envoy?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

right

)


def protobuf_repositories(bind=True):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We have one protobuf_repositories in contrib/endpoints/repositories.bzl, can we merge them?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

forgot to save that file...

@lizan lizan merged commit 1d67c4c into istio:master Dec 20, 2016
@lizan lizan deleted the envoy_workspace branch December 21, 2016 00:21
lizan added a commit to cloudendpoints/esp that referenced this pull request Dec 21, 2016
qiwzhang added a commit that referenced this pull request Feb 16, 2018
* Add grpc transport implementation.

* Move GrpcStream to cc file.

* Use grpc as default transport.

* add -lrt to some tests.
howardjohn pushed a commit that referenced this pull request Mar 3, 2020
Previously, SNI verifier didn't support TLSv1.3 and clients configured
to use only TLSv1.3 were not recognized as TLS clients.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
duderino pushed a commit to duderino/proxy that referenced this pull request Mar 17, 2020
)

Previously, SNI verifier didn't support TLSv1.3 and clients configured
to use only TLSv1.3 were not recognized as TLS clients.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
duderino pushed a commit to duderino/proxy that referenced this pull request Mar 17, 2020
)

Previously, SNI verifier didn't support TLSv1.3 and clients configured
to use only TLSv1.3 were not recognized as TLS clients.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
duderino pushed a commit to duderino/proxy that referenced this pull request Mar 17, 2020
)

Previously, SNI verifier didn't support TLSv1.3 and clients configured
to use only TLSv1.3 were not recognized as TLS clients.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
istio-testing pushed a commit that referenced this pull request Mar 17, 2020
…2773)

Previously, SNI verifier didn't support TLSv1.3 and clients configured
to use only TLSv1.3 were not recognized as TLS clients.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>

Co-authored-by: Piotr Sikora <piotrsikora@google.com>
kyessenov pushed a commit to kyessenov/proxy that referenced this pull request Aug 8, 2022
reintroduce fork for orig dest port override
jwendell pushed a commit to jwendell/proxy that referenced this pull request Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants