rinku 2.0.0 → 2.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 07c65aa1e4fe77d917eeeb0d4ab03889f5b72443
-  data.tar.gz: 2e77157e5b959fbda3660e0676d20686e59bd469
+  metadata.gz: 2982203836f1449fe5344a32f7ab1782f3cb4cc7
+  data.tar.gz: 8461bfe12528f128cba4e81e9fb8f64e9903e6e5
 SHA512:
-  metadata.gz: e4c692320131620c6b1398a0d8784091bd1952660bfb1ab14ec7ee32fd115f7c0a39a6faddd924045f4085ee91a1062b67cd6a0600df768a7ccb092a1083e772
-  data.tar.gz: bc6774b49bb020deaa479ce8145c881e49e037c1539924e3f4cd2a0e8e0e1fb9871356b848e8090d3f3d79239131b0c4d7d6c8afdad117e0dd35edcbe1b5fa14
+  metadata.gz: 3587f9e83406d7829cdbec671980536803f3b9e0ed87e6daf8e21389dad8d2cc521d49527b602e3ccaa7e40f3c41ef365ba4bbb42a0b69d6ed7c412bbdd914f3
+  data.tar.gz: 68aa755836af8269edc44bcd4ae8134abe6a3bd7836140218f4bd618069f22ad16ff78c6814610f2525ca5f64cfac8657e2021c0ee238e1ba5d319e6efeb728c

data/COPYING

@@ -1,3 +1,5 @@
+ISC License
+
 Copyright (c) 2011, Vicent Marti
 
 Permission to use, copy, modify, and distribute this software for any

data/README.markdown

@@ -1,6 +1,9 @@
 Rinku does linking
 ==================
 
