Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support update cookies according to the responses #281

Open
wants to merge 460 commits into
base: master
Choose a base branch
from

Conversation

gsfish
Copy link

@gsfish gsfish commented Sep 24, 2019

This is for accepting cookies from the website during redirections. The --update-cookies option is added to determine whether whatweb shoud use dynamic cookies.

I have recently encountered a case that a website uses SSO and there are serval redirections before getting to the final target. Which means that, in addition to specify the basic cookie using --cookie and --follow-redirect, the client need accept and update cookies from response, and follow serval redirection to complete the authorization.

urbanadventurer and others added 30 commits December 2, 2017 23:47
…class. Moved error output into lib/output/errors.rb
…lper Module with OutputHelper::certainty_to_words so it works from both instantiated and uninstantiated classes.
@urbanadventurer
Copy link
Owner

Thanks for submitting this PR @gsfish. WhatWeb's cookie handling is currently under review for an update.

lib/whatweb.rb Show resolved Hide resolved
lib/whatweb/scan.rb Outdated Show resolved Hide resolved
lib/whatweb/scan.rb Outdated Show resolved Hide resolved
lib/whatweb/scan.rb Outdated Show resolved Hide resolved
lib/whatweb/scan.rb Outdated Show resolved Hide resolved
lib/whatweb/scan.rb Outdated Show resolved Hide resolved
whatweb Outdated Show resolved Hide resolved
lib/whatweb/scan.rb Outdated Show resolved Hide resolved
lib/whatweb/scan.rb Outdated Show resolved Hide resolved
winezer0 added a commit to winezer0/whatweb-plus that referenced this pull request May 10, 2023
合并urbanadventurer/WhatWeb#281

支持根据响应更新 cookie
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.