Skip to content

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

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

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

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

@yxue
Copy link
Copy Markdown
Member Author

yxue commented Jun 14, 2019

@crazyxy you need to run make lint

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

I don't know why merging to master branch doesn't have format issue. I sent another pr to master branch and will cherry pick that pr to this after merged.

@duderino duderino merged commit ae671c5 into istio:release-1.1 Jun 14, 2019
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