Skip to content

Remove TLS_PERMISSIVE mode in authn policy.#518

Merged
istio-testing merged 1 commit intoistio:masterfrom
incfly:api-clean
Jun 8, 2018
Merged

Remove TLS_PERMISSIVE mode in authn policy.#518
istio-testing merged 1 commit intoistio:masterfrom
incfly:api-clean

Conversation

@incfly
Copy link
Copy Markdown

@incfly incfly commented Jun 8, 2018

This is not used by any customer which offers us opportunities to delete it now, and then update this into istio/istio:master.

istio/istio#5059

@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Jun 8, 2018
@incfly
Copy link
Copy Markdown
Author

incfly commented Jun 8, 2018

@diemtvu

@incfly
Copy link
Copy Markdown
Author

incfly commented Jun 8, 2018

@diemtvu Could you /lgtm & /approve as well? Thanks

@diemtvu
Copy link
Copy Markdown
Contributor

diemtvu commented Jun 8, 2018

/lgtm
/approve

@istio-testing
Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: diemtvu, incfly

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@istio-testing istio-testing merged commit 2d2fd0b into istio:master Jun 8, 2018
incfly pushed a commit to incfly/api that referenced this pull request Jun 13, 2018
incfly pushed a commit to incfly/proxy that referenced this pull request Jun 18, 2018
istio-testing pushed a commit to istio/proxy that referenced this pull request Jun 18, 2018
* Update the api sha, istio/api#518.

* Remove TLS_PERMISSIVE usage.
incfly pushed a commit to incfly/proxy that referenced this pull request Jun 20, 2018
* Update the api sha, istio/api#518.

* Remove TLS_PERMISSIVE usage.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants