url_scrubber 0.8.2 → 0.8.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZDlkYzlmMDBiOTY5MzU0N2ExYTZjMjI4YzUyZDQ0MzFiZWNiYmM1NA==
+    ZWNmYjEyOTYxNjVmMGI4Mjg5ODVjMzc1ZjQ3ZmMwODA1NmYxZTMwNg==
   data.tar.gz: !binary |-
-    Y2NiYzUzMDAwZWRlZjA3MTYzZmM5ZTI2MjkyNDI2ZmQ3ZjRhMzcyMg==
+    N2Y4NzNkMWZkMWNlMjA4M2M2Y2QxZmQ0MWFmYTg0NzRlN2UxNmI3ZA==
 SHA512:
   metadata.gz: !binary |-
-    NTIyOGY2ZTc3YTNiMDQwMmE5NzNiZDhlOWMxMTcyYThhZDBmMTBhNDAxNTc5
-    MTIzM2RiMzVmM2E4ZTJmMDBlNjdjNmM2MDJiNWZhMWJjZjM0MDE5NjNlNTQ3
-    ZmFiYWYwYjJiNDRlMDg0MWUxMTdhNmY3MjdjYmQ2ZjI0YWFlNmQ=
+    ZGY3NmVkOTBmNjI5ZDUzYmEwZDE2OTcwZWU2ZDM2YTJjMzI2NzQwYzNhMzdm
+    MTIyNzUwNTY0M2Q3ODI0MzZlZWJhOGQ0M2Y4MzE0MWQ4ZjA4YjliZjIxZDdh
+    MTZkNTM3YjNiN2U0NWViMmNjZTViOTBlN2E5NGVkNjkzNTQyZDI=
   data.tar.gz: !binary |-
-    YTQwYmIwMGYzOGU2ZGZjOTA1NGE3MTNkNDk4NGMzNDkyOTBlMmM3MWFjYTRi
-    ODdhMDhjNmM0ZTIwYWY3MzZkY2UyMjQ2N2I4YThjNDQ1OGFkMDFlODU1OTU5
-    NTY2YzQxNzk0ZTk0MTE5ZTdlZGY2MzNmY2VlNTg4NTA5NTA3ZmY=
+    YmExM2U0NjY4YzM1ZmY3ZTU2MmU0ZTdiNmY5ZDU5YzQ5ZWI4MWM1YWUwNjM0
+    NmY2Y2Q5MDgwNWVjYWQ0Yzk5OTFmMWY3Nzg2MmYyZGExZWFkMzQ5NThmMjhm
+    YzQ4YTY3ZDdkYjgwZGZkMzM5Y2U0ODMwNWFiNzM3NzY4MmFhZjk=

data/lib/url_scrubber/version.rb

@@ -1,3 +1,3 @@
 module UrlScrubber
-  VERSION = "0.8.2"
+  VERSION = "0.8.3"
 end

data/lib/url_scrubber.rb

@@ -22,7 +22,7 @@ module UrlScrubber
     remove_www!(url)
     remove_html_tags!(url)
     url = drop_anchor!(special_cases(url))
-    url.chomp(',')
+    url.sub!(/,+$/, "")    # remove one or more trailing commas at the end of the URL
     url.gsub!(/\/+$/, '') # remove any trailing slashes (/) in the resulting URL
     return url
   end
@@ -198,7 +198,9 @@ module UrlScrubber
   end
 
   def self.sc_youtube(url)
-    url.sub!('youtube.com/user/', 'youtube.com/')
+    # We need to allow the /user version of the URL due to how YouTube allows users to have their own URL
+    # which is not separate channel with it's own customUrl.
+    # url.sub!('youtube.com/user/', 'youtube.com/')
     url.sub!('youtube.com/profile?user=', 'youtube.com/')
     url
   end

data/spec/url_scrubber_spec.rb

@@ -32,8 +32,8 @@ describe UrlScrubber do
 
 
     describe 'with youtube urls' do
-      it 'should drop /user/ if it exists' do
-        UrlScrubber.scrub('http://youtube.com/user/absolutely').should eq('http://youtube.com/absolutely')
+      it 'should NOT drop /user/ if it exists' do
+        UrlScrubber.scrub('http://youtube.com/user/absolutely').should eq('http://youtube.com/user/absolutely')
       end
 
       it 'should handle /profile?user= urls' do
@@ -41,7 +41,7 @@ describe UrlScrubber do
       end
 
       it 'should not kick in for a non-youtube url' do
-        UrlScrubber.scrub('http://example.com/user/absolutely').should_not eq('http://example.com/absolutely')
+        UrlScrubber.scrub('http://example.com/profile?user=absolutely').should_not eq('http://example.com/absolutely')
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: url_scrubber
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.8.3
 platform: ruby
 authors:
 - Colin Langton
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-26 00:00:00.000000000 Z
+date: 2016-12-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -124,7 +124,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.6
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Clean up URLs.