Added full URL (keeping path and get parameters) in multiprotocol mode.

maaaaz / webscreenshot

A simple script to screenshot a list of websites

GNU Lesser General Public License v3.0

653 stars 162 forks source link

Added full URL (keeping path and get parameters) in multiprotocol mode. #52

Closed zonicdoe closed 4 years ago

zonicdoe commented 4 years ago

I wasn't able to screen shot full URLs in multiprotocol mode, the script was stripping out the path and GET parameters, leaving only the domain part as the target URL, I use it a lot with Dirsearch, so I made some changes to the code so the script will respect the full URL. If you consider these changes to be unnecessary, please, feel free to discard them and I will close this pull request.

Regards ;)

maaaaz commented 4 years ago

Hello @zonicdoe,

Very good catch, this was totally a bug. Fixed in v2.91(https://github.com/maaaaz/webscreenshot/commit/744ff9deb683d32134a78297a3154d7d5cdff32d)

Cheers.