Commit 12bb1d8f39d655576a98eabd6307cab5103e616c
Exists in
master
and in
4 other branches
Merge pull request #775 from netdata/gravatar_empty_issue
When the email address is empty gitlab will give a 500 error
Showing
1 changed file
with
1 additions
and
1 deletions
Show diff stats
app/helpers/application_helper.rb
| 1 | require 'digest/md5' | 1 | require 'digest/md5' |
| 2 | module ApplicationHelper | 2 | module ApplicationHelper |
| 3 | 3 | ||
| 4 | - def gravatar_icon(user_email, size = 40) | 4 | + def gravatar_icon(user_email = '', size = 40) |
| 5 | gravatar_host = request.ssl? ? "https://secure.gravatar.com" : "http://www.gravatar.com" | 5 | gravatar_host = request.ssl? ? "https://secure.gravatar.com" : "http://www.gravatar.com" |
| 6 | user_email.strip! | 6 | user_email.strip! |
| 7 | "#{gravatar_host}/avatar/#{Digest::MD5.hexdigest(user_email.downcase)}?s=#{size}&d=identicon" | 7 | "#{gravatar_host}/avatar/#{Digest::MD5.hexdigest(user_email.downcase)}?s=#{size}&d=identicon" |