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

W3C.pm line 223 does not view a redirect as a success #3

Open
TomasDawe opened this issue Nov 7, 2016 · 0 comments
Open

W3C.pm line 223 does not view a redirect as a success #3

TomasDawe opened this issue Nov 7, 2016 · 0 comments

Comments

@TomasDawe
Copy link

The default URL for string validation is currently under a 302 rule.
According to LWP this is not a is_success.
it might be wise to change 223 to
if(index($response->{_content},"Status: 30") > -1 || $response->is_success)
as this will take into account redirects.

I am hesitant to hard code the 302 URL as this will likely be removed at some point.
Opinion?

@TomasDawe TomasDawe changed the title line 223 does not view a redirect as a success W3C.pm line 223 does not view a redirect as a success Nov 7, 2016
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