Skip to content

No reply from proxy if interceptor releases request resources and returns a failed future#142

Merged
tsegismont merged 1 commit intoeclipse-vertx:mainfrom
tsegismont:issue/141
Feb 2, 2026
Merged

No reply from proxy if interceptor releases request resources and returns a failed future#142
tsegismont merged 1 commit intoeclipse-vertx:mainfrom
tsegismont:issue/141

Conversation

@tsegismont
Copy link
Member

See #141

This happens because ProxiedRequest#release is not protected from repeated invocations.

…urns a failed future

See eclipse-vertx#141

This happens because ProxiedRequest#release is not protected from repeated invocations.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
@tsegismont tsegismont merged commit d4b32bb into eclipse-vertx:main Feb 2, 2026
5 checks passed
@tsegismont tsegismont added this to the 5.1.0 milestone Feb 2, 2026
@tsegismont tsegismont deleted the issue/141 branch February 2, 2026 15:58
tsegismont added a commit to tsegismont/vertx-http-proxy that referenced this pull request Feb 2, 2026
…urns a failed future (eclipse-vertx#142)

See eclipse-vertx#141

This happens because ProxiedRequest#release is not protected from repeated invocations.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
tsegismont added a commit to tsegismont/vertx-http-proxy that referenced this pull request Feb 2, 2026
…urns a failed future (eclipse-vertx#142)

See eclipse-vertx#141

This happens because ProxiedRequest#release is not protected from repeated invocations.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
tsegismont added a commit that referenced this pull request Feb 2, 2026
…urns a failed future (#142) (#143)

See #141

This happens because ProxiedRequest#release is not protected from repeated invocations.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
tsegismont added a commit that referenced this pull request Feb 2, 2026
…urns a failed future (#142) (#144)

See #141

This happens because ProxiedRequest#release is not protected from repeated invocations.

Signed-off-by: Thomas Segismont <tsegismont@gmail.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.

1 participant