Eli
10/29/2022, 11:29 AMRustam Siniukov
10/31/2022, 9:54 AMEli
10/31/2022, 10:06 AMRustam Siniukov
10/31/2022, 10:45 AMEli
10/31/2022, 11:46 AMif !call.isCommitted
, i.e - keep in mind always a very specific framework behavior.
2. access to finish() method is restricted, for example the “on() { call ->.. }” method inside Hook, does not give you option to get to the “finish” method
3. having more explicit/expressive code with “proceed()” and “finish()” for every plugin/hook, could result to more predictable and less error prone code. My opinion at least.
Thank you Rustam 🙂Rustam Siniukov
10/31/2022, 1:09 PMcall.respond
processing already handled call is safe. Also, if you really need it, it’s easy to write your own wrappers that will check it.
2. Hook
has custom generic parameter and you can pass PipelineContext
as a parameter which has finish()
/`proceed()` methods
3. In complex cases I may agree with you. But majority of plugins and developers, creating this plugins do not need it. So with current approach we provide simplest behaviour by default, while allowing more complex as wellEli
10/31/2022, 2:14 PM