Apache - Reload instead of Restart

[Ballistic509]

Is it possible to have ispCP use the reload command for apache instead of restart or is this unsafe? If It can be done, where can I edit the script or command that controls this?

Thanks..

[joximu]

engine/ispcp-serv-mngr
sub restart_httpd {... }

should work, but restart is safer :-) (but takes more time in big installations)

[raphael]

Open a ticket

[Ballistic509]

So just to double check this, If I made the following change would this be correct? lol

Original

Code:

sub restart_httpd {

    my ($cmd, $rs, $rdata) = (undef, undef, undef);

    push_el(\@main::el, 'restart_httpd()', 'Starting...');

    $cmd = $main::cfg{'CMD_HTTPD'};

    sys_command_rs("$cmd stop");

    sleep(5);

    sys_command_rs("$cmd start");

    sleep(5);

    push_el(\@main::el, 'restart_httpd()', 'Ending...');

}

Change

Code:

sub restart_httpd {

    my ($cmd, $rs, $rdata) = (undef, undef, undef);

    push_el(\@main::el, 'restart_httpd()', 'Starting...');

    $cmd = $main::cfg{'CMD_HTTPD'};

    sys_command_rs("$cmd reload");

    sleep(5);

    push_el(\@main::el, 'restart_httpd()', 'Ending...');

}

raphael Wrote:Open a ticket

I figured there was really no error, or bug in this so I thought I would ask here.

[joximu]

well, try it out - should work...

maybe "reload" is not _that_ official in all *nix...

[Ballistic509]

"reload" works on Debian lol..I guess if it don't work I'll get an error or something somewhere lol

[joximu]

Sure it works on Debian but I don't know i it works on any BSD - maybe that's why the developer took "stop" and "start"....
I was just thinking if it's a good idea to change it in the trunk...

[Ballistic509]

True. It's needed for a script on my server that I run. If Apache is restarted then I get errors so for my personal use I can change it if needed, as can other users.

[Ballistic509]

All appears to be working. No errors found in logs when adding sub-domain and I'm assuming that this is when apache would run that piece of code to update the virtual hosts.

[raphael]

An other way is: apache2 graceful