Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
CI_Source CI_Source
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Labels
    • Labels
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Metrics
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Members
    • Members
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • Support
  • CI_SourceCI_Source
  • Merge requests
  • !32

Merged
Created Feb 21, 2023 by Ben Avison@bavisonDeveloper

Add a job to check usernames in commit logs

  • Overview 4
  • Commits 1
  • Pipelines 2
  • Changes 1

For auditing purposes, ROOL's policy is to require real names in commit logs (both author and committer), however there has been at least one example of a pseudonym slipping through the review net. GitLab doesn't make this any easier, because when displaying users in merge request pages, it looks up email addresses against those assigned to GitLab users, and substitutes the GitLab username instead. Only after the MR is accepted do you get to see the commits in pages where GitLab shows the underlying usernames from the git metadata, by which time it's too late to do anything about it!

This new CI job is an attempt to catch at least some future instances automatically. It iterates over the commits in an MR, and tests for one-word usernames (with the exception of "ROOL"), causing a job failure if any are found. All usernames are printed in full in the job log for human inspection.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: check-mr-users