jekyll_href 1.2.6 → 1.2.7

data/lib/href_private.rb

@@ -0,0 +1,143 @@
+module MSlinn
+  class HRefTag
+    private
+
+    # Does not look at or compute @link
+    def compute_linkk
+      return @link, nil if @link
+
+      linkk = @url
+      if linkk.nil? || !linkk
+        linkk = @helper.argv&.shift
+        @helper.params&.shift
+        @helper.keys_values&.delete(linkk)
+        return nil, error_no_uri if linkk.nil?
+      elsif @url.to_s.empty?
+        return nil, error_no_uri
+      end
+      [linkk, nil]
+    rescue HRefError => e
+      [nil, e]
+    end
+
+    def error_no_uri
+      msg = <<~END_MESSAGE
+        No URL was provided on #{@path}:#{@line_number} (after front matter).
+          <pre>{% href #{@argument_string.strip} %}</pre>
+      END_MESSAGE
+      @logger.error { JekyllPluginHelper.remove_html_tags msg.strip }
+      "<div class='href_error'>HRefError: #{msg}</div>"
+    end
+
+    # Sets @follow, @helper, @match, @path, @shy, @target, @url, @wbr
+    def globals_initial
+      @path = @page['path']
+      AllCollectionsHooks.compute(@site)
+
+      @blank           = @helper.parameter_specified? 'blank'
+      @klass           = @helper.parameter_specified? 'class'
+      @follow          = @helper.parameter_specified?('follow') ? '' : " rel='nofollow'"
+      @label           = @helper.parameter_specified? 'label'
+      @match           = @helper.parameter_specified? 'match'
+      @shy             = @helper.parameter_specified? 'shy'
+      @style           = @helper.parameter_specified? 'style'
+      @summary         = @helper.parameter_specified? 'summary'
+      @summary_exclude = @helper.parameter_specified? 'summary_exclude'
+      @target          = @blank ? " target='_blank'" : nil
+      @target        ||= @helper.parameter_specified?('notarget') ? '' : " target='_blank'"
+      @url             = @helper.parameter_specified? 'url'
+      @wbr             = @helper.parameter_specified? 'wbr'
+
+      @label_source    = if @helper.parameter_specified? 'page_title'
+                           :from_page_title
+                         elsif @label
+                           :from_explicit_label
+                         else
+                           :from_implicit_label
+                         end
+
+      return unless @tag_config
+
+      @die_on_href_error = @tag_config['die_on_href_error'] == true
+      @die_on_nomatch    = @tag_config['die_on_nomatch']    == true
+      @pry_on_href_error = @tag_config['pry_on_href_error'] == true
+    end
+
+    # Might set @external_link, @follow, @local_link, @linkk, @target, and @text
+    def globals_update(tokens, linkk)
+      if linkk.start_with? 'mailto:'
+        handle_mailto linkk
+        return
+      else
+        if @label_source == :from_page_title
+          handle_page_title linkk
+          return
+        end
+        @text = @label || tokens.join(' ').strip
+        if @text.to_s.empty?
+          handle_empty_text linkk
+        else
+          handle_text linkk
+        end
+        @link_save = @link
+      end
+
+      if @external_link
+        @target = ''
+        return
+      end
+      @follow = ''
+      @target = '' unless @blank
+    end
+
+    def handle_empty_text(linkk)
+      text = linkk
+      text = linkk&.gsub('/', '/&shy;') if @shy
+      text = linkk&.gsub('/', '/<wbr>') if @wbr
+      @text = "<code>#{text}</code>"
+      @external_link = linkk.start_with? 'http'
+      @local_link = !@external_link
+      @mailto_link = false
+      @link = if @external_link
+                linkk
+              elsif insecure_ip_address linkk
+                "http://#{linkk}"
+              else
+                "https://#{linkk}"
+              end
+    end
+
+    def handle_mailto(linkk)
+      @mailto_link = true
+      @local_link = @external_link = false
+      @link = linkk
+      @target = @follow = ''
+      @text = @helper.argv.join(' ')
+      return unless @text.empty?
+
+      text = linkk.delete_prefix('mailto:')
+      @text = "<code>#{text}</code>"
+    end
+
+    def handle_text(linkk)
+      @link = if @shy
+                linkk&.gsub('/', '/&shy;')
+              elsif @wbr
+                linkk&.gsub('/', '/<wbr>')
+              else
+                linkk
+              end
+    end
+
+    def insecure_ip_address(string)
+      return true if string.start_with? 'localhost'
+
+      return false unless IPAddress.valid? string
+
+      ip_address = IPAddress string
+      true if ip_address.loopback? || ip_address.private?
+    ensure
+      false
+    end
+  end
+end

data/lib/href_summary.rb

@@ -0,0 +1,39 @@
+module MSlinn
+  class HRefTag
+    private
+
+    def save_summary
+      return if @summary_exclude || @link_save.start_with?('mailto:') || @link_save.start_with?('#')
+
+      @summary = @summary.to_s.empty? ? @text : @summary.to_s
+      if @summary == true
+        warning = <<~END_WARNING
+          Warning: a href plugin keyword option was detected in the link text for #{@path} on line #{line_number}.
+          The href tag will not be included in the summary, and the link text will not have the word summary included.
+          This is probably unintentional. Consider using the label option to correct this problem."
+        END_WARNING
+        puts warning.red
+        return
+      end
+      return if @summary.class != String || @summary.empty?
+
+      @summary = @summary[0].upcase + @summary[1..]
+      @summary_href = "<a href='#{@link_save}'#{@target}#{@follow}>#{@summary}</a>"
+      mini_href = MiniHref.new(
+        follow:          @follow,
+        html:            @summary_href,
+        line_number:     @line_number,
+        link:            @link_save,
+        link_save:       @link_save,
+        path:            @path,
+        summary_exclude: @summary_exclude,
+        summary_href:    @summary_href
+      )
+      if @link_save.start_with? 'http'
+        add_global_link_for_page mini_href
+      else
+        add_local_link_for_page mini_href
+      end
+    end
+  end
+end

data/lib/href_summary_tag.rb

@@ -2,8 +2,8 @@ require 'jekyll_plugin_logger'
 require 'jekyll_plugin_support'
 require_relative 'jekyll_href/version'
 
-module HrefSummaryTag
-  class HrefSummary < JekyllSupport::JekyllTag # rubocop:disable Metrics/ClassLength
+module MSlinn
+  class HrefSummary < JekyllSupport::JekyllTag
     include JekyllHrefVersion
 
     # Class instance variables accumulate hrefs across invocations.
@@ -28,6 +28,11 @@ module HrefSummaryTag
     def render_impl
       @helper.gem_file __FILE__ # Enables attribution
       @include_local = @helper.parameter_specified? 'include_local'
+
+      @die_on_href_error = @tag_config['die_on_href_error'] == true if @tag_config
+      @pry_on_href_error = @tag_config['pry_on_href_error'] == true if @tag_config
+
+      @path = @page['path']
       global_refs = render_global_refs
       local_refs  = render_local_refs
       have_refs = !(global_refs + local_refs).empty?
@@ -36,12 +41,19 @@ module HrefSummaryTag
         #{local_refs}
         #{@helper.attribute if @helper.attribution && have_refs}
       END_RENDER
+    rescue HRefError => e # jekyll_plugin_support handles StandardError
+      e.shorten_backtrace
+      msg = format_error_message e.message
+      @logger.error "#{e.class} raised #{msg}"
+      binding.pry if @pry_on_img_error # rubocop:disable Lint/Debugger
+      raise e if @die_on_href_error
+
+      "<div class='href_error'>#{e.class} raised in #{self.class};\n#{msg}</div>"
     end
 
     def render_global_refs
       hrefs = HashArray.instance_variable_get(:@global_hrefs)
-      path = @page['path']
-      entries = hrefs[path]&.select { |h| h.path == path }
+      entries = hrefs[@path]&.select { |h| h.path == @path }
       return '' if entries.nil? || entries.empty?
 
       summaries = entries.map { |href| "<li>#{href.summary_href}</li>" }
@@ -61,8 +73,7 @@ module HrefSummaryTag
       return '' unless @include_local
 
       hrefs = HashArray.instance_variable_get(:@local_hrefs)
-      path = @page['path']
-      entries = hrefs[path]&.select { |h| h.path == path }
+      entries = hrefs[@path]&.select { |h| h.path == @path }
       return '' if entries.nil? || entries.empty?
 
       summary = entries.map { |href| "<li>#{href.summary_href}</li>" }

data/lib/href_tag.rb

@@ -3,7 +3,6 @@ require 'jekyll_all_collections'
 require 'jekyll_plugin_logger'
 require 'jekyll_plugin_support'
 require 'liquid'
-require 'sanitize'
 require_relative 'jekyll_href/version'
 require_relative 'hash_array'
 
@@ -11,11 +10,13 @@ require_relative 'hash_array'
 # @license SPDX-License-Identifier: Apache-2.0
 # Generates an href.
 
-module HrefTag
+module MSlinn
   MiniHref = Struct.new(:follow, :html, :link, :line_number, :link_save, :path, :summary_exclude, :summary_href, keyword_init: true)
 
+  HRefError = JekyllSupport.define_error
+
   # Implements href Jekyll tag
-  class HrefTag < JekyllSupport::JekyllTag # rubocop:disable Metrics/ClassLength
+  class HRefTag < JekyllSupport::JekyllTag
     attr_reader :follow, :helper, :line_number, :link_save, :match, :page, :path, :site,
                 :summary, :summary_exclude, :summary_href, :target, :text, :url
     attr_accessor :link
@@ -30,10 +31,6 @@ module HrefTag
       [follow, match, path, target, text] <=> [other.follow, other.match, other.path, other.target, other.text]
     end
 
-    def to_s
-      "On line #{line_number} of #{path}: #{follow} #{match} #{target} #{link} => '#{text}'"
-    end
-
     # Method prescribed by the Jekyll plugin lifecycle.
     # @param liquid_context [Liquid::Context]
     # @return [String]
@@ -42,10 +39,10 @@ module HrefTag
       linkk, error_msg = compute_linkk
       return error_msg unless linkk
 
-      linkk = replace_vars(linkk)
-      linkk.delete_prefix('./') # normalize relative links
+      linkk.delete_prefix './' # normalize relative links
       @url = linkk if @url
       @link_save = linkk
+
       @helper_save = @helper.clone
       globals_update(@helper.argv, linkk) # Sets @link and @text, might clear @follow and @target
       handle_match if @match
@@ -53,196 +50,25 @@ module HrefTag
       klass = " class='#{@klass}'" if @klass
       style = " style='#{@style}'" if @style
       "<a href='#{@link}'#{klass}#{style}#{@target}#{@follow}>#{@text}</a>"
+    rescue HRefError => e # jekyll_plugin_support handles StandardError
+      e.shorten_backtrace
+      msg = format_error_message e.message
+      @logger.error "#{e.class} raised #{msg}"
+      binding.pry if @pry_on_img_error # rubocop:disable Lint/Debugger
+      raise e if @die_on_href_error
+
+      "<div class='href_error'>#{e.class} raised in #{self.class};\n#{msg}</div>"
     end
 
-    private
-
-    def save_summary
-      return if @summary_exclude || @link_save.start_with?('mailto:') || @link_save.start_with?('#')
-
-      @summary = @summary.to_s.empty? ? @text : @summary.to_s
-      if @summary == true
-        warning = <<~END_WARNING
-          Warning: a href plugin keyword option was detected in the link text for #{@path} on line #{line_number}.
-          The href tag will not be included in the summary, and the link text will not have the word summary included.
-          This is probably unintentional. Consider using the label option to correct this problem."
-        END_WARNING
-        puts warning.red
-        return
-      end
-      return if @summary.class != String || @summary.empty?
-
-      @summary = @summary[0].upcase + @summary[1..]
-      @summary_href = "<a href='#{@link_save}'#{@target}#{@follow}>#{@summary}</a>"
-      mini_href = MiniHref.new(
-        follow:          @follow,
-        html:            @summary_href,
-        line_number:     @line_number,
-        link:            @link_save,
-        link_save:       @link_save,
-        path:            @path,
-        summary_exclude: @summary_exclude,
-        summary_href:    @summary_href
-      )
-      if @link_save.start_with? 'http'
-        add_global_link_for_page mini_href
-      else
-        add_local_link_for_page mini_href
-      end
-    end
-
-    # Does not look at or compute @link
-    def compute_linkk
-      return @link, nil if @link
-
-      linkk = @url
-      if linkk.nil? || !linkk
-        linkk = @helper.argv&.shift
-        @helper.params&.shift
-        @helper.keys_values&.delete(linkk)
-        return nil, error_no_uri if linkk.nil?
-      elsif @url.to_s.empty?
-        return nil, error_no_uri
-      end
-      [linkk, nil]
-    end
-
-    def error_no_uri
-      msg = <<~END_MESSAGE
-        Error: no url was provided on #{@path}:#{@line_number} (after front matter).
-          <pre>{% href #{@argument_string}%}</pre>
-      END_MESSAGE
-      @logger.error { Sanitize.fragment msg }
-      "<span class='error'>Error: #{msg}</span>"
-    end
-
-    # Sets @follow, @helper, @match, @path, @shy, @target, @url, @wbr
-    def globals_initial
-      @path = @page['path']
-      AllCollectionsHooks.compute(@site)
-
-      @blank           = @helper.parameter_specified? 'blank'
-      @klass           = @helper.parameter_specified? 'class'
-      @follow          = @helper.parameter_specified?('follow') ? '' : " rel='nofollow'"
-      @match           = @helper.parameter_specified? 'match'
-      @label           = @helper.parameter_specified? 'label'
-      @summary_exclude = @helper.parameter_specified? 'summary_exclude'
-      @shy             = @helper.parameter_specified? 'shy'
-      @style           = @helper.parameter_specified? 'style'
-      @summary         = @helper.parameter_specified? 'summary'
-      @target          = @blank ? " target='_blank'" : nil
-      @target        ||= @helper.parameter_specified?('notarget') ? '' : " target='_blank'"
-      @url             = @helper.parameter_specified? 'url'
-      @wbr             = @helper.parameter_specified? 'wbr'
-    end
-
-    # Might set @follow, @linkk, @target, and @text
-    def globals_update(tokens, linkk)
-      if linkk.start_with? 'mailto:'
-        @link = linkk
-        @target = @follow = ''
-        @text = @helper.argv.join(' ')
-        if @text.empty?
-          text = linkk.delete_prefix('mailto:')
-          @text = "<code>#{text}</code>"
-        end
-        return
-      else
-        @text = @label || tokens.join(' ').strip
-        if @text.to_s.empty?
-          text = linkk
-          text = linkk.gsub('/', '/&shy;') if @shy
-          text = linkk.gsub('/', '/<wbr>') if @wbr
-          @text = "<code>#{text}</code>"
-          @link = if linkk.start_with?('http')
-                    linkk
-                  elsif insecure_ip_address linkk
-                    "http://#{linkk}"
-                  else
-                    "https://#{linkk}"
-                  end
-        else
-          @link = if @shy
-                    linkk.gsub('/', '/&shy;')
-                  elsif @wbr
-                    linkk.gsub('/', '/<wbr>')
-                  else
-                    linkk
-                  end
-        end
-        @link_save = @link
-      end
-
-      return if @link.start_with? 'http'
-
-      @follow = ''
-      @target = '' unless @blank
-    end
-
-    def handle_match
-      match_post
-      @follow = ''
-      @target = '' unless @blank
-    end
-
-    # Might set @link and @text
-    def match_post
-      config = @site.config['href']
-      @die_if_nomatch = !config.nil? && config['nomatch'] && config['nomatch'] == 'fatal'
-      @path, @fragment = @link.split('#')
-
-      @logger.debug do
-        <<~END_DEBUG
-          @link=#{@link}
-          @site.posts.docs[0].url  = #{@site.posts.docs[0].url}
-          @site.posts.docs[0].path = #{@site.posts.docs[0].path}
-        END_DEBUG
-      end
-
-      all_urls = @site.all_collections.map(&:url)
-      compute_link_and_text(all_urls)
-    end
-
-    def compute_link_and_text(all_urls)
-      url_matches = all_urls.select { |url| url&.include? @path }
-      case url_matches.length
-      when 0
-        abort "href error: No url matches '#{@link}'" if @die_if_nomatch
-        @link_save = @link = '#'
-        @text = "<i>#{@link} is not available</i>"
-      when 1
-        @link = url_matches.first
-        @link = "#{@link}##{@fragment}" if @fragment
-        @link_save = @link
-      else
-        abort "Error: More than one url matched '#{@path}': #{url_matches.join(', ')}"
-      end
-    end
-
-    def insecure_ip_address(string)
-      return true if string.start_with? 'localhost'
-
-      return false unless IPAddress.valid? string
-
-      ip_address = IPAddress string
-      return true if ip_address.loopback? || ip_address.private?
-    rescue StandardError
-      false
-    ensure
-      false
-    end
-
-    # Replace names in plugin-vars with values
-    def replace_vars(text)
-      variables = @site.config['plugin-vars']
-      return text unless variables
-
-      variables.each do |name, value|
-        text = text.gsub "{{#{name}}}", value
-      end
-      text
+    def to_s
+      "On line #{line_number} of #{path}: #{follow} #{match} #{target} #{link} => '#{text}'"
     end
 
     JekyllPluginHelper.register(self, 'href')
   end
 end
+
+require_relative 'href_match'
+require_relative 'href_private'
+require_relative 'href_page_title'
+require_relative 'href_summary'

data/lib/jekyll_href/version.rb

@@ -1,3 +1,3 @@
 module JekyllHrefVersion
-  VERSION = '1.2.6'.freeze
+  VERSION = '1.2.7'.freeze
 end

data/lib/jekyll_href.rb

@@ -4,6 +4,5 @@ require_relative 'href_summary_tag'
 HrefError = Class.new(Liquid::Error)
 
 module JekyllHrefModule
-  include HrefTag
-  include HrefSummaryTag
+  include MSlinn
 end

data/spec/{hash_array_spec.rb → hash_array_spec/hash_array_spec.rb}

@@ -1,5 +1,5 @@
 module HashArraySpec
-  require_relative '../lib/hash_array'
+  require_relative '../../lib/hash_array'
 
   RSpec.describe HashArray do
     let(:logger) do

data/spec/spec_helper.rb

@@ -6,9 +6,7 @@ require 'yaml'
 require_relative "../lib/jekyll_href"
 
 RSpec.configure do |config|
-  config.filter_run :focus
   # config.order = "random"
-  config.run_all_when_everything_filtered = true
 
   # See https://relishapp.com/rspec/rspec-core/docs/command-line/only-failures
   config.example_status_persistence_file_path = "../spec/status_persistence.txt"

data/spec/status_persistence.txt

@@ -1,15 +1,15 @@
 example_id                                                        | status | run_time        |
 ----------------------------------------------------------------- | ------ | --------------- |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:1]  | passed | 0.00817 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:2]  | passed | 0.00349 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:3]  | passed | 0.0033 seconds  |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:4]  | passed | 0.00307 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:5]  | passed | 0.00315 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:6]  | passed | 0.0032 seconds  |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:7]  | passed | 0.0035 seconds  |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:8]  | passed | 0.00323 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:9]  | passed | 0.0032 seconds  |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:10] | passed | 0.00307 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:11] | passed | 0.00317 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:12] | passed | 0.00357 seconds |
