Skip to content

SafeSettings removes existing team permission for all the repos #350

Open

Description

@sajithvasu

Github Safesettings app removes the existing teams permissions for all existing repositories within the org when the following mentioned settings are enabled. On the other hand, documentation does not specify removal of any existing permission when controlled through safesettings app. Is this the expected behavior?

settings.yml

repository:
teams:
  - name: <team_name>
    permission: <team_permission>

Our goal is to add specific permissions/teams to all the existing/new repos without impacting existing team permission for all the repos.

Activity

  1. martinm82 commented on Dec 8, 2022

    @martinm82
    Contributor

    I have discovered this issue as well and @decyjphr confirmed this here. It would be indeed good to update the existing settings instead of overriding them fully.

  2. decyjphr commented on Dec 8, 2022

    @decyjphr
    Collaborator

    This fix has been released in 2.0.18

  3. martinm82 commented on Jan 24, 2023

    @martinm82
    Contributor

    @decyjphr should we close this issue? Also worth to add a reference to the pull request solving this problem.

  4. anderssonjohan commented on Jan 25, 2023

    @anderssonjohan
    Contributor

    Isn't this contradicting to #251? In my situation, I want to remove all manually added collaborators because it should only be a single source of truth when using this for policy-as-code. Related to fixes in #382

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions