The protection to prevent email addresses from being displayed to logged out users is not perfect. When viewing a bug in the "format for printing" mode, you can see the email addresses (in the "header") of the reporter, assignee and QA contact. E.g.: https://bugzilla.redhat.com/bugzilla/long_list.cgi?buglist=170519 I can read: Assigned To: dkl Reported By: bressers QA Contact: dkl Note that we haven't fixed this problem on Bugzilla yet.
Red Hat Bugzilla is now using version 3.2 of the Bugzilla codebase and therefore this bug will need to be re-verified against the new release. With the updated code this bug may no longer be relevant or may have been fixed in the new code. Updating bug version to 3.2.
Cc list is also available. URL of "format for printing" is changed to https://bugzilla.redhat.com/show_bug.cgi?format=multiple&id=189835
Created attachment 317337 [details] patch to fix login name display in bug long listing Hi Dave, This is a patch to display users realnames only when users are not logged in in the long bug listing format. Please take a look when you can. Thanks, Noura
Comment on attachment 317337 [details] patch to fix login name display in bug long listing Cc list members are still shown with the login names when user is not logged in. Please also have that display real names only or omit the Cc list section. Thanks Dave
Created attachment 317504 [details] patch with cc list dsiplaying realnames and loginnames Thanks for the review Dave, here is another version of the patch with the cclist fixed in the bug long listing format. Noura
Comment on attachment 317504 [details] patch with cc list dsiplaying realnames and loginnames >+ [% FOREACH c = bug.cc_users %] >+ [%# REDHAT EXTENSION START 189835 %] >+ [% IF user.id %] >+ [% c.login FILTER html %][% ", " IF not loop.last() %] >+ [% ELSE %] >+ [% c.name FILTER html %][% ", " IF not loop.last() %] >+ [% END %] >+ [%# REDHAT EXTENSION END 189835 %] > [% END %] Nit: To be cleaner, put the [% IF not loop.last() %] in it's own block like this: [% FOREACH c = bug.cc_users %] [%# REDHAT EXTENSION START 189835 %] [% IF NOT loop.last() %] [% IF user.id %] [% c.login FILTER html %] [% ELSE %] [% c.name FILTER html %] [% END %] [% END %] [%# REDHAT EXTENSION END 189835 %] [% END %] After that, please check this in. Thanks Dave
Thanks for the review Dave committed the patch to cvs with your suggestions. Noura