Problem/Motivation

Right now you could set up the auto-update script to run as server cron job run as root. You should not run this as root

Steps to reproduce

Proposed resolution

We should stop auto-update script from running as script but right now we should just warn otherwise we could stop a site from receiving an update if they were relying on current server cron job

Create a RootUserValidator(or something) that warns if being run as root.

Remaining tasks

In a follow-up in 3.1.x we should stop auto-update script

User interface changes

API changes

Data model changes

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

tedbow created an issue. See original summary.

tedbow’s picture

Assigned: Unassigned » phenaproxima
Status: Active » Needs review
phenaproxima’s picture

Status: Needs review » Reviewed & tested by the community

  • tedbow committed 73009dfd on 3.0.x
    Issue #3432476: Warn if auto-update script is running as root
    
tedbow’s picture

Status: Reviewed & tested by the community » Fixed
tedbow’s picture

Assigned: phenaproxima » tedbow
Status: Fixed » Active

@phenaproxima pointed out we didn't make this a service 🤦🏼

  • tedbow committed e08e7ddf on 3.0.x
    Issue #3432476 by tedbow: Warn if auto-update script is running as root
    
tedbow’s picture

Status: Active » Fixed

This doesn't need to go into 3.1.x because we not allow running as root then

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.