Skip to content

Loglevel debug in http codec_client on PrematureResponseException#6778

Merged
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
sumukhs:issue/6636
May 2, 2019
Merged

Loglevel debug in http codec_client on PrematureResponseException#6778
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
sumukhs:issue/6636

Conversation

@sumukhs
Copy link
Contributor

@sumukhs sumukhs commented May 2, 2019

Signed-off-by: Sumukh Shivaprakash sumukhs@microsoft.com
Description: change log level to debug on prematureresponseexception
Risk Level: Low
Testing: //test/...
Docs Changes: n/a
Release Notes: n/a
Fixes #6636

Signed-off-by: Sumukh Shivaprakash <sumukhs@microsoft.com>
@zuercher zuercher self-assigned this May 2, 2019
Copy link
Member

@zuercher zuercher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@mattklein123 mattklein123 merged commit 281fa1a into envoyproxy:master May 2, 2019
jeffpiazza-google pushed a commit to jeffpiazza-google/envoy that referenced this pull request May 3, 2019
Signed-off-by: Sumukh Shivaprakash <sumukhs@microsoft.com>
Signed-off-by: Jeff Piazza <jeffpiazza@google.com>
@sumukhs sumukhs deleted the issue/6636 branch May 3, 2019 19:05
oschaaf added a commit to oschaaf/envoy that referenced this pull request May 9, 2019
Description:
envoyproxy#6778 fixed
envoyproxy#6636

For consistency, it may be good to handle the other
case as well. So this PR can be considered as a proposed
addendum to that.

Risk Level: Low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Otto van der Schaaf <oschaaf@we-amp.com>
mattklein123 pushed a commit that referenced this pull request May 10, 2019
Description:
#6778 fixed
#6636

For consistency, it may be good to handle the other
case as well. So this PR can be considered as a proposed
addendum to that.

Risk Level: Low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Otto van der Schaaf <oschaaf@we-amp.com>
mpuncel added a commit to mpuncel/envoy that referenced this pull request May 10, 2019
* master: (88 commits)
  upstream: Null-deref on TCP health checker if setsockopt fails  (envoyproxy#6793)
  ci: switch macOS CI to azure pipelines (envoyproxy#6889)
  os syscalls lib: break apart syscalls used for hot restart (envoyproxy#6880)
  Kafka codec: precompute request size before serialization, so we do n… (envoyproxy#6862)
  upstream: move static and strict_dns clusters to dedicated files (envoyproxy#6886)
  Rollforward of api: Add total_issued_requests to Upstream Locality and Endpoint Stats. (envoyproxy#6692) (envoyproxy#6784)
  fix explicit constructor in copy-initialization (envoyproxy#6884)
  stats: use tag iterator rather than constructing the tag-array and searching that. (envoyproxy#6853)
  common: use unscoped build target in generate_version_linkstamp (envoyproxy#6877)
  Addendum to envoyproxy#6778 (envoyproxy#6882)
  ci: add minimum Linux build for Azure Pipelines (envoyproxy#6881)
  grpc: utilities for inter-converting grpc::ByteBuffer and Buffer::Instance. (envoyproxy#6732)
  upstream: allow excluding hosts from lb calculations until initial health check (envoyproxy#6794)
  stats: prevent unused counters from leaking across hot restart (envoyproxy#6850)
  network filters: add `injectDataToFilterChain(data, end_stream)` method to network filter callbacks (envoyproxy#6750)
  delete things that snuck back in (envoyproxy#6873)
  config: scoped rds (2b): support delta APIs in ConfigProvider framework (envoyproxy#6781)
  string == string! (envoyproxy#6868)
  config: add mssing imports to delta_subscription_state (envoyproxy#6869)
  protobuf: add missing default case to enum (envoyproxy#6870)
  ...

Signed-off-by: Michael Puncel <mpuncel@squareup.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.

Log level of "premature response" messages

3 participants