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

Unified Diff: logdog/appengine/coordinator/endpoints/registration/service.go

Issue 2626433004: Move "common/config" common types into cfgtypes. (Closed)
Patch Set: Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: logdog/appengine/coordinator/endpoints/registration/service.go
diff --git a/logdog/appengine/coordinator/endpoints/registration/service.go b/logdog/appengine/coordinator/endpoints/registration/service.go
index 83acb0a97b3a1173247357684612a4751cb52aff..a6e155a421750aabe3326050ddeec44f80db3939 100644
--- a/logdog/appengine/coordinator/endpoints/registration/service.go
+++ b/logdog/appengine/coordinator/endpoints/registration/service.go
@@ -6,12 +6,12 @@ package registration
import (
"github.com/golang/protobuf/proto"
- "github.com/luci/luci-go/common/config"
log "github.com/luci/luci-go/common/logging"
"github.com/luci/luci-go/grpc/grpcutil"
"github.com/luci/luci-go/logdog/api/endpoints/coordinator/registration/v1"
"github.com/luci/luci-go/logdog/appengine/coordinator"
"github.com/luci/luci-go/logdog/appengine/coordinator/endpoints"
+ "github.com/luci/luci-go/luci_config/common/cfgtypes"
"golang.org/x/net/context"
"google.golang.org/grpc/codes"
@@ -33,7 +33,7 @@ func New() logdog.RegistrationServer {
if ok {
// Enter the requested project namespace. This validates that the
// current user has READ access.
- project := config.ProjectName(pbm.GetMessageProject())
+ project := cfgtypes.ProjectName(pbm.GetMessageProject())
if project == "" {
return nil, grpcutil.Errf(codes.InvalidArgument, "project is required")
}

Powered by Google App Engine
This is Rietveld 408576698