-/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:13] | passed | 0.0035 seconds  |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:1]  | passed | 0.00938 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:2]  | failed | 0.01389 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:3]  | failed | 0.00319 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:4]  | failed | 0.00211 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:5]  | failed | 0.00196 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:6]  | failed | 0.00205 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:7]  | passed | 0.00212 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:8]  | failed | 0.00209 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:9]  | failed | 0.0021 seconds  |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:10] | passed | 0.00278 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:11] | passed | 0.00237 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:12] | passed | 0.00218 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_href/spec/href_spec.rb[1:13] | passed | 0.00211 seconds |

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll_href
 version: !ruby/object:Gem::Version
-  version: 1.2.6
+  version: 1.2.7
 platform: ruby
 authors:
 - Mike Slinn
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-06-19 00:00:00.000000000 Z
+date: 2023-12-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ipaddress
@@ -58,28 +58,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: 0.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
-- !ruby/object:Gem::Dependency
-  name: sanitize
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.8.1
 description: 'Generates an ''a href'' tag, possibly with target=''_blank'' and rel=''nofollow''.
 
   '
@@ -96,11 +82,15 @@ files:
 - Rakefile
 - jekyll_href.gemspec
 - lib/hash_array.rb
+- lib/href_match.rb
+- lib/href_page_title.rb
+- lib/href_private.rb
+- lib/href_summary.rb
 - lib/href_summary_tag.rb
 - lib/href_tag.rb
 - lib/jekyll_href.rb
 - lib/jekyll_href/version.rb
-- spec/hash_array_spec.rb
+- spec/hash_array_spec/hash_array_spec.rb
 - spec/href_spec.rb
 - spec/spec_helper.rb
 - spec/status_persistence.txt
@@ -131,7 +121,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.3
+rubygems_version: 3.3.7
 signing_key:
 specification_version: 4
 summary: Generates an 'a href' tag, possibly with target='_blank' and rel='nofollow'.