GoogleChrome/lighthouse

Canonical URL audit mistakes invalid URLs for relative ones

クローズ

#12,008 opened on 2021/01/26

GitHub で見るコントリビューターガイド
 (7 件のコメント) (0 件のリアクション) (0 人の担当者)JavaScript (9,278 件のフォーク)batch import
P3good first issue

Repository metrics

Stars
 (27,768 個のスター)
PR merge metrics
 (平均マージ 18d 18h) (30d で 40 merged PRs)

説明

Provide the steps to reproduce

  1. Run LH on https://output.jsbin.com/vulerujohi for just the canonical audit

What is the current behavior?

The audit fails with explanation Relative URL (https:// example.com)

What is the expected behavior?

The audit fails with explanation Invalid URL (https:// example.com)

Environment Information

  • Affected Channels: CLI, Node, DevTools
  • Lighthouse version: 7.0.1
  • Chrome version: 88.0.4324.96
  • Node.js version: 14.15.4
  • Operating System: macOS Mojave 10.14.6

コントリビューターガイド

調査方針
The issue is that the canonical URL audit incorrectly labels a URL with a space after 'https://' as 'Relative URL' instead of 'Invalid URL'. The fix is to locate the audit logic, likely in `lighthouse core/audits/seo/canonical.js` or similar, find the condition that outputs 'Relative URL', and change it to 'Invalid URL'. The test file for this audit should also be updated to reflect the corrected message.
技術スタック
javascriptnodejs
領域
frontendperformance
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
新着
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
familiarity with Lighthousebasic JavaScript
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
85