1284990 - Use email address of user for bugfiler to avoid hashed usernames

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P4)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

I filed bug 1284982 via the bugfiler on autoland. In the comment 0 you can see the following name of the bug filer:

> Filed by: Wnf6VPw98Cn_qQasqiL7WyBmHjg

As Ed pointed out on IRC we should better use the email address as the username. Whereby we would have to take care about the privacy. I doubt we want to show it to everyone.

Comment 1

[GitHub Autolander Bot]

Attachment: [treeherder] KWierso:1284990 > mozilla:master

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Comment on attachment 8768550 [details] [review]
[treeherder] KWierso:1284990 > mozilla:master

I'm still not super confident in my python, but I think this should grab the username portion of a user's email address, accomplishing what I originally wanted this to accomplish. 
I had only tested with my own account, which showed as wkocher, so I wasn't aware that newer users didn't get treated similarly.

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Comment on attachment 8768550 [details] [review]
[treeherder] KWierso:1284990 > mozilla:master

This does 'user [at] domain.com', which is marginally better, I guess.

Comment 4

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/c9333d7e36c4dfac00c6f1823efaa69ec7cd981c
Bug 1284990 - Use user email address instead of username (#1654) r=emorley

I originally used usernames so I wouldn't be printing full email addresses
into bugs. Apparently older users in Treeherder have a readable name as
the username field, but newer users have a random string of
letters/numbers, which is much less useful.

This change makes it so we print the email address in the form 'user [at]
domain.com', which is slightly better.