>>109105793
>https://github.com/lobsters/lobsters/pull/1218
<% if @user&.is_admin && @user&.show_email? %>
vs
<% if @user.is_admin? || @user.show_email? %>
Seems different to me.
That said:
https://github.com/lobsters/lobsters/commit/7037494eb85003603ecd87fb00bde994e111f103
>The fix is literally adding an extra "&" for the "show_user" and cleaning up that call to be more appropriate.
I don't think she did anything overly wrong (besides the obvious failure in security NOW) since that push was available for 3 years (nearly) and nobody else fixed it until now.
Ironically enough, it's the *PERSON THAT PUSHED IT* that is the one that fixed it. So ultimately it's on them for failing to see this the first time.