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

Meta-refresh inconsistencies with ACT #3496

Closed
WilcoFiers opened this issue Jun 2, 2022 · 1 comment
Closed

Meta-refresh inconsistencies with ACT #3496

WilcoFiers opened this issue Jun 2, 2022 · 1 comment
Labels
ACT Rules fix Bug fixes pr A pr has been created for the issue
Milestone

Comments

@WilcoFiers
Copy link
Contributor

Axe-core's meta-refresh rule is inaccurate in two ways:

  1. It should account for the 20-hour timeout. Anything timeout more than 20 hours should pass the rule
  2. Axe's method of determining the duration is far more permissive than browsers are, which results in it failing elements that do not redirect at all.

Passed example 3

<meta http-equiv="refresh" content="72001; https://w3.org" />

Inapplicable Example 3

<meta http-equiv="refresh" content="0: https://w3.org" />

Inapplicable Example 4

<meta http-equiv="refresh" content="-00.12 foo" />

Inapplicable Example 5

<meta http-equiv="refresh" content="; 30" />

Inapplicable Example 7

<meta http-equiv="refresh" content="+5; https://w3.org" />

Inapplicable Example 8

<meta http-equiv="refresh" content="foo; URL='https://w3.org'" />
@WilcoFiers WilcoFiers added this to the Axe-core 4.5 milestone Jun 2, 2022
@WilcoFiers WilcoFiers added the pr A pr has been created for the issue label Jun 29, 2022
@padmavemulapati
Copy link

Validated with latest develop branch code base,
meta-refresh-no-exceptions rule for pass and fail cases:

<head>
    <meta charset="utf8" />
    <title>Meta-refresh-no-exceptions fail 1</title>
    <meta http-equiv="refresh" content="72001: https://w3.org" />
    <meta http-equiv="refresh" content="90000; https://deque.com/">
    
    <link
      rel="stylesheet"
      type="text/css"
      href="/node_modules/mocha/mocha.css"
    />

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ACT Rules fix Bug fixes pr A pr has been created for the issue
Projects
None yet
Development

No branches or pull requests

2 participants