nostr
/
nostrweb
3
1
Fork 1
Code Issues 24 Pull Requests 3 Projects 1 Releases 27 Activity

content: exclude links to ip addresses #53

Merged
offbyn merged 2 commits from content-exclude-iplinks into master 2022-12-20 21:56:58 +00:00
Conversation 0 Commits 2 Files Changed 1
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/domutil.js
View File

@ -31,6 +31,9 @@ function isValidURL(url) {
if (!['', '443', '80'].includes(url.port)) { if (!['', '443', '80'].includes(url.port)) {
return false; return false;
} }
if (url.hostname === 'localhost') {
return false;
}
const lastDot = url.hostname.lastIndexOf('.'); const lastDot = url.hostname.lastIndexOf('.');
if (lastDot < 1) { if (lastDot < 1) {
return false; return false;
@ -38,6 +41,12 @@ function isValidURL(url) {
if (url.hostname.slice(lastDot) === '.local') { if (url.hostname.slice(lastDot) === '.local') {
offbyn marked this conversation as resolved
x1ddos commented 2022-12-20 21:33:55 +00:00
Review

btw, also refuse localhost?

btw, also refuse localhost?
return false; return false;
} }
if (url.hostname.slice(lastDot + 1).match(/^[\d]+$/)) { // there should be no tld with numbers, possible ipv4
return false;
}
if (url.hostname.includes(':')) { // possibly an ipv6 addr; certainly an invalid hostname
return false;
}
return true; return true;
} }