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

Website fails to acknowledge that the Norway problem was fixed in YAML 1.2 #186

Open
andersk opened this issue Jul 3, 2022 · 2 comments

Comments

@andersk
Copy link

andersk commented Jul 3, 2022

Your website includes a lot of criticism of YAML’s Norway problem (where NO would parse as the boolean false), but it neglects to mention that this behavior was already removed thirteen years ago in YAML 1.2, and even includes this incorrect statement:

The most tragic aspect of this bug, however, is that it is intended behavior according to the YAML 1.2 specification.

It was intended according to the YAML 1.1 specification, but in YAML 1.2, the only recognized booleans are true, True, TRUE, false, False, FALSE.

Now, there are of course still arguments to be made—for example, some popular libraries still only support YAML 1.1. But these arguments should be made clearly and fairly.

@FichteFoll
Copy link

I landed on the "Norway problem" page today because JetBrains included a reference to it in the change notes for IntelliJ IDEA 2023.2 (search for "Norway").

Since many more people are likely going to discover this page (not necessarily the problem itself), please correct this statement.

@sciencectn
Copy link

Here are the lines in question:

The most tragic aspect of this bug, however, is that it is
*intended* behavior according to the [YAML 1.2 specification](https://yaml.org/spec/1.2.2/).
The real fix requires explicitly disregarding the spec - which
is why most YAML parsers have it.

Suggestion: drop these lines or replace them with something like this:

The most tragic aspect of this bug, however, is that it is intended behavior according to the YAML 1.1 specification.
Although this was fixed in YAML 1.2, there are popular parsers in the wild like PyYAML that still use 1.1.

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

3 participants