search

eclipse-vertx / vertx-http-proxy

vertx http proxy
Eclipse Public License 2.0
55 stars 36 forks source link

Unified head interception #104

Closed vietj closed 6 days ago

vietj commented 4 weeks ago

Motivation:

The new interceptors for changing request/response head each do a separate change yet they only modify the request/response head.

We should instead have a single interceptor that can implement all of them a batch the modification of the head in a single interceptor.

The builder pattern can be used to build such interceptor with the desired modification of the developer

Changes:

A new HeadInterceptor with a builder that aggregates the existing head individual interceptors.

tsegismont commented 1 week ago

@vietj I've updated PR, can you please take a look? Thank you

tsegismont commented 1 week ago

@vietj PTAL