Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Access revoked for a number of team members #250

Closed
bjpirt opened this issue Sep 6, 2021 · 11 comments
Closed

Access revoked for a number of team members #250

bjpirt opened this issue Sep 6, 2021 · 11 comments

Comments

@bjpirt
Copy link
Contributor

bjpirt commented Sep 6, 2021

Hello,
Today a number of our team were unable to access the repositories specified in the Terraform - please could you check if there have been any issues running this script recently?

@AntonyBishop
Copy link
Contributor

Hi @bjpirt please can you advise which users and which repos?

We recently ran a script to tidy up users. This included a large volume of users who had invites to repos but had never accepted them. In those instances the users who not have access in any event due to the expiry of the invites.

If you can advise names and repos I can check.

@bjpirt
Copy link
Contributor Author

bjpirt commented Sep 6, 2021

It looks like this is what's happened - a bunch of our users have been removed from the files.

cf95e41

I'll make a PR to reinstate them

@bjpirt
Copy link
Contributor Author

bjpirt commented Sep 6, 2021

I wonder if the 'admin' permission confused the script as both myself and @brettminnie were removed from all repos but we both definitely had full admin access to them before we were removed

@AntonyBishop
Copy link
Contributor

It's very possible. We'll take a look. Thanks for the PR.

@AntonyBishop
Copy link
Contributor

Will keep open while investigation is ongoing.

@bjpirt
Copy link
Contributor Author

bjpirt commented Sep 6, 2021

FYI - I've got access again (thanks) and as an admin I wasn't asked to accept an invite. I have full access to the repos now

@AntonyBishop
Copy link
Contributor

Hi @bjpirt. Great news.

Out of interest (and possibly why this issue occured), why as a member of the MoJ Org are you being added to these repos as an external collaborator?

@bjpirt
Copy link
Contributor Author

bjpirt commented Sep 6, 2021

I understood that all users with access to the repo had to go int he collaborators repo - is that not the case?

@AntonyBishop
Copy link
Contributor

Hi @bjpirt we have recently refreshed the guidance here.

As a member of MadeTech it is appropriate for you to be an external collaborator.

But the question for me is why have you been made a member of the MoJ Org?

@bjpirt
Copy link
Contributor Author

bjpirt commented Sep 7, 2021

I do have an MoJ email address, and have added it to my account so that I can access the Modernisation Platform as it uses GitHub for auth. However, I agree it is more appropriate for me to be an external collaborator. The same applies to @brettminnie

@AntonyBishop
Copy link
Contributor

That explains it. We have some similar users for the Cloud Platform. So if I make the accounts just collaborators it'll cut off your Modernisation Platform access.

That leave us in a halfway house and is something we'll need to think about in our clean-up process. Lets keep things as they are for now. We'll just need to think up how we manage clean-up.

Many thanks. I'll close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants