Defect #9365
closed
Gravatar don't utilize HTTPS
Added by Christian Speich about 13 years ago.
Updated over 8 years ago.
Category:
Accounts / authentication
Description
When I set the scheme to https in the configuration and access my redmine installation via https, gravatars are not embedded via https. This may open security issues.
About your application's environment
Ruby version 1.8.7 (x86_64-linux)
RubyGems version 1.3.7
Rack version 1.1.2
Rails version 2.3.11
Active Record version 2.3.11
Active Resource version 2.3.11
Action Mailer version 2.3.11
Active Support version 2.3.11
Application root /var/www-vhosts/redmine-christian-speich/redmine
Environment production
Database adapter mysql
Database schema version 20110511000000
- Category set to Accounts / authentication
- Status changed from New to Confirmed
- Assignee set to Jean-Baptiste Barth
- Target version set to Candidate for next major release
- Affected version (unused) changed from 1.2.1 to devel
- Affected version deleted (
1.2.1)
Still a problem on Redmine 2.x, and it's a problem for me too as Chrome issues a certificate error because of that.
Analysis: In fact in turns out that the gravatar URL scheme (http or https) does not depend on the "http/https" option in the first tab. It depends on wether your request is run in SSL mode or not. See here in the code. If your Redmine runs behind a reverse-proxy, with this kind of architecture: HTTPS->RP->HTTP->APP_SERVER, the application server thinks it runs in HTTP mode while your clients access Redmine in HTTPS mode.
Proposal: I think we should rely on the "HTTP/HTTPS" setting on the first page, and do not make assumptions on how Redmine is hosted.
Etienne or Jean-Philippe: let me know what you think, I'll take care of this issue if we agree on the proposal.
- Target version changed from Candidate for next major release to 2.0.4
Jean-Baptiste Barth wrote:
Proposal: I think we should rely on the "HTTP/HTTPS" setting on the first page, and do not make assumptions on how Redmine is hosted.
Agreed.
Toshi MARUYAMA wrote:
Should we change to gravatarify?
Is this change supposed to fix anything?
Jean-Philippe Lang wrote:
Toshi MARUYAMA wrote:
Should we change to gravatarify?
Is this change supposed to fix anything?
Sorry, I misunderstood.
Changing avatar plugin has no effect to this issue.
- Status changed from Confirmed to Resolved
- Resolution set to Fixed
Fix committed in r10184.
Toshi MARUYAMA wrote:
Sorry, I misunderstood.
Changing avatar plugin has no effect to this issue.
No problem, upgrading would be fine anyway.
I think r10184 will break the scenario where users optionally use SSL to access a Redmine installation. In our environment SSL is forced so it's not a concern personally, but you'll end up loading mixed content if the settings are configured for HTTP but a user explicitly makes an HTTPS request. This was previously reported in #5929.
It seems the ideal fix would load Gravatar images over SSL if either HTTPS is configured in Redmine settings, OR the current request is using HTTPS.
(Note: we did have to set Apache up to use X-Forwarded-Proto headers to get Gravatar images properly served over SSL through Thin, and r10184 would prevent that since we do have HTTPS configured in the Redmine settings - so that's an improvement for me, but I'm not sure how many folks may be using optional HTTPS and have similar concerns to those raised in #5929).
- Status changed from Resolved to Closed
- Target version deleted (
2.0.4)
- Resolution changed from Fixed to Wont fix
You're right, I've reverted r10184. X-Forwarded-Proto should be used instead, so I'm closing it.
Fine with that, I didn't know Rails (or Rack) was following X-Forwarded-Proto, thanks.
Hi, I'm using Redmine 2.4.1.stable with Protocol option set to HTTPS (and obviously use HTTPS itself to connect) and I have the issue of gravatar icons not being loaded in SSL.
As initially reported, this may open security issues. Apparently it was fixed 2 years ago, so I guess I am reporting some kind of regression.
- Related to Patch #18911: check Setting.protocol when determining gravatar protocol added
Yannick Warnier wrote:
Hi, I'm using Redmine 2.4.1.stable with Protocol option set to HTTPS (and obviously use HTTPS itself to connect) and I have the issue of gravatar icons not being loaded in SSL.
As initially reported, this may open security issues. Apparently it was fixed 2 years ago, so I guess I am reporting some kind of regression.
Did you try X-Forwarded-Proto?
- Related to Defect #21855: Gravatar get images over http instead https added
Still a problem with version 3.2.0
Yann Collette wrote:
Still a problem with version 3.2.0
See #21855.
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by Toshi MARUYAMA 
Updated by 
Updated by 
Updated by 
Updated by 
Updated by