Add X-Correlator and other fixes

[jlurien]

What type of PR is this?

• correction

What this PR does / why we need it:

• Add x-correlator to all requests and responses
• Added some missing descriptions to silent linting warnings
• Remove weird use of allOf for example

Which issue(s) this PR fixes:

Fixes #271
Fixes #282

Changelog input

- Added x-correlator header to requests and responses

[linux-foundation-easycla]

• ❌ - login: @jlurien / name: Jose Luis Urien . The commit (bbf091c, bc9df92) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

[eric-murray]

Parameter name in the Design Guidelines is X-Correlator. I think the YAML should be consistent with the guidelines.

[jlurien]

Parameter name in the Design Guidelines is X-Correlator. I think the YAML should be consistent with the guidelines.

Yes, situation is that almost all CAMARA APIs are specifying x-correlator in lower case, so we either fix the guidelines or fix all the specs. I will raise this in Commonalities:

camaraproject/Commonalities#191

[hdamker]

Yes, situation is that almost all CAMARA APIs are specifying x-correlator in lower case, so we either fix the guidelines or fix all the specs.

We shouldn't wait with the merge of the PR until this is decided in Commonalities, as that will block the work on the planned API split. Therefore my proposal: we go here with lower case x-correlator (it's only "wip") but adding a new issue that we have to observe the outcome of camaraproject/Commonalities#191 and, if the decision there will be for upper case have to adapt.

@eric-murray @RandyLevensalor: if that makes sense for you, please approve the PR.

The second point, about "Missing CLA Authorisation": CAMARA Board has decided to postpone the EasyCLA introduction at least until the next board meeting. Therefore I asked @wrathwolf to deactivate the check also for our repository for the time being.