GoogleChrome/lighthouse

Canonical URL audit mistakes invalid URLs for relative ones

Fechada

#12.008 aberto em 26 de jan. de 2021

Ver no GitHubGuia do colaborador
 (7 comentários) (0 reação) (0 responsável)JavaScript (9.278 forks)batch import
P3good first issue

Métricas do repositório

Stars
 (27.768 estrelas)
Métricas de merge de PR
 (Mesclagem média 18d 18h) (40 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
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.
Pilha de tecnologia
javascriptnodejs
Domain
frontendperformance
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
familiarity with Lighthousebasic JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85