Skip to content

Normalizing valid targets #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
sorensen wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Normalizing valid targets #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 22 additions & 14 deletions absolutify.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,25 +9,27 @@
* @return {String} replaced html source
*/

function replace(str, url) {
if (typeof url === 'function') return replace.iterate(str, url)
return str.replace(replace.rx, '$1' + url + '/$4')
function absolutify(str, url) {
if (typeof url === 'function') return absolutify.iterate(str, url)
return str
.replace(absolutify.rx, '$1' + url + '/$6') // Inject the URL into the attribute
.replace(new RegExp(url + '//', 'g'), url + '/') // Fix `attr="/"` edgecase causing a `//` issue
}

/*!
* The magic, find all occurences of `attr="/`, ignoring any `//` found,
* ensure that the leading `/` of the url is not captured
*
* HTML attribute list from: http://stackoverflow.com/questions/2725156/complete-list-of-html-tag-attributes-which-have-a-url-value
*/

replace.rx = /((href|src|codebase|cite|background|cite|action|profile|formaction|icon|manifest|archive)=["'])(([.]+\/)|(?:\/)|(?=#))(?!\/)/g

/*!
* Match the same as above, but capture the full URL for iteration
1. find any possible element containing a url: `prop="`
2. ignore leading protocols: `prop="http:`
3. ignore protocol skips `prop="//`
4. find and omit leading slash to normalize: `prop="/url`
5.
*/

replace.captureRx = /((href|src|codebase|cite|background|cite|action|profile|formaction|icon|manifest|archive)=["'])((([.]+\/)|(?:\/)|(?:#))(?!\/)[a-zA-Z0-9._-]+)/g
absolutify.rx = /((href|src|codebase|cite|background|action|profile|formaction|icon|manifest|archive)=["'])(?!(http|https|ftp|file|filesystem|gopher|ws|wss|about|blob|data|mailto):|(\/\/))((?:\/)?([^'"]+))/g // jshint ignore:line

/**
* URL replacement using function iteration, this is handled slightly
Expand All @@ -39,17 +41,23 @@ replace.captureRx = /((href|src|codebase|cite|background|cite|action|profile|for
* @return {String} replaced html source
*/

replace.iterate = function(str, iterator) {
return str.replace(replace.captureRx, function(full, prefix, prop, url) {
return prefix + iterator(url, prop)
absolutify.iterate = function(str, iterator) {
return str.replace(absolutify.rx, function() {
var url = arguments[6]

return arguments[1] + iterator(
url === '/' ? '' : url // URL without leading `/` (check for `attr="/"` edgecase)
, arguments[2] // HTML attribute
, arguments[5] // Contains leading `/` if found
)
})
}

/*!
* Exports
*/

if (typeof exports !== 'undefined') module.exports = replace
else this.absolutify = replace
if (typeof exports !== 'undefined') module.exports = absolutify
else this.absolutify = absolutify

}.call(this));
Loading