Header definition clashing with naming conventions

[acabarbaye]

Describe the bug
When describing header parameters which do not follow any naming convention such as Accept-Version or if-none-match, I get naming convention errors for PATH and Properties

To Reproduce
e.g.

 parameters:
    acceptVersionParam:
      name: Accept-Version
      in: header

results in
*ERROR* in path GET /... 'Accept-Version' -> parameter should be in camelCase
*ERROR* in path GET / 'Accept-Version' -> parameter should be in camelCase

Expected behavior
No error raised for headers as we have no control over their naming

[JFCote]

Hi @acabarbaye ,

There will be a new parameter for this, it has been added last week with this issue: #135

We should release a new version with this improvement somewhere next week. In the meantime, you can compile the code of the project in the master.

Let me know if this new feature is ok for your use case.

[JFCote]

@acabarbaye It has been released in version 1.5. Please let us know if it's working for you and we will close this issue :)

[acabarbaye]

@JFCote : has version 1.5 made it to maven central or Jcenter?

   > Could not find org.openapitools.openapistylevalidator:openapi-style-validator-cli:1.5.
     Searched in the following locations:
       - https://jcenter.bintray.com/org/openapitools/openapistylevalidator/openapi-style-validator-cli/1.5/openapi-style-validator-cli-1.5.pom
       - https://jcenter.bintray.com/org/openapitools/openapistylevalidator/openapi-style-validator-cli/1.5/openapi-style-validator-cli-1.5-all.jar
       - https://repo.maven.apache.org/maven2/org/openapitools/openapistylevalidator/openapi-style-validator-cli/1.5/openapi-style-validator-cli-1.5.pom
       - https://repo.maven.apache.org/maven2/org/openapitools/openapistylevalidator/openapi-style-validator-cli/1.5/openapi-style-validator-cli-1.5-all.jar

Just to be sure I understand the new feature, for my case, I should define the following options?

{... 
  "validateNaming": true,
  "ignoreHeaderXNaming": true,
  "headerNamingStrategyConvention": HyphenCase,
}

[JFCote]

@acabarbaye Oh! It seems the release didn't make its way to Maven. I will check with @jmini who did the release. Will update here when it's ready.

As for your problem, I now see that it will not fix your issue because the naming of your header is not the same. At first glance, I thought that it was the same thing as #135

What you need would be an option to completely ignore the naming of headers. I will think about it. Maybe a new "naming strategy convention" that will be Anything?

[JFCote]

@acabarbaye There was a problem during the release, the 1.5 was lost. It has been generated again and it's there! https://repo1.maven.org/maven2/org/openapitools/openapistylevalidator/openapi-style-validator-lib/1.5/

[acabarbaye]

@acabarbaye There was a problem during the release, the 1.5 was lost. It has been generated again and it's there! https://repo1.maven.org/maven2/org/openapitools/openapistylevalidator/openapi-style-validator-lib/1.5/

Perfect.
I will give it another try but I think your suggestion of having a Anything option would make a lot of sense

[JFCote]

@acabarbaye I created a PR last friday. It just needs to be reviewed and the feature should work.

[gervaisb]

I have documented some official headers like Accept-Encoding and If-None-Match. However they did not pass any naming convention. I tried AnyCase but received a NullPointerException.

Anyway, Can't we exclude official headers from the naming convention validation or include a naming convention that match the official "Hyphen-Upper-Case" ? And/or maybe ensure that official headers match the official naming convention ?

Edit; The NullPointerException is gone with v1.6.

[JFCote]

@gervaisb Just to be sure, is it ok now or do you still have problems with the AnyCase naming convention?