... otherwise it can't change the permissions on the sites directory.

So that needs to be documented in the docs or something.

(I need to word this better, but I don't want't to forget it and i do not have time)

Comments

adrian’s picture

adrian commented
Version: » 5.x-0.1-alpha1

I added a check for this into provision_path.

It will flag this as an error when new sites try to be created.

I can integrate this into the provision form too.

adrian’s picture

adrian commented

still need to add validation to the submission form, and info in help.

but verify now checks if the user is in the right group.

http://drupal.org/cvs?commit=120930

adrian’s picture

adrian commented
Status: Active » Needs work
adrian’s picture

adrian commented
Status: Needs work » Fixed

fixed in latest commit

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

  • Commit 76e90b7 on debian, dev-dns, dev-envobject, dev-koumbit, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newhooks, dev-nginx, dev-platform_management, dev-ports, dev-purgebackup, dev-restore, dev-services, dev-simplerinstaller, dev-site_rename, dev-ssl, dev_716166_apache_conf, dev_dns, dev_server_verify, prod-koumbit, ssl, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x authored by adrian: 
    Practically completely rewrote the provision_check_path function.

It is...
  • Commit 4f7794b on debian, dev-dns, dev-envobject, dev-koumbit, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newhooks, dev-nginx, dev-platform_management, dev-ports, dev-purgebackup, dev-restore, dev-services, dev-simplerinstaller, dev-site_rename, dev-ssl, dev_716166_apache_conf, dev_dns, dev_server_verify, prod-koumbit, ssl, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x authored by adrian:
    #245664 - add validation to check that user is in the correct group...