opengraph_parser 0.1.0 → 0.1.1

data/lib/open_graph.rb

@@ -1,5 +1,6 @@
 require 'nokogiri'
 require 'redirect_follower'
+require "addressable/uri"
 
 class OpenGraph
   attr_accessor :src, :url, :type, :title, :description, :images, :metadata, :response
@@ -13,6 +14,7 @@ class OpenGraph
     check_images_path
   end
 
+  private
   def parse_opengraph
     begin
       @response = RedirectFollower.new(@src).resolve
@@ -60,19 +62,19 @@ class OpenGraph
   end
 
   def check_images_path
-    uri = URI.parse(URI.escape(@src))
+    uri = Addressable::URI.parse(@src)
     imgs = @images.dup
     @images = []
     imgs.each do |img|
-      if URI.parse(URI.escape(img)).host.nil?
-        add_image("#{uri.scheme}://#{uri.host}:#{uri.port}#{img}")
+      if Addressable::URI.parse(img).host.nil?
+        full_path = generate_path(img, uri)
+        add_image(full_path)
       else
         add_image(img)
       end
     end
   end
 
-  private
   def add_image(image_url)
     @images << image_url unless @images.include?(image_url) || image_url.to_s.empty?
   end
@@ -82,4 +84,15 @@ class OpenGraph
       add_image(link.attribute(attr).to_s.strip)
     end
   end
+
+  def generate_path(relative_path, uri)
+    host = "#{uri.scheme}://#{uri.host}#{':' + uri.port.to_s if uri.port}"
+    if relative_path.start_with?('/')
+      "#{host}#{relative_path}"
+    elsif uri.path.to_s.end_with?('/')
+      "#{host}#{uri.path}#{relative_path}"
+    else
+      "#{host}#{uri.path}/#{relative_path}"
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opengraph_parser
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-12 00:00:00.000000000 Z
+date: 2012-11-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70116370351140 !ruby/object:Gem::Requirement
+  requirement: &70223353182580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,10 +22,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70116370351140
+  version_requirements: *70223353182580
 - !ruby/object:Gem::Dependency
   name: rdoc
-  requirement: &70116370350460 !ruby/object:Gem::Requirement
+  requirement: &70223353181860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -33,10 +33,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70116370350460
+  version_requirements: *70223353181860
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70116370349400 !ruby/object:Gem::Requirement
+  requirement: &70223353203980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -44,10 +44,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70116370349400
+  version_requirements: *70223353203980
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70116370365040 !ruby/object:Gem::Requirement
+  requirement: &70223353203500 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -55,10 +55,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70116370365040
+  version_requirements: *70223353203500
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70116370364240 !ruby/object:Gem::Requirement
+  requirement: &70223353203000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -66,7 +66,18 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70116370364240
+  version_requirements: *70223353203000
+- !ruby/object:Gem::Dependency
+  name: addressable
+  requirement: &70223353202480 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70223353202480
 description: A simple Ruby library for parsing Open Graph Protocol information from
   a website. It also includes a fallback solution when the website has no Open Graph
   information.
@@ -97,7 +108,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2116295098415564887
+      hash: 3401434948042854406
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: