Merge branch 'gihnius-master'

html.go

@@ -32,6 +32,7 @@ const (
 	HTML_SANITIZE_OUTPUT                      // strip output of everything that's not known to be safe
 	HTML_SAFELINK                             // only link to trusted protocols
 	HTML_NOFOLLOW_LINKS                       // only link with rel="nofollow"
+	HTML_HREF_TARGET_BLANK                    // add a blank target
 	HTML_TOC                                  // generate a table of contents
 	HTML_OMIT_CONTENTS                        // skip the main contents (for a standalone table of contents)
 	HTML_COMPLETE_PAGE                        // generate a complete HTML page
@@ -411,6 +412,15 @@ func (options *Html) AutoLink(out *bytes.Buffer, link []byte, kind int) {
 		out.WriteString("mailto:")
 	}
 	entityEscapeWithSkip(out, link, skipRanges)
+
+	if options.flags&HTML_NOFOLLOW_LINKS != 0 && !isRelativeLink(link) {
+		out.WriteString("\" rel=\"nofollow")
+	}
+	// blank target only add to external link
+	if options.flags&HTML_HREF_TARGET_BLANK != 0 && !isRelativeLink(link) {
+		out.WriteString("\" target=\"_blank")
+	}
+
 	out.WriteString("\">")
 
 	// Pretty print: if we get an email address as
@@ -498,9 +508,14 @@ func (options *Html) Link(out *bytes.Buffer, link []byte, title []byte, content
 		out.WriteString("\" title=\"")
 		attrEscape(out, title)
 	}
-	if options.flags&HTML_NOFOLLOW_LINKS != 0 {
+	if options.flags&HTML_NOFOLLOW_LINKS != 0 && !isRelativeLink(link) {
 		out.WriteString("\" rel=\"nofollow")
 	}
+	// blank target only add to external link
+	if options.flags&HTML_HREF_TARGET_BLANK != 0 && !isRelativeLink(link) {
+		out.WriteString("\" target=\"_blank")
+	}
+
 	out.WriteString("\">")
 	out.Write(content)
 	out.WriteString("</a>")
@@ -800,3 +815,23 @@ func doubleSpace(out *bytes.Buffer) {
 		out.WriteByte('\n')
 	}
 }
+
+func isRelativeLink(link []byte) (yes bool) {
+	yes = false
+
+	// a tag begin with '#'
+	if link[0] == '#' {
+		yes = true
+	}
+
+	// link begin with '/' but not '//', the second maybe a protocol relative link
+	if len(link) >= 2 && link[0] == '/' && link[1] != '/' {
+		yes = true
+	}
+
+	// only the root '/'
+	if len(link) == 1 && link[0] == '/' {
+		yes = true
+	}
+	return
+}

inline_test.go

@@ -524,10 +524,28 @@ func TestInlineLink(t *testing.T) {
 
 func TestNofollowLink(t *testing.T) {
 	var tests = []string{
-		"[foo](http://bar.baz/)\n",
+		"[foo](http://bar.com/foo/)\n",
-		"<p><a href=\"http://bar.baz/\" rel=\"nofollow\">foo</a></p>\n",
+		"<p><a href=\"http://bar.com/foo/\" rel=\"nofollow\">foo</a></p>\n",
 	}
 	doTestsInlineParam(t, tests, 0, HTML_SAFELINK|HTML_NOFOLLOW_LINKS|HTML_SANITIZE_OUTPUT)
+	// HTML_SANITIZE_OUTPUT won't allow relative links, so test that separately:
+	tests = []string{
+		"[foo](/bar/)\n",
+		"<p><a href=\"/bar/\">foo</a></p>\n",
+	}
+	doTestsInlineParam(t, tests, 0, HTML_SAFELINK|HTML_NOFOLLOW_LINKS)
+}
+
+func TestHrefTargetBlank(t *testing.T) {
+	var tests = []string{
+        // internal link
+		"[foo](/bar/)\n",
+		"<p><a href=\"/bar/\">foo</a></p>\n",
+
+		"[foo](http://example.com)\n",
+		"<p><a href=\"http://example.com\" target=\"_blank\">foo</a></p>\n",
+	}
+	doTestsInlineParam(t, tests, 0, HTML_SAFELINK|HTML_HREF_TARGET_BLANK)
 }
 
 func TestSafeInlineLink(t *testing.T) {