summaryrefslogtreecommitdiffstats
path: root/fnet
diff options
context:
space:
mode:
authorTor Brede Vekterli <vekterli@yahooinc.com>2022-06-29 13:14:40 +0000
committerTor Brede Vekterli <vekterli@yahooinc.com>2022-06-29 13:14:40 +0000
commit44722c9ed8f45fe5ced186fc22476d35b3e3f3da (patch)
tree36af7ee8ed1e01688eb574b5576f5f3a650ebea3 /fnet
parentcc44b799f0d78a5e26f12ecb8b868301095570c4 (diff)
Clarify why we're setting the return handler prior to filter invocation
Diffstat (limited to 'fnet')
-rw-r--r--fnet/src/vespa/fnet/frt/invoker.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/fnet/src/vespa/fnet/frt/invoker.cpp b/fnet/src/vespa/fnet/frt/invoker.cpp
index f75526d51f1..d4b35720a8d 100644
--- a/fnet/src/vespa/fnet/frt/invoker.cpp
+++ b/fnet/src/vespa/fnet/frt/invoker.cpp
@@ -52,7 +52,7 @@ FRT_RPCInvoker::FRT_RPCInvoker(FRT_Supervisor *supervisor,
std::string methodName(_req->GetMethodName(), _req->GetMethodNameLen());
LOG(debug, "invoke(server) init: '%s'", methodName.c_str());
}
- req->SetReturnHandler(this); // Must be set prior to any access filter being invoked
+ req->SetReturnHandler(this); // RPC req -> FNET_Connection link is via this ptr; set prior to access filter invocation.
if (_method == nullptr) {
if (!req->IsError()) { // may be BAD_REQUEST
req->SetError(FRTE_RPC_NO_SUCH_METHOD);