Merge pull request #9 from luchoman08/master

badoux · web-flow · commit 36995ce6b19f · 2019-08-27T18:11:53.000+02:00
fix: icon links are no only with exact rel="ico"
diff --git a/goscraper.go b/goscraper.go
@@ -197,7 +197,7 @@ func (scraper *Scraper) parseDocument(doc *Document) error {
 				if cleanStr(attr.Key) == "rel" && cleanStr(attr.Val) == "canonical" {
 					canonical = true
 				}
-				if cleanStr(attr.Key) == "rel" && cleanStr(attr.Val) == "icon" {
+				if cleanStr(attr.Key) == "rel" && strings.Contains(cleanStr(attr.Val),  "icon") {
 					hasIcon = true
 				}
 				if cleanStr(attr.Key) == "href" {

Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,7 @@ func (scraper Scraper) parseDocument(doc Document) error {`
`197`	`197`	`if cleanStr(attr.Key) == "rel" && cleanStr(attr.Val) == "canonical" {`
`198`	`198`	`canonical = true`
`199`	`199`	`}`
`200`		`- if cleanStr(attr.Key) == "rel" && cleanStr(attr.Val) == "icon" {`
	`200`	`+ if cleanStr(attr.Key) == "rel" && strings.Contains(cleanStr(attr.Val), "icon") {`
`201`	`201`	`hasIcon = true`
`202`	`202`	`}`
`203`	`203`	`if cleanStr(attr.Key) == "href" {`