Following up on #2821365: Clients cannot create sites because of missing database options, I felt is was rather strange that a client was able to view an error message. Since aegir offers an out-of-the-box experience (it works directly after installing), people might forget about switching error reporting off.

I think either hosting or hostmaster should set the error_level variable to ERROR_REPORT_HIDE during hook_install(). Probably requiring an update hook to set it for existing sites.

CommentFileSizeAuthor
#2 hosting_or_hostmaster-2821366-2.patch723 bytesNeograph734
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Neograph734 created an issue. See original summary.

Neograph734’s picture

Neograph734
he/him
Location Netherlands
CreditAttribution: Neograph734 commented
Issue summary: View changes
FileSize
hosting_or_hostmaster-2821366-2.patch723 bytes
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented

Thanks for the suggestion.

It's ERROR_REPORTING_HIDE though with an extra 'ING'

I've committed it to my review branch, see https://github.com/aegir-project/hosting/pull/8

  • helmo committed e1d663d on 7.x-3.x authored by Neograph734 
    Issue #2821366 by Neograph734: Hosting or hostmaster should suppress...
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
Status: Needs review » Fixed

merged, thanks.

Status: Fixed » Closed (fixed)

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