Skip to content

test: fix thread local mock shutdown#1367

Merged
mattklein123 merged 1 commit intomasterfrom
fix_tls_mock
Aug 1, 2017
Merged

test: fix thread local mock shutdown#1367
mattklein123 merged 1 commit intomasterfrom
fix_tls_mock

Conversation

@mattklein123
Copy link
Copy Markdown
Member

No description provided.

@mattklein123
Copy link
Copy Markdown
Member Author

@jamessynge @htuch @dnoe

@jamessynge
Copy link
Copy Markdown
Contributor

Thanks, LGTM.

@mattklein123 mattklein123 merged commit 5ad6390 into master Aug 1, 2017
@mattklein123 mattklein123 deleted the fix_tls_mock branch August 1, 2017 17:56
rshriram pushed a commit to rshriram/envoy that referenced this pull request Oct 30, 2018
)

Automatic merge from submit-queue.

Add the support of the allow_tls option for Istio authn

**What this PR does / why we need it**: This PR adds the support of the allow_tls option for Istio authn.
When allow_tls is true, the client certificate does not have to be present. If it is present, it will be extracted after the authentication.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes envoyproxy#1365 

**Special notes for your reviewer**:

**Release note**:

```release-note
```
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Signed-off-by: Alan Chiu <achiu@lyft.com>

For an explanation of how to fill out the fields, please see the relevant section
in [PULL_REQUESTS.md](https://github.com/envoyproxy/envoy/blob/master/PULL_REQUESTS.md)

Description: android: update engine to take context
Risk Level: low
Testing: ci
Docs Changes: n/a
Release Notes: n/a
[Optional Fixes #Issue]
[Optional Deprecated:]

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Signed-off-by: Alan Chiu <achiu@lyft.com>

For an explanation of how to fill out the fields, please see the relevant section
in [PULL_REQUESTS.md](https://github.com/envoyproxy/envoy/blob/master/PULL_REQUESTS.md)

Description: android: update engine to take context
Risk Level: low
Testing: ci
Docs Changes: n/a
Release Notes: n/a
[Optional Fixes #Issue]
[Optional Deprecated:]

Signed-off-by: JP Simard <jp@jpsim.com>
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.

3 participants