Skip to content

Commit

Permalink
feat(security): add UserPermissionEvaluator
Browse files Browse the repository at this point in the history
so S3ArtifactStoreGetter can call a username-based hasPermission method.
FiatPermissionEvaluator has this method, but using FiatPermissionEvaluator in kork would
create a circular dependency.

The plan is:

- publish a kork jar with UserPermissionEvaluator,
- consume it in fiat and change FiatPermissionEvaluator to implement UserPermissionEvaluator instead of PermissionEvaluator
- publish a fiat jar and consume it everywhere
- change S3ArtifactStoreGetter to use UserPermissionEvaluator to fix the bug that #1178 demonstrates
- publish yet another kork jar and consume it everywhere to fix use of fetchReference in Evaluate Variables stages
  • Loading branch information
dbyron-sf committed Apr 26, 2024
1 parent d57eaf4 commit 791c9e8
Showing 1 changed file with 30 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
/*
* Copyright 2024 Salesforce, Inc.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package com.netflix.spinnaker.security;

import java.io.Serializable;
import org.springframework.security.access.PermissionEvaluator;

/**
* Make it possible to authorize by username in kork (e.g. in S3ArtifactStoreGetter), as
* FiatPermissionEvaluator currently does.
*/
public interface UserPermissionEvaluator extends PermissionEvaluator {

boolean hasPermission(
String username, Serializable resourceName, String resourceType, Object authorization);
}

0 comments on commit 791c9e8

Please sign in to comment.