From b04700d199ae1558aacc431400303aa9348ba43c Mon Sep 17 00:00:00 2001
From: Pulkit Kathuria <kevincobain2000@gmail.com>
Date: Mon, 6 May 2024 18:39:45 +0900
Subject: [PATCH] (bug) fix relative to absolute

---
 frontend/src/components/Extractor.astro |  4 ++--
 pkg/extract_handler.go                  |  2 +-
 pkg/strings.go                          | 29 ++++++++++++++++++-------
 3 files changed, 24 insertions(+), 11 deletions(-)

diff --git a/frontend/src/components/Extractor.astro b/frontend/src/components/Extractor.astro
index a84a92b..094e102 100644
--- a/frontend/src/components/Extractor.astro
+++ b/frontend/src/components/Extractor.astro
@@ -98,7 +98,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
             for="input-depth"
             class="block mb-2 text-sm font-medium text-slate-400"
           >
-            Depth <sup class="text-xs text-slate-500">-1 to 5</sup>
+            Depth <sup class="text-xs text-slate-500">-1 to 20</sup>
           </label>
           <input
             id="input-depth"
@@ -220,7 +220,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
   baseURL = baseURL.startsWith("http") ? baseURL : currentURL + baseURL;
   let input = Alpine.reactive({
     url: "",
-    depth: 2,
+    depth: 1,
     ignore_queries: true,
     limit_urls: 1000,
     limit_emails: 10000,
diff --git a/pkg/extract_handler.go b/pkg/extract_handler.go
index 1f567bc..94f11bc 100644
--- a/pkg/extract_handler.go
+++ b/pkg/extract_handler.go
@@ -25,7 +25,7 @@ func NewExtractHandler() *ExtractHandler {
 
 type ExtractorRequest struct {
 	URL           string `json:"url"  query:"url" validate:"required" message:"url is required"`
-	Depth         int    `json:"depth" query:"depth" default:"1" validate:"required,numeric,gte=-1,lte=5" message:"depth must be a number between -1 and 5"`
+	Depth         int    `json:"depth" query:"depth" default:"1" validate:"required,numeric,gte=-1,lte=5" message:"depth must be a number between -1 and 20"`
 	IgnoreQueries string `json:"ignore_queries" default:"true" query:"ignore_queries" validate:"required,eq=true|eq=false" message:"ignore_queries must be true or false"`
 	LimitUrls     int    `json:"limit_urls" query:"limit_urls" default:"100" validate:"required,numeric,gte=1,lte=1000" message:"limit_urls must be a number between 1 and 1000"`
 	LimitEmails   int    `json:"limit_emails" query:"limit_emails" default:"1000" validate:"required,numeric,gte=1,lte=10000" message:"limit_emails must be a number between 1 and 10000"`
diff --git a/pkg/strings.go b/pkg/strings.go
index 46e3213..5ab5b3c 100644
--- a/pkg/strings.go
+++ b/pkg/strings.go
@@ -103,19 +103,32 @@ func RelativeToAbsoluteURL(href, currentURL, baseURL string) string {
 	if strings.HasPrefix(href, "http") {
 		return href
 	}
-	// ignore if it is javascript:void(0)
-	if strings.HasPrefix(href, "javascript:void(0)") {
-		return ""
+	// Parse the URLs
+	u, err := url.Parse(href)
+	if err != nil {
+		return "" // handle error, return empty string or appropriate value
 	}
 
-	if strings.HasPrefix(href, "/") {
-		return baseURL + href
+	// If href is absolute, return it directly
+	if u.IsAbs() {
+		return u.String()
 	}
-	if strings.HasPrefix(href, "./") {
-		return currentURL + href[2:]
+
+	// Use baseURL if currentURL is not provided
+	if currentURL == "" {
+		parsedBaseURL, err := url.Parse(baseURL)
+		if err != nil {
+			return "" // handle error, return empty string or appropriate value
+		}
+		return parsedBaseURL.ResolveReference(u).String()
 	}
 
-	return currentURL + href
+	// Resolve the relative URL using currentURL
+	parsedCurrentURL, err := url.Parse(currentURL)
+	if err != nil {
+		return "" // handle error, return empty string or appropriate value
+	}
+	return parsedCurrentURL.ResolveReference(u).String()
 }
 
 func CountPerDomain(emails []string) map[string]int {