commercetools / commercetools-api-reference

commercetools API reference documentation
http://docs.commercetools.com/
57 stars 27 forks source link

(spec): oauth.raml "securedBy" property must be a list #281

Closed osxhacker closed 1 year ago

osxhacker commented 1 year ago

As per the RAML specification:

The value assigned to the securedBy node MUST be a list of any of the security schemes previously defined in the securitySchemes node of RAML document root.

When securedBy in oauth.raml is updated from:

  securedBy: basic

To be:

  securedBy: [ basic ]

The definition is fully conformant and immediately usable 👍 .

🏴‍☠️

jenschude commented 1 year ago

Oh nice catch.

Also the oauth.raml is more like a test. As typically you don't generate a oauth client. Nevertheless will update it

osxhacker commented 1 year ago

No worries Jens, I just was in the neighborhood and opened a ticket before I forgot again 😃 . With this one edit, scraml was able to generate OAuth2 compatible scaffolding a while back.

jenschude commented 1 year ago

It's fixed now