jekyll-paginate icon indicating copy to clipboard operation
jekyll-paginate copied to clipboard

validate output for paginate_path fix #8

Open rusikf opened this issue 10 years ago • 3 comments

rusikf avatar Sep 01 '14 13:09 rusikf

to pass Travis CI , merge my previous pull request :smirk:

rusikf avatar Sep 01 '14 14:09 rusikf

Don't you think this kind of sanitization should occur in the URL class (and that we should use URL for handling url generation in this class)?

parkr avatar Sep 01 '14 20:09 parkr

But colons is safe in url, why we must sanitize it? I think colons in url must be allowed , but writing paginate_path with colons in config.yml must not be allowed. May be it should occure in Pager#initialize ?

rusikf avatar Sep 02 '14 11:09 rusikf