Skip to content

33 request to modify url canonicalization for compatibility with onion sites#34

Merged
MikeMeliz merged 2 commits intomasterfrom
33-request-to-modify-url-canonicalization-for-compatibility-with-onion-sites
Nov 16, 2024
Merged

33 request to modify url canonicalization for compatibility with onion sites#34
MikeMeliz merged 2 commits intomasterfrom
33-request-to-modify-url-canonicalization-for-compatibility-with-onion-sites

Conversation

@MikeMeliz
Copy link
Owner

Description

As described in #33, when http:// was defined, the url_canon() wasn't parsing it correctly. Therefore, this PR closes #33.

How Has This Been Tested?

python torcrawl.py -w -u google.com -c -d 1 -p 1 -l -v -e
python torcrawl.py -w -u google.com -c -d 1 -p 1 -l -v
python torcrawl.py -w -u google.com -c -d 3 -l -v

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@MikeMeliz MikeMeliz added the bug label Nov 16, 2024
@MikeMeliz MikeMeliz self-assigned this Nov 16, 2024
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 16, 2024

@MikeMeliz MikeMeliz merged commit 3d54889 into master Nov 16, 2024
@MikeMeliz MikeMeliz deleted the 33-request-to-modify-url-canonicalization-for-compatibility-with-onion-sites branch November 16, 2024 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Request to Modify URL Canonicalization for Compatibility with .onion Sites

1 participant