Skip to content

Commit 0abea32

Browse files
Chris Howsonparamt
authored andcommitted
Resolving issue with multiple blacklist links
1 parent f63349c commit 0abea32

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

check_links.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,21 +26,23 @@ def remove_duplicates(urls):
2626

2727
# Remove mailto links
2828
links = [url for url in file_links if "mailto://" not in url]
29+
linksToRequest = []
2930

3031
# Remove blacklisted links
3132
for link in links:
3233
if link in blacklisted:
33-
links.remove(link)
3434
print(f"Removed {link}")
35+
else:
36+
linksToRequest.append(link)
3537

3638
print(f"Checking URLs from {file}")
3739

3840
# Remove Duplicate links
39-
links = remove_duplicates(links)
41+
linksToRequest = remove_duplicates(linksToRequest)
4042

4143
print(f"Removing duplicate URLs from {file}")
4244

43-
for url in links:
45+
for url in linksToRequest:
4446
try:
4547
request = requests.get(url)
4648
if request.status_code == 200:

0 commit comments

Comments
 (0)