+[![Build Status](https://travis-ci.org/vmg/rinku.svg?branch=master)](https://travis-ci.org/vmg/rinku)
+[![Dependency Status](https://www.versioneye.com/ruby/rinku/badge.svg)](https://www.versioneye.com/ruby/rinku)
+
 Rinku is a Ruby library that does autolinking.
 It parses text and turns anything that remotely resembles a link into an HTML link,
 just like the Ruby on Rails `auto_link` method -- but it's about 20 times faster,
@@ -91,6 +94,12 @@ choose to use Rinku instead.
 
 ~~~~ruby
 require 'rails_rinku'
+include ActionView::Helpers::TextHelper
+post_body = "Welcome to my new blog at http://www.myblog.com/."
+auto_link(post_body, :html => { :target => '_blank' }) do |text|
+  truncate(text, :length => 15)
+end
+# => "Welcome to my new blog at <a href=\"http://www.myblog.com/\" target=\"_blank\">http://www.m...</a>."
 ~~~~
 
 The `rails_rinku` package monkeypatches Rails with an `auto_link` method that

data/Rakefile

@@ -1,50 +1,12 @@
-require 'date'
-require 'rake/clean'
+require 'bundler/setup'
+require 'bundler/gem_tasks'
 require 'rake/extensiontask'
-require 'digest/md5'
-
-task :default => :test
-
-# ==========================================================
-# Ruby Extension
-# ==========================================================
-
-Rake::ExtensionTask.new('rinku')
-
-# ==========================================================
-# Testing
-# ==========================================================
-
 require 'rake/testtask'
-Rake::TestTask.new('test') do |t|
-  t.test_files = FileList['test/*_test.rb']
-  t.ruby_opts += ['-rubygems'] if defined? Gem
-end
-task 'test' => [:compile]
 
-# PACKAGING =================================================================
+task default: :test
 
-require 'rubygems'
-$spec = eval(File.read('rinku.gemspec'))
+Rake::ExtensionTask.new('rinku') # defines compile task
 
-def package(ext='')
-  "pkg/rinku-#{$spec.version}" + ext
-end
-
-desc 'Build packages'
-task :package => package('.gem')
-
-desc 'Build and install as local gem'
-task :install => package('.gem') do
-  sh "gem install #{package('.gem')}"
-end
-
-desc 'Update the gemspec'
-task :update_gem => file('rinku.gemspec')
-
-directory 'pkg/'
-
-file package('.gem') => %w[pkg/ rinku.gemspec] + $spec.files do |f|
-  sh "gem build rinku.gemspec"
-  mv File.basename(f.name), f.name
+Rake::TestTask.new(test: :compile) do |t|
+  t.test_files = FileList['test/*_test.rb']
 end

data/ext/rinku/autolink.c

@@ -99,14 +99,10 @@ autolink_delim(const uint8_t *data, struct autolink_pos *link)
 	}
 
 	if (copen != 0) {
-		size_t closing = 0;
-		size_t opening = 0;
-		size_t i = link->start;
-
-		/* Try to close the final punctuation sign in this same line;
-		 * if we managed to close it outside of the URL, that means that it's
-		 * not part of the URL. If it closes inside the URL, that means it
-		 * is part of the URL.
+		/* Try to close the final punctuation sign in this link; if
+		 * there's more closing than opening punctuation symbols in the
+		 * URL, we conservatively remove one closing punctuation from
+		 * the end of the URL.
 		 *
 		 * Examples:
 		 *
@@ -117,12 +113,16 @@ autolink_delim(const uint8_t *data, struct autolink_pos *link)
 		 *		=> http://www.pokemon.com/Pikachu_(Electric)
 		 *
 		 *	foo http://www.pokemon.com/Pikachu_(Electric)) bar
-		 *		=> http://www.pokemon.com/Pikachu_(Electric))
+		 *		=> http://www.pokemon.com/Pikachu_(Electric)
 		 *
 		 *	(foo http://www.pokemon.com/Pikachu_(Electric)) bar
-		 *		=> foo http://www.pokemon.com/Pikachu_(Electric)
+		 *		=> http://www.pokemon.com/Pikachu_(Electric)
 		 */
 
+		size_t closing = 0;
+		size_t opening = 0;
+		size_t i = link->start;
+
 		while (i < link->end) {
 			if (data[i] == copen)
 				opening++;
@@ -132,13 +132,32 @@ autolink_delim(const uint8_t *data, struct autolink_pos *link)
 			i++;
 		}
 
-		if (closing != opening)
+		if (closing > opening)
 			link->end--;
 	}
 
 	return true;
 }
 
+static bool
+autolink_delim_iter(const uint8_t *data, struct autolink_pos *link)
+{
+	size_t prev_link_end;
+	int iterations = 0;
+
+	while(link->end != 0) {
+		prev_link_end = link->end;
+		if (!autolink_delim(data, link))
+			return false;
+		if (prev_link_end == link->end || iterations > 5) {
+			break;
+		}
+		iterations++;
+	}
+
+	return true;
+}
+
 static bool
 check_domain(const uint8_t *data, size_t size,
 		struct autolink_pos *link, bool allow_short)
@@ -198,7 +217,7 @@ autolink__www(
 		return false;
 
 	link->end = utf8proc_find_space(data, link->end, size);
-	return autolink_delim(data, link);
+	return autolink_delim_iter(data, link);
 }
 
 bool
@@ -244,7 +263,7 @@ autolink__email(
 			break;
 	}
 
-	if ((link->end - pos) < 2 || nb != 1 || np == 0)
+	if ((link->end - pos) < 2 || nb != 1 || np == 0 || (np == 1 && data[link->end - 1] == '.'))
 		return false;
 
 	return autolink_delim(data, link);
@@ -278,5 +297,5 @@ autolink__url(
 	if (!autolink_issafe(data + link->start, size - link->start))
 		return false;
 
-	return autolink_delim(data, link);
+	return autolink_delim_iter(data, link);
 }

data/lib/rinku.rb

@@ -1,5 +1,5 @@
 module Rinku
-  VERSION = "1.7.3"
+  VERSION = "2.0.2"
 
   class << self
     attr_accessor :skip_tags

data/rinku.gemspec

@@ -1,8 +1,6 @@
-# encoding: UTF-8
-
 Gem::Specification.new do |s|
   s.name = 'rinku'
-  s.version = '2.0.0'
+  s.version = '2.0.2'
   s.summary = "Mostly autolinking"
   s.description = <<-EOF
     A fast and very smart autolinking library that
@@ -11,6 +9,7 @@ Gem::Specification.new do |s|
   s.email = 'vicent@github.com'
   s.homepage = 'https://github.com/vmg/rinku'
   s.authors = ["Vicent Marti"]
+  s.license = 'ISC'
   # = MANIFEST =
   s.files = %w[
     COPYING
@@ -39,5 +38,7 @@ Gem::Specification.new do |s|
 
   s.add_development_dependency "rake"
   s.add_development_dependency "rake-compiler"
-  s.add_development_dependency "minitest"
+  s.add_development_dependency "minitest", ">= 5.0"
+
+  s.required_ruby_version = '>= 2.0.0'
 end

data/test/autolink_test.rb

@@ -1,6 +1,5 @@
-# encoding: utf-8
-rootdir = File.dirname(File.dirname(__FILE__))
-$LOAD_PATH.unshift "#{rootdir}/lib"
+require 'bundler/setup'
+$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
 
 require 'minitest/autorun'
 require 'cgi'
@@ -10,7 +9,7 @@ require 'rinku'
 class RinkuAutoLinkTest < Minitest::Test
   def generate_result(link_text, href = nil)
     href ||= link_text
-    href = "http://" + href unless href =~ %r{\A\w+://}
+    href = "http://" + href unless href =~ %r{\A(\w+://|mailto:)}
     %{<a href="#{CGI.escapeHTML href}">#{CGI.escapeHTML link_text}</a>}
   end
 
@@ -363,6 +362,13 @@ This is just a test. <a href="http://www.pokemon.com">http://www.pokemon.com</a>
     assert_linked "email#{NBSP}<a href=\"mailto:#{email_raw}\">#{email_raw}</a>", "email#{NBSP}#{email_raw}"
   end
 
+  def test_identifies_unicode_spaces
+    assert_linked(
+      %{This is just a test. <a href="http://www.pokemon.com">http://www.pokemon.com</a>\u202F\u2028\u2001},
+      "This is just a test. http://www.pokemon.com\u202F\u2028\u2001"
+    )
+  end
+
   def test_www_is_case_insensitive
     url = "www.reddit.com"
     assert_linked generate_result(url), url
@@ -376,4 +382,34 @@ This is just a test. <a href="http://www.pokemon.com">http://www.pokemon.com</a>
     url = "WwW.reddit.CoM"
     assert_linked generate_result(url), url
   end
+
+  def test_non_emails_ending_in_periods
+    assert_linked "abc/def@ghi.", "abc/def@ghi."
+    assert_linked "abc/def@ghi. ", "abc/def@ghi. "
+    assert_linked "abc/def@ghi. x", "abc/def@ghi. x"
+    assert_linked "abc/def@ghi.< x", "abc/def@ghi.< x"
+    assert_linked "abc/<a href=\"mailto:def@ghi.x\">def@ghi.x</a>", "abc/def@ghi.x"
+    assert_linked "abc/<a href=\"mailto:def@ghi.x\">def@ghi.x</a>. a", "abc/def@ghi.x. a"
+  end
+
+  def test_urls_with_entities_and_parens
+    assert_linked "&lt;<a href=\"http://www.google.com\">http://www.google.com</a>&gt;", "&lt;http://www.google.com&gt;"
+
+    assert_linked "&lt;<a href=\"http://www.google.com\">http://www.google.com</a>&gt;)", "&lt;http://www.google.com&gt;)"
+
+    # this produces invalid output, but limits how much work we will do
+    assert_linked "&lt;<a href=\"http://www.google.com&gt;\">http://www.google.com&gt;</a>)&lt;)&lt;)&lt;)&lt;)&lt;)&lt;)", "&lt;http://www.google.com&gt;)&lt;)&lt;)&lt;)&lt;)&lt;)&lt;)"
+
+    url = "http://pokemon.com/bulbasaur"
+    assert_linked "URL is #{generate_result(url)}.", "URL is #{url}."
+    assert_linked "(URL is #{generate_result(url)}.)", "(URL is #{url}.)"
+
+    url = "www.pokemon.com/bulbasaur"
+    assert_linked "URL is #{generate_result(url)}.", "URL is #{url}."
+    assert_linked "(URL is #{generate_result(url)}.)", "(URL is #{url}.)"
+
+    url = "abc@xyz.com"
+    assert_linked "URL is #{generate_result(url, "mailto:#{url}")}.", "URL is #{url}."
+    assert_linked "(URL is #{generate_result(url, "mailto:#{url}")}.)", "(URL is #{url}.)"
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rinku
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.2
 platform: ruby
 authors:
 - Vicent Marti
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-19 00:00:00.000000000 Z
+date: 2016-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
 description: |2
       A fast and very smart autolinking library that
       acts as a drop-in replacement for Rails `auto_link`
@@ -80,7 +80,8 @@ files:
 - rinku.gemspec
 - test/autolink_test.rb
 homepage: https://github.com/vmg/rinku
-licenses: []
+licenses:
+- ISC
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -90,7 +91,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -98,7 +99,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.5
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Mostly autolinking