Opened at 2012-03-12T18:03:21Z
Last modified at 2012-03-13T17:52:24Z
#1684 new defect
duplicate settings in tahoe.cfg should produce an error
| Reported by: | davidsarah | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | undecided |
| Component: | code-nodeadmin | Version: | 1.9.1 |
| Keywords: | error config usability backward-compatibility | Cc: | |
| Launchpad Bug: |
Description
If there are multiple lines for the same setting in tahoe.cfg (for example, multiple introducer.furl = lines), Tahoe will silently pick one of them. I think it picks the one later in the file. This is most likely a user error that should be reported. It can happen when you have commented settings and intend only one of them to be uncommented, but make a mistake, for instance.
(I know this is just the standard behaviour of ConfigParser.SafeConfigParser, but it's still broken IMHO.)
There is a minor backward-compatibility issue in that config files that worked accidentally will stop working if we make this an error. I still think it should be an error rather than a warning, though.
+1 on making duplicate settings be an error. Here's a recent patch which inspected the config file and stopped with an error if there was something fishy. The person working on this ticket might want to emulate this patch, including its unit tests: e5c4e83f4cfe3769.