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

Cookie support #38

Open
grzegorz-janoszka opened this issue Apr 10, 2019 · 0 comments
Open

Cookie support #38

grzegorz-janoszka opened this issue Apr 10, 2019 · 0 comments

Comments

@grzegorz-janoszka
Copy link

Hi,
This plugin doesn't like redirect with cookies.

Just add to your site a few links like:
http://physrev.physiology.org/content/81/3/1031.full
http://www.fasebj.org/content/25/1_Supplement/528.2
https://www.nature.com/articles/pr201551

There is a quick fix to get rid of some of those errors. In the file broken-link-checker/modules/checkers/http.php around line 200 add another parameter to CURL:

curl_setopt($ch, CURLOPT_COOKIEFILE, "");

This causes curl to accept cookies and to store them temporarily in the memory. That is fine and it is enough.

It solves some issues, the problem remains with some links and their logic. Because BLC reports me as a redirect link which eventually gets redirected to itself. I think there has to be a check if the final url equals the original url, then there is nothing to report and it is not a redirect, despite there are different url's in between (for example to set cookies).

I will try to to add this logic to BLC a little bit later.

HongPong pushed a commit that referenced this issue Oct 19, 2019
Add filter for HTML Link parser content
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant