search

Azure / bicep-types-az

Bicep type definitions for ARM resources
MIT License
84 stars 27 forks source link

typo on property IgnoreHttpsStatusCode in Microsoft.Insights/webtests@2018-05-01-preview #1420

Open slavizh opened 2 years ago

slavizh commented 2 years ago

Bicep version Bicep CLI version 0.5.6 (5f2f88f0f0)

Describe the bug The property IgnoreHttpsStatusCode documented here https://docs.microsoft.com/en-us/azure/templates/microsoft.insights/2018-05-01-preview/webtests?tabs=bicep has typo. The actual property is IgnoreHttpStatusCode instead of IgnoreHttpsStatusCode. There is extra character s after Http. So when you use it in Bicep template the actual property value will not be applied as such property does not exists and you have to add the property as IgnoreHttpStatusCode in order to apply it.

TiagoPT commented 1 year ago

Hello,

I have the same problem but in version Microsoft.Insights/webtests@2022-06-15. In the docs it's defined as IgnoreHttpsStatusCode, but in VS Code if I use that I get the following warning:

The property "IgnoreHttpsStatusCode" is not allowed on objects of type "WebTestPropertiesValidationRules". Did you mean "IgnoreHttpStatusCode"?bicep(BCP089)

If I fix it and have it as IgnoreHttpStatusCode (Http vs Https) then no warning is issued but running bicep build in the pipeline, the warning is issued again. I am using bicep.exe 0.11.1 x64.

Warning BCP089: The property "IgnoreHttpStatusCode" is not allowed on objects of type "WebTestPropertiesValidationRules". Did you mean "IgnoreHttpsStatusCode"?

Can you please confirm what's the correct value?

TomasMalecek commented 1 year ago

Just to be sure, in my template, I specified both the spelling variants of the property with the same value and suppressed the warning. Hopefully, the inconsistency gets fixed soon and I can remove one of the variants and the suppression.