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

zone_settings_overrid: Remap 0rtt attribute name to zero_rtt #557

Merged
merged 1 commit into from
Dec 12, 2019

Conversation

sean-
Copy link
Contributor

@sean- sean- commented Dec 11, 2019

0rtt is not a valid HCL attribute name (the scanner begins to process this as
a digit, not a string). See my comment in #542 for details.

Fixes: #555

@patryk
Copy link
Contributor

patryk commented Dec 11, 2019

Thanks @sean- and @jacobbednarz. I got some signals about this issue internally, but had no time to fix it.

Copy link
Contributor

@patryk patryk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sean-, could you add test please?

@patryk patryk added the kind/bug Categorizes issue or PR as related to a bug. label Dec 11, 2019
`0rtt` is not a valid HCL attribute name (the scanner begins to process this as
a digit, not a string).

Fixes: cloudflare#555
@sean-
Copy link
Contributor Author

sean- commented Dec 11, 2019

@patryk Done. Ball's in your court.

@jacobbednarz
Copy link
Member

Thanks for this @sean-, appreciate it! 🤜 🤛

@jacobbednarz jacobbednarz merged commit 78a84df into cloudflare:master Dec 12, 2019
@sean- sean- deleted the gh-555 branch December 12, 2019 02:58
@sodabrew
Copy link

Thanks, looking forward to the 2.2.1 release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. kind/documentation Categorizes issue or PR as related to documentation. size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aren't attributes supposed to be identifiers that don't start with a number?
4 participants