Merge pull request #202 from projectdiscovery/fix-certspotter

Fixed a bound check issue with certspotter
projectdiscovery · Dec 14, 2019 · ab7fc96 · ab7fc96
2 parents ade6065 + e315870
commit ab7fc96
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/pkg/subscraping/sources/certspotter/certspotter.go b/pkg/subscraping/sources/certspotter/certspotter.go
@@ -49,6 +49,12 @@ func (s *Source) Run(ctx context.Context, domain string, session *subscraping.Se
 			}
 		}
 
+		// if the number of responses is zero, close the channel and return.
+		if len(response) == 0 {
+			close(results)
+			return
+		}
+
 		id := response[len(response)-1].ID
 		for {
 			reqURL := fmt.Sprintf("https://api.certspotter.com/v1/issuances?domain=%s&include_subdomains=true&expand=dns_names&after=%s", domain, id)