veyron2/ipc: 4 of n. 'Invoker and Signature' rationalisation.
- first CL to remove use of ReflectInvoker from our app code.
Change-Id: Ia2ef1ae14f248fa16c4f798d8fc68136c0e1d017
diff --git a/services/mgmt/node/impl/dispatcher.go b/services/mgmt/node/impl/dispatcher.go
index 679a459..3ebc87c 100644
--- a/services/mgmt/node/impl/dispatcher.go
+++ b/services/mgmt/node/impl/dispatcher.go
@@ -350,7 +350,7 @@
disp: d,
uat: d.uat,
})
- return ipc.ReflectInvoker(receiver), d.auth, nil
+ return receiver, d.auth, nil
case appsSuffix:
// Requests to apps/*/*/*/logs are handled locally by LogFileInvoker.
// Requests to apps/*/*/*/pprof are proxied to the apps' __debug/pprof object.
@@ -365,7 +365,7 @@
case "logs":
logsDir := filepath.Join(appInstanceDir, "logs")
suffix := naming.Join(components[5:]...)
- return logsimpl.NewLogFileInvoker(logsDir, suffix), d.auth, nil
+ return logsimpl.NewLogFileServer(logsDir, suffix), d.auth, nil
case "pprof", "stats":
info, err := loadInstanceInfo(appInstanceDir)
if err != nil {
@@ -404,7 +404,7 @@
if err != nil {
return nil, nil, err
}
- return ipc.ReflectInvoker(receiver), appSpecificAuthorizer, nil
+ return receiver, appSpecificAuthorizer, nil
case configSuffix:
if len(components) != 2 {
return nil, nil, errInvalidSuffix
@@ -420,7 +420,7 @@
// TODO(caprita,rjkroege): We should further refine this, by
// only allowing the app to update state referring to itself
// (and not other apps).
- return ipc.ReflectInvoker(receiver), nil, nil
+ return receiver, nil, nil
default:
return nil, nil, errInvalidSuffix
}
