aboutsummaryrefslogtreecommitdiffstats
path: root/interfaces/email/interactive/be-handle-mail
Commit message (Collapse)AuthorAgeFilesLines
* Fixed a few typos in be-handle-mailW. Trevor King2009-09-131-4/+4
|
* RCS -> VCS, BUGDIR_DISK_VERSION -> v1.2W. Trevor King2009-08-311-2/+2
|
* Broke up be-handle-mail.parse() into parse_new(), _comment(), and _control().W. Trevor King2009-08-161-57/+75
| | | | Also fleshed out the be-handle-mail docstring.
* be-handle-mail no longer sends blank subscriber emails when no change occurs.W. Trevor King2009-07-271-4/+15
| | | | | It had been mailing out the empty MIMEMultipart root returned by diff_tree.report().
* Fixed _procmailrc rules.W. Trevor King2009-07-271-14/+18
| | | | | | "[" is a special character. Also a few minor cleanups in be-handle-mail.
* Cleaned up be-handle-mail's subscriber notification emails (fewer attachments).W. Trevor King2009-07-271-8/+46
| | | | | | | | | | | Previously, every node in the DiffTree created it's own attachment. Now they're consolidated into a single attachment per bug. higher level nodes are still one attachment per node. Also: * added send_pgp_mime.append_text() * pulled guess_encoding() out of send_pgp_mime.encodedMIMEText(). * renamed data_string -> data_part in libbe.diff, since it needn't be a string.
* Allow "subscribe" command via the email interface.W. Trevor King2009-07-271-1/+1
|
* Tweaked the output text for modified bugs in libbe.diff and be-handle-mail.W. Trevor King2009-07-271-1/+5
|
* Added "be-handle-mail --notify-since REVISION".W. Trevor King2009-07-271-27/+62
|
* Added an additional format check to be-handle-mail's pseudoheader parsing.W. Trevor King2009-07-271-0/+2
| | | | Marked outstanding email interface bugs as fixed.
* Moved be-handle-mail over to new libbe.diff classes.W. Trevor King2009-07-271-96/+49
|
* be-handle-mail successfully generates bugdir-changed notification emails.W. Trevor King2009-07-231-7/+180
| | | | | This still needs a lot of cleaning up, but it worked for an "all" subscription to "DIR", so I thought I'd lock in the current status ;).
* Use shlex.split() to parse control lines in be-handle-mail.W. Trevor King2009-07-201-3/+5
| | | | Split arguments following POSIX rather than at all whitespace.
* Added psuedo-header handling to be-handle-mail.W. Trevor King2009-07-201-3/+17
| | | | | | | | Many psuedo-headers had been ignored. Now they are all implemented. Getting this working exposed a few bugs in error message generation for Commands with IDs in their argument list. These bugs should now be fixed.
* Added interfaces/email/interactive/README and be-handle-mail options.W. Trevor King2009-07-201-11/+104
| | | | | | | | | | | | | | | The README should give enough info to install and use the interface. While I was writing it, I thought that be-handle-mail could use the --be-dir, --tag-base, and --test options. generate_global_tags() helps implement the --tag-base option. I set up a unittest framework since checking is currently a pipe-in-emails-by-hand sort of arrangement, which can be slow ;). Currently only generate_global_tags() is tested. I also restored "show" to ALLOWED_COMMANDS, since it seems to have wandered off ;).
* More verbose User/UsageError reporting in be-handle-mailW. Trevor King2009-07-191-2/+4
|
* Worked out some kinks in be-handle-mail's autocommit.W. Trevor King2009-07-191-1/+6
| | | | For example, it's helpful to actually run the autocommit command ;).
* Fixed typos in be-handle-mail error message generationW. Trevor King2009-07-191-4/+4
|
* Adjusted final commit-handling in be-handle-mail.W. Trevor King2009-07-191-8/+9
| | | | | | | | Now the final commit will run whether or not the preceding commands raise any exceptions. Note that since we've added the "--allow-empty" to "be commit", we don't need to worry about empty commits after read-only actions.
* Oops, _now_ I've fixed the multipart generation in be-handle-mailW. Trevor King2009-07-191-3/+3
|
* Fixed multipart bug in be-handle-mail.Message.response_email()W. Trevor King2009-07-191-5/+4
| | | | | I hadn't attached the mutipart body to the .response_header, which meant that the reply lacked target email addresses, etc.
* Added --disable-autocommit to be-handle-mail.W. Trevor King2009-07-191-1/+15
| | | | Also restored repsonse-message logging to help track down bugs.
* be-handle-mail now commits after every successful email execution.W. Trevor King2009-07-191-7/+13
| | | | | | | | | Caveats: It will produce blank commits after emails that make no changes. Todo: --fail-on-null option to "be commit" It will not commit changes due to emails that are partly successful. Todo: add "be revert"
* Added more allowed commands and pseudo-headers to be-handle-mail.W. Trevor King2009-07-191-3/+5
| | | | The new pseudo-headers are currently ignored.
* be-handle-mail's new DBT-style interface handles the example emails now.W. Trevor King2009-07-191-30/+43
|
* Reworked be-handle-mail to be more like the Debian Bug Tracker.W. Trevor King2009-07-191-78/+160
| | | | | Changed all the example emails over to the new format. Now it's time to try them all out and fix all the bugs ;).
* Added new_with_comment ability to be-handle-mail.W. Trevor King2009-07-181-4/+40
| | | | | Waiting for a response so you can get the bug ID for your initial comment is silly. Now you don't have to :)
* Assorted bugfixes to get reworked be-handle-mail working.W. Trevor King2009-07-181-20/+26
|
* Major be-handle-mail rewrite to make things more modular.W. Trevor King2009-07-181-171/+267
| | | | | Added Command and Message classes, and use new flexibility in send_pgp_mime.py.
* Starting to seperate message handling in be-handle-mail.W. Trevor King2009-07-181-25/+37
| | | | | The goal being to make handling commands differently easier, rather than just passing off the whole interface to becommands.
* Normalized whitespace in be-handle-mail and send_pgp_mime.py.W. Trevor King2009-07-181-4/+2
| | | | | | | | | | | | | | | | | | | | | Also removed "commit after every message" from be-handle-mail, because a) not implemented yet b) don't want to commit spam, since we'd have to find a way to remove it later. Suggested future workflow: * "bzr diff" to poll for activity, blank output = no activity. * on activity: 1) look at changes 2) remove whatever 3) commit email-interface repo. 4) merge changes into your private repo * on private repo changes: * if activity in email-interface repo: 1) deal with email activity as above * push your private repo onto the email-interface repo (and update the email repos' working tree, if required)
* be-handle-mail now handles non-text comments.W. Trevor King2009-07-181-0/+6
| | | | | | | | | | | | This required replacing both the codec-wrapped sys.stdin _and_ the raw sys.__stdin__ with StringIO(stdin). becommands/comment will use only one or the other depending on the comment's content type. Caveat: Get_body_type only grabs the body and type of the first non-mulitpart section, which may not be what the user expects. Todo: Add multiple comments for each part of a multipart message, like we do in interfaces/xml/be-mbox-to-xml.
* In be-handle-mail, don't mess with stdin if the command doesn't need it.W. Trevor King2009-07-181-4/+6
| | | | This fixes problems with StringIO(None).
* In be-handle-mail, give new bug summary via command line.W. Trevor King2009-07-181-2/+3
| | | | | | Fixes incorrect implementation of _comment_ bodies via stdin in my wking@drexel.edu-20090718143517-mkd6toxmcoij3qwk commit.
* For be-handle-mail, pass comment body in via a temporary stdin.W. Trevor King2009-07-181-3/+9
| | | | | This avoids decode-recode issues inside libbe.cmdutil.execute(), as well as problems due to large comment bodies.
* Corrected author_addr -> info["author_addr"] in be-handle-mailW. Trevor King2009-07-181-2/+2
|
* Fixed typo in one of be-handle-mail's InvalidCommand calls.W. Trevor King2009-07-181-1/+1
| | | | | Also restored Makefile target to home (from local), which I'd accidentally committed two commits ago...
* Allow external override of libbe.encoding.get_encoding().W. Trevor King2009-07-181-2/+3
| | | | | | The previous procmail encoding fix failed, because the becommand execution checks libbe.encoding.get_encoding() on it's own, and got the procmail encoding. This one works.
* Fixed some missing references in be-handle-mail.InvalidSubjectW. Trevor King2009-07-181-2/+2
|
* Hardcoded UTF-8 encoding in be-handle-mail.W. Trevor King2009-07-181-2/+4
| | | | | | | When run by procmail, the encoding returned by libbe.encoding.get_encoding is ANSI_X3.4-1968, which chokes on unicode output. I can't think of a more elegant solution than hardcoding in the default encoding.
* Added "to_unicode" to send_pgp_mime.flatten()W. Trevor King2009-07-181-3/+5
| | | | | | | | | | | be-handle-mail wants unicode output, since all it's internal processing is done with unicode. However, the flatten calls in send_pgp_mime work with the encoded binary string output, and execute(sendmail, stdin=flatten(msg, to_unicode=True)) fails with Exception: u while executing /usr/sbin/sendmail -t sendmail: fatal: wking(1001): No recipient addresses found in message header
* Added send_pgp_mime.Mail.encodedMIMEText() for unicode handling.W. Trevor King2009-07-181-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | Now be-handle-mail handles examples/unicode without crashing cat examples/unicode | ./be-handle-mail -o -l - But the output email is encoded in base64: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 From: BE Bugs <wking@thor.physics.drexel.edu> To: John Doe <jdoe@example.com> Date: Sat, 18 Jul 2009 12:22:05 +0000 Subject: [be-bug] Re: show In-reply-to: <abcd@example.com> UmVzdWx0cyBvZiBydW5uaW5nOiAoZXhpdCBjb2RlIDApCiAgc2hvdyAKCnN0ZG91dDoKCjw/eG1s IHZlcnNpb249IjEuMCIgZW5jb2Rpbmc9IlVURi04IiA/Pgo8YnVnPgogIDx1dWlkPmY3Y2NkOTE2 LWI1YzctNDg5MC1hMmUzLThjOGFjZTE3YWUzYTwvdXVpZD4KICA8c2hvcnQtbmFtZT5mN2M8L3No b3J0LW5hbWU+CiAgPHNldmVyaXR5Pm1pbm9yPC9zZXZlcml0eT4KICA8c3RhdHVzPmZpeGVkPC9z ... This is perhaps the best we can get out of python < 3.1/2.7, see http://bugs.python.org/issue1368247
* Fixed options.logfile -> logpath typo in be-handle-mail.open_logfile().W. Trevor King2009-07-181-1/+1
|
* Better unicode handling in be-handle-mail.W. Trevor King2009-07-181-16/+17
| | | | | | | | | | | | | | | | | | | | | | | be-handle-mail now gets a bit further on cat examples/unicode | ./be-handle-mail -o -l - 2>&1 1>/dev/null It successfully reads in unicode output from the command execution and successfully prints that output to the log ("-l - 2>&1 1>/dev/null" sets up the log to be printed to the terminal's stdout). However, it chokes later on with responding to John Doe <jdoe@example.com>: show Traceback (most recent call last): File "./be-handle-mail", line 274, in <module> main() File "./be-handle-mail", line 266, in main response_email = compose_response(ret, out_text, err_text, info).plain() File "./be-handle-mail", line 210, in compose_response LOGFILE.write("\n%s\n\n" % send_pgp_mime.flatten(response_email.plain())) File "/home/wking/src/fun/be/be.email/interfaces/email/interactive/send_pgp_mime.py", line 165, in flatten g.flatten(msg) File "/usr/lib/python2.5/email/generator.py" ... ... UnicodeEncodeError: 'ascii' codec can't encode character u'\ufffd' in position 2581: ordinal not in range(128)
* Added tracebacks to be-handle-mail's "uncaught exceptions".W. Trevor King2009-07-181-1/+2
|
* Added "be-handle-mail --logfile LOGFILE" for sane logfile handling.W. Trevor King2009-07-171-13/+43
| | | | | The previous setup had been pretty wimpy; now there's a degree of flexibility.
* Setup be-handle-mail.BE_DIR to point to the BE repo by default.W. Trevor King2009-07-171-1/+1
| | | | | | | At least, it points to the directory where be-handle-mail lives. If you haven't moved it, that will be somewhere inside the BE repository. This removes my hardcoded BE_DIR.
* "be-handle-mail --output" added to support easy testing.W. Trevor King2009-07-171-2/+14
|
* Removed debugging reply-address adjustment from be-handle-mail.W. Trevor King2009-07-161-1/+0
| | | | Everything seems to be working now. On to the remote tests ;).
* be-hand-mail now catches errors and sends appropriate responses.W. Trevor King2009-07-161-27/+94
|