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

Side by Side Diff: scheduler/appengine/acl/acl.go

Issue 2976623002: scheduler: Move IsJobOwner to acl package. (Closed)
Patch Set: missing Created 3 years, 5 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
« no previous file with comments | « no previous file | scheduler/appengine/apiservers/scheduler.go » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The LUCI Authors. 1 // Copyright 2017 The LUCI Authors.
2 // 2 //
3 // Licensed under the Apache License, Version 2.0 (the "License"); 3 // Licensed under the Apache License, Version 2.0 (the "License");
4 // you may not use this file except in compliance with the License. 4 // you may not use this file except in compliance with the License.
5 // You may obtain a copy of the License at 5 // You may obtain a copy of the License at
6 // 6 //
7 // http://www.apache.org/licenses/LICENSE-2.0 7 // http://www.apache.org/licenses/LICENSE-2.0
8 // 8 //
9 // Unless required by applicable law or agreed to in writing, software 9 // Unless required by applicable law or agreed to in writing, software
10 // distributed under the License is distributed on an "AS IS" BASIS, 10 // distributed under the License is distributed on an "AS IS" BASIS,
11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12 // See the License for the specific language governing permissions and 12 // See the License for the specific language governing permissions and
13 // limitations under the License. 13 // limitations under the License.
14 14
15 package presentation 15 package acl
16 16
17 import ( 17 import (
18 "golang.org/x/net/context" 18 "golang.org/x/net/context"
19 19
20 "github.com/luci/luci-go/server/auth" 20 "github.com/luci/luci-go/server/auth"
21 ) 21 )
22 22
23 // TODO(tandrii): move this to new acl package.
24 func IsJobOwner(c context.Context, projectID, jobName string) bool { 23 func IsJobOwner(c context.Context, projectID, jobName string) bool {
25 // TODO(vadimsh): Do real ACLs. 24 // TODO(vadimsh): Do real ACLs.
26 ok, err := auth.IsMember(c, "administrators") 25 ok, err := auth.IsMember(c, "administrators")
27 if err != nil { 26 if err != nil {
28 panic(err) 27 panic(err)
29 } 28 }
30 return ok 29 return ok
31 } 30 }
OLDNEW
« no previous file with comments | « no previous file | scheduler/appengine/apiservers/scheduler.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698