line/armeria

Derived context doesn't call parent context hooks

yzfeng2020 opened this issue · 0 comments

The current implementation for creating a new derived context doesn't seem to keep track of the hooks of parent context, so that the parent context hooks won't be called when ctx.push is called on the derived context.

public static ClientRequestContext newDerivedContext(ClientRequestContext ctx,

As suggested by @trustin, we could fix this by making sure a derived context keeps the reference to the parent context's hook list as well as its own hook list, so that both of them are invoked when the derived context needs to call hooks.