User views title of HTTPS page

[#9462801]
veezus · Feb 2, 2012 · 7c948a7 · 7c948a7
1 parent 165055d
commit 7c948a7
Show file tree

Hide file tree

Showing 5 changed files with 624 additions and 25 deletions.
diff --git a/app/models/vurl.rb b/app/models/vurl.rb
@@ -164,10 +164,16 @@ def fetch_metadata
     logger.warn "Could not fetch data for #{construct_url}."
   end
 
-  def get_page
-    uri = URI.parse(construct_url)
-    Net::HTTP.start(uri.host, uri.port) do |http|
-      http.get(uri.request_uri, "User-Agent" => "Vurl.me Metadata Fetcher drone #9fc3po").body
+  def get_page(url = construct_url)
+    uri = URI.parse(url)
+    use_ssl = uri.scheme == 'https'
+    response = Net::HTTP.start(uri.host, uri.port, :use_ssl => use_ssl) do |http|
+      http.get(uri.request_uri, "User-Agent" => "Vurl.me Metadata Fetcher drone #9fc3po")
+    end
+    if response.header['location'].present?
+      get_page response.header['location']
+    else
+      response.body
     end
   end
 

diff --git a/spec/data/http_github_com.html b/spec/data/http_github_com.html
@@ -0,0 +1,15 @@
+HTTP/1.1 301 Moved Permanently
+Server: nginx/1.0.4
+Date: Sat, 26 Nov 2011 20:22:06 GMT
+Content-Type: text/html
+Content-Length: 184
+Connection: keep-alive
+Location: https://github.com/
+
+<html>
+<head><title>301 Moved Permanently</title></head>
+<body bgcolor="white">
+<center><h1>301 Moved Permanently</h1></center>
+<hr><center>nginx/1.0.4</center>
+</body>
+</html>