> - Should the file not be simply called 'ssl.conf' 

Can be changed.

> instead of 'httpd-ssl.conf'

Named "httpd-ssl.conf" just to be consistent with the Apache conf naming 
convention.


> - Instead of line like
> 
> #ServerName 127.0.0.1:443 in 'httpd-ssl.conf' , 
> 
> , it is better to simply comment it so that it looks like
> 
> <VirtualHost _default:443>
> #ServerName
> </VirtualHost>
> 
> 
> This would allow customers to not edit the ServerName field in multiple places

But I think it is common to define ServerName inside a VirtualHost. Why should 
it be commented ?

> 
> - The default httpd-ssl.conf does not enable SSLRandomSeed. Don't you need to 
> enable this to reflect Solaris specific value. This way, user doesn't have to 
> know this information
> 

sslconf.sed already does it. This fix did not modify that part of the sed rule.


Thanks,
Seema.

Reply via email to