[Bug 266220] Re: Approved: only removed from text/plain part

Mark Sapiro mark at msapiro.net
Tue Oct 11 20:09:12 CEST 2011


I'm not going to try to handle that one. The current code looks in the
HTML for a match to the pattern constructed by

pattern = name + ':(\xA0|\s| )*' + re.escape(passwd)

which seems to work in most cases. I'm not sure how the above was
created, but at a minimum, the text was colored dark grey. If I just
create a simple, 'rich formatting' message in gmail, I don't see that.
Even with colored text, I see only things like

<span style=3D"color: rgb(51, 51, 51);">Approved: password</span>

I suggest that your poster try creating the post using gmail's "plain
text" composition if possible. This (at least in my tests) creates a
simple text/plain message.

If the poster feels it is necessary to include whatever fonts, colors,
etc. that cause the 'Approved: password' line to be split up into those
multiple <span> segments, there's not much I can do.

I could try to detect a case where I can find the Approved: password
text if I strip out all the HTML tags, but not if I don't, and refuse to
approve the post in that case, but I'm not sure how that should work.
Probably, I should reject the post entirely with some reason like
"Unable to remove 'Approved: password' text from HTML part". Holding the
post doesn't seem right because it could be manually approved which
would expose the password.

-- 
You received this bug notification because you are a member of Mailman
Coders, which is subscribed to GNU Mailman.
https://bugs.launchpad.net/bugs/266220

Title:
  Approved: only removed from text/plain part

To manage notifications about this bug go to:
https://bugs.launchpad.net/mailman/+bug/266220/+subscriptions


More information about the Mailman-coders mailing list