services/identity: Temporary logging to help diagnose vanadium/browser#84
Change-Id: Iff32c214a6075d027c05e366cfc98349aef8b7c0
diff --git a/services/identity/internal/blesser/oauth.go b/services/identity/internal/blesser/oauth.go
index 079967e..daa4b3f 100644
--- a/services/identity/internal/blesser/oauth.go
+++ b/services/identity/internal/blesser/oauth.go
@@ -9,6 +9,8 @@
"strings"
"time"
+ "v.io/x/lib/vlog"
+
"v.io/x/ref/services/identity"
"v.io/x/ref/services/identity/internal/oauth"
"v.io/x/ref/services/identity/internal/revocation"
@@ -59,6 +61,10 @@
}
func (b *oauthBlesser) BlessUsingAccessToken(ctx *context.T, call rpc.ServerCall, accessToken string) (security.Blessings, string, error) {
+ // Temporary logging to help debug https://github.com/vanadium/browser/issues/84
+ // TODO(ashankar): Remove before release!
+ vlog.Infof("START: BlessUsingAccessToken(PublicKey:%v, Endpoint:%v)", call.Security().RemoteBlessings().PublicKey(), call.RemoteEndpoint())
+ defer vlog.Infof("END : BlessUsingAccessToken(PublicKey:%v, Endpoint:%v)", call.Security().RemoteBlessings().PublicKey(), call.RemoteEndpoint())
var noblessings security.Blessings
email, clientName, err := b.oauthProvider.GetEmailAndClientName(accessToken, b.accessTokenClients)
if err != nil {
diff --git a/services/identity/internal/blesser/util_test.go b/services/identity/internal/blesser/util_test.go
index 0700e14..08f05fe 100644
--- a/services/identity/internal/blesser/util_test.go
+++ b/services/identity/internal/blesser/util_test.go
@@ -6,6 +6,7 @@
import (
"v.io/v23/context"
+ "v.io/v23/naming"
"v.io/v23/rpc"
"v.io/v23/security"
)
@@ -15,7 +16,9 @@
secCall security.Call
}
-func (c rpcCall) Security() security.Call { return c.secCall }
+func (c rpcCall) Security() security.Call { return c.secCall }
+func (c rpcCall) LocalEndpoint() naming.Endpoint { return nil }
+func (c rpcCall) RemoteEndpoint() naming.Endpoint { return nil }
func fakeContextAndCall(provider, user security.Principal) (*context.T, rpc.ServerCall) {
ctx, _ := context.RootContext()
