Skip to content

fix error request id in the dubbo local reply#18741

Merged
lizan merged 1 commit intoenvoyproxy:mainfrom
wbpcode:dubbo-request-id
Oct 27, 2021
Merged

fix error request id in the dubbo local reply#18741
lizan merged 1 commit intoenvoyproxy:mainfrom
wbpcode:dubbo-request-id

Conversation

@wbpcode
Copy link
Member

@wbpcode wbpcode commented Oct 25, 2021

Signed-off-by: wbpcode comems@msn.com

Commit Message: fix error request id in the dubbo local reply
Additional Description: fix #18623
Risk Level: Low.
Testing: Added.
Docs Changes: N/A.
Release Notes: N/A.
Platform Specific Features: N/A.

Signed-off-by: wbpcode <comems@msn.com>
@wbpcode wbpcode requested a review from lizan as a code owner October 25, 2021 12:11
@wbpcode wbpcode changed the title fix error request id in the dubbbo local reply fix error request id in the dubbo local reply Oct 25, 2021
@lizan lizan merged commit 3e9655e into envoyproxy:main Oct 27, 2021
mpuncel added a commit to mpuncel/envoy that referenced this pull request Oct 29, 2021
* main: (221 commits)
  deps: Bump `protobuf` -> 3.19.0 (envoyproxy#18471)
  tooling: auto-assign dependency shephards (envoyproxy#18794)
  clang-tidy: Return from diff fun if empty diff (envoyproxy#18815)
  repokitteh: Block PRs pending deps approval (envoyproxy#18814)
  deps: Bump `org_llvm_llvm` -> 12.0.1, `com_github_wavm_wavm` -> 9ffd3e2 (envoyproxy#18747)
  dns resolvers: add All lookup mode (envoyproxy#18464)
  doc: fix link formatting for TLS session_timeout (envoyproxy#18790)
  ext_authz: Set response flag and code details to UAEX when denied (envoyproxy#18740)
  socket options: add support for directly creating ipv4/ipv6 pairs (envoyproxy#18769)
  ecds: make onConfigUpdate generic over filter type (envoyproxy#18061)
  bazel: update CMake instructions in EXTERNAL_DEPS.md (envoyproxy#18799)
  upstream: fix typo in comment (envoyproxy#18798)
  runtime: removing envoy.reloadable_features.grpc_json_transcoder_adhere_to_buffer_limits (envoyproxy#18696)
  bazel: Add CC=clang to clang configuration (envoyproxy#18732)
  fix error request id in the dubbbo local reply (envoyproxy#18741)
  event: assert the case of both read and closed event registered (envoyproxy#18265)
  tcp proxy connect tunneling: improved testing (envoyproxy#18784)
  deps: Bump `protoc-gen-validate` -> 0.6.2 (envoyproxy#18742)
  deps: Bump `rules_pkg` -> ad57589 (envoyproxy#18746)
  bazel: copy .bazelversion for envoy filter examples (envoyproxy#18730)
  ...

Signed-off-by: Michael Puncel <mpuncel@squareup.com>
barroca added a commit to barroca/envoy that referenced this pull request Aug 28, 2025
Remove outdated decodingBuffer() checks that prevented stream
recreation for requests with bodies. The underlying infrastructure
now supports internal redirects with request bodies.

Fixes envoyproxy#18741
barroca added a commit to barroca/envoy that referenced this pull request Sep 2, 2025
Remove outdated decodingBuffer() checks that prevented stream
recreation for requests with bodies. The underlying infrastructure
now supports internal redirects with request bodies.

Fixes envoyproxy#18741

Signed-off-by: Leonardo da Mata <ldamata@spotify.com>
barroca added a commit to barroca/envoy that referenced this pull request Sep 9, 2025
Remove outdated decodingBuffer() checks that prevented stream
recreation for requests with bodies. The underlying infrastructure
now supports internal redirects with request bodies.

Fixes envoyproxy#18741

Signed-off-by: Leonardo da Mata <ldamata@spotify.com>
barroca added a commit to barroca/envoy that referenced this pull request Sep 9, 2025
Remove outdated decodingBuffer() checks that prevented stream
recreation for requests with bodies. The underlying infrastructure
now supports internal redirects with request bodies.

Fixes envoyproxy#18741

Signed-off-by: Leonardo da Mata <ldamata@spotify.com>
barroca added a commit to barroca/envoy that referenced this pull request Sep 15, 2025
Remove outdated decodingBuffer() checks that prevented stream
recreation for requests with bodies. The underlying infrastructure
now supports internal redirects with request bodies.

Fixes envoyproxy#18741

Signed-off-by: Leonardo da Mata <ldamata@spotify.com>
barroca added a commit to barroca/envoy that referenced this pull request Oct 3, 2025
Remove outdated decodingBuffer() checks that prevented stream
recreation for requests with bodies. The underlying infrastructure
now supports internal redirects with request bodies.

Fixes envoyproxy#18741

Signed-off-by: Leonardo da Mata <ldamata@spotify.com>
barroca added a commit to barroca/envoy that referenced this pull request Oct 3, 2025
Remove outdated decodingBuffer() checks that prevented stream
recreation for requests with bodies. The underlying infrastructure
now supports internal redirects with request bodies.

Fixes envoyproxy#18741

Signed-off-by: Leonardo da Mata <ldamata@spotify.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.

dubbo: wrong request id is set in local reply

2 participants