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

Unified Diff: logdog/appengine/coordinator/endpoints/logs/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/logs/service.go
diff --git a/logdog/appengine/coordinator/endpoints/logs/service.go b/logdog/appengine/coordinator/endpoints/logs/service.go
index 8b0d647cafbef4310a3cc96023c0cb398ed4d1b7..b5972b969b39074a96fe4b66503a83045296bb7e 100644
--- a/logdog/appengine/coordinator/endpoints/logs/service.go
+++ b/logdog/appengine/coordinator/endpoints/logs/service.go
@@ -6,12 +6,12 @@ package logs
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/logs/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"
@@ -43,7 +43,7 @@ func newService(svr *server) logdog.LogsServer {
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