Mercurial > p > roundup > code
comparison roundup/configuration.py @ 5098:99e289359798
issue2550803: Replying to NOSY mail goes to the tracker through
reply-to, not original message author.
Created new [tracker] replyto_address config.ini option to allow:
1) setting reply-to header to the tracker
2) setting reply-to header to the address of the author
of the change
3) setting it to a fixed address (like noreply@some.place)
Proposal by Peter Funk (pefu) in discussion with Tom Ekberg
(tekberg).
I chose to re-retrieve the email address for the author from the
database rather than adding a new variable. Also managed to make
a test case for each of the three settings.
| author | John Rouillard <rouilj@ieee.org> |
|---|---|
| date | Sun, 26 Jun 2016 00:36:23 -0400 |
| parents | e74c3611b138 |
| children | 96dc9f07340a |
comparison
equal
deleted
inserted
replaced
| 5097:156cbc1d182c | 5098:99e289359798 |
|---|---|
| 586 "that is required to get to the home page of the tracker.\n" | 586 "that is required to get to the home page of the tracker.\n" |
| 587 "You MUST include a trailing '/' in the URL."), | 587 "You MUST include a trailing '/' in the URL."), |
| 588 (MailAddressOption, "email", "issue_tracker", | 588 (MailAddressOption, "email", "issue_tracker", |
| 589 "Email address that mail to roundup should go to.\n" | 589 "Email address that mail to roundup should go to.\n" |
| 590 "If no domain is specified then mail_domain is added."), | 590 "If no domain is specified then mail_domain is added."), |
| 591 (Option, "replyto_address", "", | |
| 592 "Controls the reply-to header address used when sending\n" | |
| 593 "nosy messages.\n" | |
| 594 "If the value is unset (default) the roundup tracker's\n" | |
| 595 "email address (above) is used.\n" | |
| 596 "If set to \"AUTHOR\" then the primary email address of the\n" | |
| 597 "author of the change will be used as the reply-to\n" | |
| 598 "address. This allows email exchanges to occur outside of\n" | |
| 599 "the view of roundup and exposes the address of the person\n" | |
| 600 "who updated the issue, but it could be useful in some\n" | |
| 601 "unusual circumstances.\n" | |
| 602 "If set to some other value, the value is used as the reply-to\n" | |
| 603 "address. It must be a valid RFC2822 address or people will not be\n" | |
| 604 "able to reply."), | |
| 591 (NullableOption, "language", "", | 605 (NullableOption, "language", "", |
| 592 "Default locale name for this tracker.\n" | 606 "Default locale name for this tracker.\n" |
| 593 "If this option is not set, the language is determined\n" | 607 "If this option is not set, the language is determined\n" |
| 594 "by OS environment variable LANGUAGE, LC_ALL, LC_MESSAGES,\n" | 608 "by OS environment variable LANGUAGE, LC_ALL, LC_MESSAGES,\n" |
| 595 "or LANG, in that order of preference."), | 609 "or LANG, in that order of preference."), |