Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Side by Side Diff: appengine/logdog/coordinator/endpoints/services/service.go

Issue 1971493003: LogDog: Project READ access for user endpoints. (Closed) Base URL: https://github.com/luci/luci-go@logdog-project-service-config
Patch Set: Updated patchset dependency Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package services 5 package services
6 6
7 import ( 7 import (
8 "github.com/golang/protobuf/proto" 8 "github.com/golang/protobuf/proto"
9 "github.com/luci/luci-go/appengine/logdog/coordinator" 9 "github.com/luci/luci-go/appengine/logdog/coordinator"
10 "github.com/luci/luci-go/appengine/logdog/coordinator/endpoints" 10 "github.com/luci/luci-go/appengine/logdog/coordinator/endpoints"
(...skipping 23 matching lines...) Expand all
34 // Not a membership error. Something wen t wrong on the server's end. 34 // Not a membership error. Something wen t wrong on the server's end.
35 return nil, grpcutil.Internal 35 return nil, grpcutil.Internal
36 } 36 }
37 return nil, grpcutil.PermissionDenied 37 return nil, grpcutil.PermissionDenied
38 } 38 }
39 39
40 // Enter a datastore namespace based on the message type . 40 // Enter a datastore namespace based on the message type .
41 // 41 //
42 // We use a type switch here because this is a shared de corator. 42 // We use a type switch here because this is a shared de corator.
43 if pbm, ok := req.(endpoints.ProjectBoundMessage); ok { 43 if pbm, ok := req.(endpoints.ProjectBoundMessage); ok {
44 » » » » if err := coordinator.WithProjectNamespaceNoAuth (&c, config.ProjectName(pbm.GetMessageProject())); err != nil { 44 » » » » project := config.ProjectName(pbm.GetMessageProj ect())
45 » » » » log.Fields{
46 » » » » » "project": project,
47 » » » » }.Debugf(c, "Request is entering project namespa ce.")
48 » » » » if err := coordinator.WithProjectNamespaceNoAuth (&c, project); err != nil {
45 return nil, grpcutil.Internal 49 return nil, grpcutil.Internal
46 } 50 }
47 } 51 }
48 52
49 return c, nil 53 return c, nil
50 }, 54 },
51 } 55 }
52 } 56 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698