From 81404d2e3c8bf17de5570006376f1508e13c7b0e Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Thu, 30 Apr 2020 21:07:26 +0300 Subject: [PATCH] search.js: make the check for URL stricter (#30695) --- site/assets/js/search.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/site/assets/js/search.js b/site/assets/js/search.js index ab57ad2f00..bb97c5cf80 100644 --- a/site/assets/js/search.js +++ b/site/assets/js/search.js @@ -35,14 +35,14 @@ transformData: function (hits) { return hits.map(function (hit) { var currentUrl = getOrigin() - var liveUrl = 'https://getbootstrap.com' + var liveUrl = 'https://getbootstrap.com/' - // When in production, return the result as is, - // otherwise remove our url from it. - // eslint-disable-next-line no-negated-condition - hit.url = currentUrl.indexOf(liveUrl) !== -1 ? // lgtm [js/incomplete-url-substring-sanitization] + hit.url = currentUrl.lastIndexOf(liveUrl, 0) === 0 ? + // On production, return the result as is hit.url : - hit.url.replace(liveUrl, '') + // On development or Netlify, replace `hit.url` with a trailing slash, + // so that the result link is relative to the server root + hit.url.replace(liveUrl, '/') // Prevent jumping to first header if (hit.anchor === 'content') {