Skip to content

outlier: handle cross thread destroy#322

Merged
mattklein123 merged 1 commit intomasterfrom
outlier_destroy
Jan 5, 2017
Merged

outlier: handle cross thread destroy#322
mattklein123 merged 1 commit intomasterfrom
outlier_destroy

Conversation

@mattklein123
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 commented Jan 5, 2017

This is a little complicated, see the comment in the commit. Needed for
dynamic cluster removal.

This is a lot complicated, see the comment in the commit. Needed for
dynamic cluster removal.
@mattklein123
Copy link
Copy Markdown
Member Author

@lyft/network-team

@mattklein123 mattklein123 merged commit faa4968 into master Jan 5, 2017
@mattklein123 mattklein123 deleted the outlier_destroy branch January 5, 2017 18:16
rshriram pushed a commit to rshriram/envoy that referenced this pull request Oct 30, 2018
…roxy#322)

* Updated the latest commit id of cloudendpoints/service-control-client-cxx

* Use the original error description from the quota control API

* fixed code formatting

* Fixed test cases

* Add ConfigManager to ApiManager

* Revert "Add ConfigManager to ApiManager"

This reverts commit 26fb7f0eda5ba92e16adc5e0ca3a048e8880e142.
rshriram pushed a commit to rshriram/envoy that referenced this pull request Oct 30, 2018
Automatic merge from submit-queue.

[DO NOT MERGE] Auto PR to update dependencies of mixerclient

This PR will be merged automatically once checks are successful.
```release-note
none
```
yxue pushed a commit to yxue/envoy that referenced this pull request Dec 3, 2019
Signed-off-by: Yaroslav Skopets <y.skopets@gmail.com>
liverbirdkte referenced this pull request in liverbirdkte/envoy Oct 18, 2022
Fix issue of incorrectly copy cert subject to mimic cert. Use cert expiration time when expiration_time config is absent.

Signed-off-by: LeiZhang <lei.a.zhang@intel.com>
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Description: This PR inlines certificates for trusted certificate authorities into the config template and adds subject alt name verification to the base cluster. The certificate bundle is the same as that presented in MacOS 10.14.6
Risk Level: High
Testing: Simulator
Fixes #322

Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Description: This PR inlines certificates for trusted certificate authorities into the config template and adds subject alt name verification to the base cluster. The certificate bundle is the same as that presented in MacOS 10.14.6
Risk Level: High
Testing: Simulator
Fixes #322

Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
mathetake pushed a commit that referenced this pull request Mar 3, 2026
**Commit Message**

Use regex to check.


**Related Issues/PRs (if applicable)**


Fixes #267

---------

Signed-off-by: Loong <long0dai@foxmail.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Commit Message**

Previously, square brackets without being followed by () was also
rejected. This fixes the regex and allows it.

**Related Issues/PRs (if applicable)**

Follow up on #322

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.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.

2 participants