Skip to content

Report StopIteration if connection is closed (#2270)#2278

Merged
duderino merged 2 commits intoistio:release-1.2from
yxue:release-1.2
Jun 14, 2019
Merged

Report StopIteration if connection is closed (#2270)#2278
duderino merged 2 commits intoistio:release-1.2from
yxue:release-1.2

Conversation

@yxue
Copy link
Copy Markdown
Member

@yxue yxue commented Jun 14, 2019

  • Report StopIteration if connection is closed

  • Change ENVOY_LOG to ENVOY_CONN_LOG

What this PR does / why we need it:

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

Special notes for your reviewer:

Release note:

* Report StopIteration if connection is closed

* Change ENVOY_LOG to ENVOY_CONN_LOG
@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 14, 2019
@istio-testing
Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: crazyxy

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

@duderino
Copy link
Copy Markdown

@crazyxy you need to run make lint

Source file(s) not formatted:
src/envoy/tcp/mixer/filter.cc

@utka utka removed their request for review June 14, 2019 18:32
@duderino
Copy link
Copy Markdown

@fpesce I'm exceeding my mandate as 1.1 release manager and merging this for you so we can kick off that brutal 1.5 hour build of the proxy image.

Please revert if you have any objections

@duderino duderino merged commit 7767d3a into istio:release-1.2 Jun 14, 2019
@yxue yxue deleted the release-1.2 branch October 11, 2019 00:05
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