[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [RFR] templates://dtc/dtc.templates



On Sat, Mar 10, 2007 at 06:46:37PM +0100, Christian Perrier wrote:
> Here is the now complete review of the dtc package templates.
> 
> Please send review as unified diffs.
> 
> In about 7 days, or earlier if no comments are sent, I'll post a Last
> Chance For Comments mail, then later send the rewritten templates as a
> bug report against the package.

Here are some questions/proposal in addition to the patch.

dtc's maintainer: there are also some questions for you.

> Template: dtc/conf_ipaddr
> Type: string
> Default: 192.168.0.2
> _Description: Primary IP address:
>  Please enter this host's primary IP address. This IP will be used for
>  the domain name you just provided, and will be used as the default for
>  most DTC variables.  If you're using NAT, please enter your external
>  IP address.

Another style question: shall we use "you're" or "you are"?

> Template: dtc/conf_admpass
> Type: password
> _Description: Main domain admin password:
>  Please choose the main domain administrator's password. Access
>  to the control panel must be managed manually through an .htpasswd
>  file in the root path.

"an .htpasswd" or "a .htpasswd"?

> Template: dtc/conf_mta_type
> Type: select
> Choices: postfix, qmail
> _Description: Mail Transfer Agent (MTA) type:
>  Please select the type of the MTA that will be used with DTC. It
>  should be already installed on the system. If no such MTA is actually
>  installed, please complete the setup of DTC, install a MTA
>  package, then run "dpkg-reconfigure dtc".

Maybe:
Please select the MTA that will be used with DTC.

Maybe we can recommend installing a mail-transport-agent package instead
of a MTA?

> Template: dtc/conf_use_cyrus
> Type: boolean
> Default: false
> _Description: Use Cyrus mail system?
>  Please choose this option if you are using Cyrus for mail
>  delivery, imap and MTA. This option is only compatible with Postfix.

I still have the same issue with this one.
This may be related to my inexperience with Cyrus. Is Cyrus an MTA?

dtc maintainer: What is the meaning of "imap and MTA"?
Is cyrus an IMAP server and an MTA?

> Template: dtc/conf_use_nated_vhosts
> Type: boolean
> Default: false
> _Description: Use "NATed" vhosts?
>  DTC can generate configuration files for Apache to have it use an IP
>  address of your server. In case the server is firewalled with NAT and
>  port redirections of public IP(s) address(es), you can request for
>  the generation of "NATed" vhosts.
>  .
>  Do not choose this option if the server is directly connected to the
>  Internet, except when using dynamic IP addresses. In such case, NAT
>  and port forwarding are mandatory for DTC.

This one looks strange.
Here is another en_FR attempt:

DTC can configure Apache to use one of your IP addresses. If the server is
firewalled with NAT and port redirections of public IP(s) address(es), a
"NATed" vhost configuration can be generated.
...

I'm not sure about the last paragraph.

I'm not sure neither about the plural form of "IP address" (I'm talking
about the '(s)' after IP)

> Template: dtc/conf_nated_vhosts_ip
> Type: string
> Default: 192.168.0.2
> _Description: NATed LAN IP address:
>  Please enter the IP address of the server for DTC to
>  generate all vhosts that will be used by Apache.

dtc maintainer: 

Are there multiple vhosts?
If we can enter multiple IPs, what's the separator?

> Template: dtc/conf_gen_ssl_cert
> Type: boolean
> Default: false
> _Description: Generate an OpenSSL certificate for DTC?
>  If you choose this option, DTC will generate a self signed OpenSSL
>  certificate and will use SSL to browse the panel. SSL will also be
>  activated and the generated Apache configuration will activate HTTPS
>  URLs for the control panel.
>  .
>  This certificate can later be changed for a root CA certificate.
>  .
>  Previously generated certificates will never be overwritten. To
>  regenerate the certificate, you need to remove all the files in
>  /usr/share/dtc/etc/ssl.

Maybe "SSL certificate" of just "certificate" instead of
"OpenSSL certificate".

> Template: dtc/conf_cert_challenge_pass
> Type: password
> _Description: DTC SSL certificate challenge password:

dtc maintainer:

What is this challenge password? There was already a "DTC certificate
passphrase:" template.

Given the wording, it looks like a certificate request's challenge
password.

> Template: dtc/conf_upgrade_pear
> Type: boolean
> Default: true
> _Description: Update PEAR installation?
>  DTC can check the PEAR installation during its own installation
>  process. This requires your server to be connected to the network. If
>  you can't do so here, please check whether the Crypt_CBC and
>  Auth_SASL PEAR modules are. The relevant files are located in
>  /usr/share/php.

dtc maintainer:

Is there a Debian package?
Wouldn't it be better to drop this template and replace it with a
dependency?

> Template: dtc/conf_dnsbl_list
> Type: string
> Default: sbl-xbl.spamhaus.org,list.dsbl.org,relays.ordb.org
> _Description: DNSBL (DNS BlackList) list:
>  Please enter the list of preferred DNSBL servers to add to your Postfix
>  mail server configuration.

I agree with MJ that it would be better not to promote DNSBL (i.e. I would
prefer a blank default). But it's up to the maintainer.

> Template: dtc/conf_report_setup_stat
> Type: boolean
> Default: true
> _Description: Allow to report anonymous statistics to GPLHost?
>  DTC installations can be reported to GPLHost web site. The only

reported to _the_ GPLHost web site?

>  collected data are the operating system name (Debian) and the IP
>  address (used as a unique identifier only). An Internet connection
>  and the wget binary are mandatory for statistics collection..

double stop, also I think we are talking about sending stats (not collect
them)
...mandatory to report the statistics.


Best Regards,
-- 
Nekral
--- dtc.templates.orig	2007-03-10 22:11:44.000000000 +0100
+++ dtc.templates	2007-03-10 23:50:36.000000000 +0100
@@ -9,7 +9,7 @@
 Default: root
 _Description: MySQL administrator:
  Please enter the login name of a MySQL user with administrative
- privileges. DTC will use it to to grant privileges for the tables to
+ privileges. DTC will use it to grant privileges for the tables to
  its users.
 
 Template: dtc/conf_mysqlpass
@@ -52,7 +52,7 @@
 _Description: Primary IP address:
  Please enter this host's primary IP address. This IP will be used for
  the domain name you just provided, and will be used as the default for
- most DTC variables.  If you're using NAT, please enter your external
+ most DTC variables. If you're using NAT, please enter your external
  IP address.
 
 Template: dtc/conf_hostingpath
@@ -78,7 +78,7 @@
 Default: dtc
 _Description: Main domain admin name:
  Each DTC domain must have an administrator. Please enter the login
- name of the administator for the domain name holding the control
+ name of the administrator for the domain name holding the control
  panel installation.
 
 Template: dtc/conf_admpass
@@ -172,7 +172,7 @@
 Template: dtc/conf_cert_unit
 Type: string
 Default: no-unit
-_Description: Organization unit for the DTC SSL:
+_Description: Organization unit for the DTC SSL certificate:
 
 Template: dtc/conf_cert_email
 Type: string
@@ -189,7 +189,7 @@
  DTC can check the PEAR installation during its own installation
  process. This requires your server to be connected to the network. If
  you can't do so here, please check whether the Crypt_CBC and
- Auth_SASL PEAR modules are. The relevant files are located in
+ Auth_SASL PEAR modules are installed. The relevant files are located in
  /usr/share/php.
 
 Template: dtc/conf_dnsbl_list

Reply to: