Delete Camo references & Addressable gem
At one point we piloted a "Camo" service to proxy HTTPS image urls for us, but it doesn't exist anymore. We already have proxies and stuff for this, so I left `Image` as a placeholder for this, but it's not working yet! This also deletes our final reference to the Addressable gem, so we can remove it!
This commit is contained in:
parent
7e3658d329
commit
c44daea318
6 changed files with 4 additions and 26 deletions
2
Gemfile
2
Gemfile
|
@ -19,8 +19,6 @@ gem 'RocketAMF', :git => 'https://github.com/rubyamf/rocketamf.git'
|
||||||
gem 'msgpack', '~> 0.5.3'
|
gem 'msgpack', '~> 0.5.3'
|
||||||
gem 'openneo-auth-signatory', '~> 0.1.0'
|
gem 'openneo-auth-signatory', '~> 0.1.0'
|
||||||
|
|
||||||
gem 'addressable', :require => ['addressable/template', 'addressable/uri']
|
|
||||||
|
|
||||||
gem 'right_aws', '~> 2.1.0'
|
gem 'right_aws', '~> 2.1.0'
|
||||||
|
|
||||||
gem "character-encodings", "~> 0.4.1", :platforms => :ruby_18
|
gem "character-encodings", "~> 0.4.1", :platforms => :ruby_18
|
||||||
|
|
|
@ -226,7 +226,6 @@ PLATFORMS
|
||||||
|
|
||||||
DEPENDENCIES
|
DEPENDENCIES
|
||||||
RocketAMF!
|
RocketAMF!
|
||||||
addressable
|
|
||||||
capistrano (~> 2.15.5)
|
capistrano (~> 2.15.5)
|
||||||
character-encodings (~> 0.4.1)
|
character-encodings (~> 0.4.1)
|
||||||
compass-rails (~> 3.1)
|
compass-rails (~> 3.1)
|
||||||
|
|
|
@ -235,9 +235,5 @@ module ApplicationHelper
|
||||||
:contributions_link_url => user_contributions_path(user),
|
:contributions_link_url => user_contributions_path(user),
|
||||||
:user_points => user.points
|
:user_points => user.points
|
||||||
end
|
end
|
||||||
|
|
||||||
def camo_image_url(image_url)
|
|
||||||
Image.from_insecure_url(image_url).secure_url
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
|
@ -7,19 +7,8 @@ class Image
|
||||||
end
|
end
|
||||||
|
|
||||||
def self.from_insecure_url(insecure_url)
|
def self.from_insecure_url(insecure_url)
|
||||||
Image.new insecure_url, proxy_insecure_url(insecure_url)
|
# TODO: We used to use a "Camo" server for this, but we don't anymore.
|
||||||
end
|
# Replace this with actual logic to actually secure the URLs!
|
||||||
|
Image.new insecure_url, insecure_url
|
||||||
private
|
|
||||||
|
|
||||||
def self.proxy_insecure_url(insecure_url)
|
|
||||||
if CAMO_HOST && CAMO_KEY
|
|
||||||
hexdigest = OpenSSL::HMAC.hexdigest(OpenSSL::Digest.new('sha1'), CAMO_KEY, insecure_url)
|
|
||||||
uri = Addressable::URI.parse("#{CAMO_HOST}/#{hexdigest}")
|
|
||||||
uri.query_values = { url: insecure_url }
|
|
||||||
uri.to_s
|
|
||||||
else
|
|
||||||
insecure_url
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -80,7 +80,7 @@
|
||||||
|
|
||||||
%li
|
%li
|
||||||
= link_to bulk_pets_path do
|
= link_to bulk_pets_path do
|
||||||
= image_tag camo_image_url('http://images.neopets.com/items/mall_ac_garland_spotlight.gif')
|
= image_tag 'http://images.neopets.com/items/mall_ac_garland_spotlight.gif'
|
||||||
%h3= link_to t('modeling_hub'), bulk_pets_path
|
%h3= link_to t('modeling_hub'), bulk_pets_path
|
||||||
%div
|
%div
|
||||||
%h4= t '.modeling_hub.tagline'
|
%h4= t '.modeling_hub.tagline'
|
||||||
|
|
|
@ -1,4 +0,0 @@
|
||||||
# For the openneo-camo.herokuapps.com server, which proxies assets through HTTPS.
|
|
||||||
# If you have no config, that's okay; we'll just serve the regular URL instead of the Camo URL.
|
|
||||||
CAMO_HOST = ENV['CAMO_HOST']
|
|
||||||
CAMO_KEY = ENV['CAMO_KEY']
|
|
Loading…
Reference in a new issue