rt icon indicating copy to clipboard operation
rt copied to clipboard

Decode MIME headers for better readability in web interface.

Open petrkle opened this issue 11 years ago • 9 comments

Hi,

this change "Subject: =?UTF-8?B?UMWZa....." to readable version.

Cheers Petr

petrkle avatar Nov 08 '12 11:11 petrkle

I'm not sure we actually want this behaviour. It is often useful to see exactly what RT sent out, and munging that on display means you then need to go poking through the database.

tsibley avatar Nov 08 '12 18:11 tsibley

On Thu, Nov 8, 2012 at 10:14 PM, Thomas Sibley [email protected]:

I'm not sure we actually want this behaviour. It is often useful to see exactly what RT sent out, and munging that on display means you then need to go poking through the database.

Good variant is to put a tag around with decoded value in title attribute so decoded value can be seen on mouse over.

— Reply to this email directly or view it on GitHubhttps://github.com/bestpractical/rt/pull/42#issuecomment-10198556.

Best regards, Ruslan.

ruz avatar Nov 09 '12 07:11 ruz

It would be nice that ordinary user can read subject and sender of outgoing messages. Especialy in non-ascii countries. Admins can change preferences to show raw headers.

http://git.io/oY9CCw

petrkle avatar Nov 09 '12 14:11 petrkle

I like @ruz idea. You can see the decoded and encoded version without switching a config option.

netsandbox avatar Nov 09 '12 15:11 netsandbox

On 11/09/2012 06:27 AM, Petr wrote:

It would be nice that ordinary user can read subject and sender of outgoing messages. Especialy in non-ascii countries. Admins can change preferences to show raw headers.

Ordinary users generally don't scan through full email headers to find the Subject and recipients/sender.

I don't think this warrants a config option or user preference. I prefer @ruz's suggestion.

tsibley avatar Nov 09 '12 20:11 tsibley

Not optimal in my opinion, but better than nothing.

petrkle avatar Nov 12 '12 12:11 petrkle

Why did you close this request without the commit being merged?

tsibley avatar Dec 12 '12 21:12 tsibley

After a month, nothing happened.

petrkle avatar Dec 13 '12 13:12 petrkle

On 12/13/2012 05:06 AM, Petr wrote:

After a month, nothing happened.

We're a busy team. Not everything gets dealt with within a month. :)

I agree that it would be useful to have some sort of decoding of headers, and I think the tooltip solution is a good compromise. It requires someone with a little bit of time to do some testing, but otherwise it should be a shoo-in for the RT 4.2 series (master branch currently).

Looking at your latest patch more closely, my main concern is that it runs decode("MIME-Headers", ...) over the entire header block, rather than on individual headers.

tsibley avatar Dec 13 '12 19:12 tsibley