Skip to content

Commit

Permalink
Fix merge problem in pull request sherlock-project#43. Previously, th…
Browse files Browse the repository at this point in the history
…ere was just one for loop that iterated thru all of the sites. But, with the addition of the parallel functionality, there are now two for loops. The dictionary changes were not done in the second loop, which caused bogus results.
  • Loading branch information
hoadlck committed Dec 31, 2018
1 parent f9d5927 commit 8090a96
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion sherlock.py
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ def sherlock(username, verbose=False, tor=False, unique_tor=False):
results_total[social_network] = results_site

# Core logic: If tor requests, make them here. If multi-threaded requests, wait for responses
for social_network in data:
for social_network, net_info in data.items():

# Retrieve results again
results_site = results_total.get(social_network)
Expand Down

0 comments on commit 8090a96

Please sign in to comment.