Skip to content
Snippets Groups Projects

Correct the scope of the GitHub personal access token

Merged Asmaa Hassan requested to merge asmaa.hassan-master-patch-63160 into master
All threads resolved!

What does this MR do?

Correcting the scope of the personal access token created on GitHub for configuring a pull mirror.

Screenshots or Screencasts (strongly suggested)

How to setup and validate locally (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Amy Qualls resolved all threads

    resolved all threads

  • Amy Qualls added 1 commit

    added 1 commit

    • 1960c5ab - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Amy Qualls added tw-weight3 label and removed tw-weight1 label

    added tw-weight3 label and removed tw-weight1 label

  • Amy Qualls
  • Amy Qualls resolved all threads

    resolved all threads

  • Amy Qualls added 1 commit

    added 1 commit

    • 97d2102f - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • merged

  • Amy Qualls mentioned in commit b597307d

    mentioned in commit b597307d

    • Resolved by Asmaa Hassan

      Excellent. @asmaa.hassan, I've applied a couple of changes to your work - line wraps, and a couple of little fixes. Thanks for your patience while Igor looked at your contribution. I had no reason to believe the work was wrong, but I couldn't personally verify it myself.

      All merged and done! Thanks, everyone.

  • Asmaa Hassan resolved all threads

    resolved all threads

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Please register or sign in to reply
    Loading