Exit out as early as possible if the auto-update script is run as the root user

Created on 20 March 2024, about 1 year ago
Updated 24 April 2024, about 1 year ago

Problem/Motivation

In πŸ› Warn if auto-update script is running as root Needs review we added a warning if the auto-update script is run as the root user. In 3.1.x we should not all this.

Steps to reproduce

Proposed resolution

In \Drupal\automatic_updates\Commands\AutomaticUpdatesCommandBase::runStatusChecks we should exit if the user is root. We should not do this in a validator because if the script is being run as we should exit out before we even boot drupal.

Remaining tasks

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Fixed

Version

3.1

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States tedbow Ithaca, NY, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @tedbow
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • Pipeline finished with Failed
    about 1 year ago
    Total: 650s
    #136530
  • Pipeline finished with Canceled
    about 1 year ago
    Total: 201s
    #142832
  • Pipeline finished with Canceled
    about 1 year ago
    #142842
  • Assigned to tedbow
  • Status changed to Needs review about 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • Pipeline finished with Canceled
    about 1 year ago
    Total: 967s
    #142862
  • Status changed to RTBC about 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States tedbow Ithaca, NY, USA

    Looks good if tests pass. Thanks @phenaproxima

  • Pipeline finished with Success
    about 1 year ago
    Total: 1421s
    #142875
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • Pipeline finished with Skipped
    about 1 year ago
    #142893
  • Issue was unassigned.
  • Status changed to Fixed about 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • Automatically closed - issue fixed for 2 weeks with no activity.

  • Pipeline finished with Success
    11 days ago
    Total: 820s
    #503743
  • Pipeline finished with Canceled
    10 days ago
    #504793
  • Pipeline finished with Failed
    10 days ago
    #504788
  • Pipeline finished with Failed
    10 days ago
    #504795
  • Pipeline finished with Running
    10 days ago
    #504938
  • Pipeline finished with Failed
    10 days ago
    #504926
  • Pipeline finished with Failed
    6 days ago
    Total: 720s
    #507594
  • Pipeline finished with Failed
    6 days ago
    Total: 665s
    #507643
  • Pipeline finished with Failed
    6 days ago
    Total: 839s
    #507793
  • Pipeline finished with Success
    6 days ago
    Total: 712s
    #507809
  • Pipeline finished with Skipped
    6 days ago
    #507863
Production build 0.71.5 2024