Strings that could be interpreted as sexagesimal should always be emitted quoted

Aberta
#387 3 comentários 1 reação 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
45/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
ruby, yaml
Domínio
tooling

Direção de pesquisa

Comece rastreando o caminho de emissão de to_yaml para strings e compare-o com os exemplos de Ruby 2.5 e 2.6 na issue. Verifique como os parsers YAML 1.1 interpretam os valores emitidos e, em seguida, confirme que strings semelhantes a valores sexagesimais, como endereços MAC, sejam colocadas entre aspas sem alterar as demais saídas de strings.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Not doing so causes parsers that follow the yaml 1.1 spec (crazy though it might be), to load certain unquoted mac addresses as integers.

IMO https://github.com/ruby/psych/issues/265 wasn't actually a bug. It was following the 1.1 spec. (To make things more complicated, sexagesimal was dropped from yaml in the 1.2 spec)

Ruby 2.5 and 2.6 emit mac addresses, (but not shorter strings it thinks are more likely to be sexagesimal), unquoted.

2.6.0 :002 > '52:54:00:44:49:56'.to_yaml
 => "--- 52:54:00:44:49:56\n...\n" 
2.6.0 :003 > '44:49:56'.to_yaml
 => "--- '44:49:56'\n"

ruby < 2.5 then parse them as integers.

2.4.5 :006 >   YAML.load('macaddress: 52:54:00:44:49:56')
 => {"macaddress"=>12465480}

Here is an actual example of where the change in handling of sexagesimal has been a problem.

Linguagem predominante
Ruby
Estrelas
596
Forks
224
Merge médio
5h 47min
PRs com merge (30d)
3

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de ruby/psych

Todas as issues de ruby/psych

Issues semelhantes

Mais issues de Ruby

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.