Make the hostname parsing more stable for urls · secureCodeBox/secureCodeBox@e554ce7 · GitHub

secureCodeBox / secureCodeBox Public

• Notifications You must be signed in to change notification settings
• Fork 183
• Star 985

• Code
• Issues 62
• Pull requests 3
• Discussions
• Actions
• Projects
• Security and quality 1
• Insights

Additional navigation options

Commit e554ce7

Browse filesBrowse the repository at this point in the historyBrowse files

J12934

authored and

Weltraumschaf

committed

Make the hostname parsing more stable for urls

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>

1 parent 128e1e2 commit e554ce7
Copy full SHA for e554ce7

1 file changed

+7-3Lines changed: 7 additions & 3 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• scanners/nuclei/parser
  • parser.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎scanners/nuclei/parser/parser.js‎

Copy file name to clipboardExpand all lines: scanners/nuclei/parser/parser.js

+7-3Lines changed: 7 additions & 3 deletions

Original file line number	Diff line number	Diff line change
@@ -97,11 +97,15 @@ function parseHostname(host) {
97	97	if (!host) {
98	98	return null;
99	99	}
	100	+ // If the host doesn't have a proper protocol, add one to make URL parsing work
	101	+ const hasValidProtocol = /^[a-zA-Z][a-zA-Z0-9+.-]*:\/\//.test(host);
	102	+
	103	+ if (!hasValidProtocol) {
	104	+ return host;
	105	+ }
100	106
101	107	try {
102		- // If the host doesn't have a protocol, add one to make URL parsing work
103		- const urlString = host.includes("://") ? host : `http://${host}`;
104		- const url = new URL(urlString);
	108	+ const url = new URL(host);
105	109	return url.hostname;
106	110	} catch (err) {
107	111	// If URL parsing fails, check if the host is already a hostname (no protocol, no path)

0 commit comments

Comments

0 (0)