mattermost / mattermost-api-reference

Mattermost API reference documentation.

Home Page:https://api.mattermost.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

OpenAPI spec lacks proper authorization securityDefinitions

devth opened this issue · comments

Since the openapi.yaml spec doesn't specify securityDefinitions generated clients will lack the proper code to set the Authorization header.

Any thoughts on this?

@jupenur I lack the domain knowable to properly triage this issue. Can you help?

@hanzei I can't say that I know much about the OpenAPI spec, and certainly not enough to write a fix, but this seems like a valid issue for sure. Authorization is only described in text in introduction.yaml and not in a format usable for client generation.

Mattermost has a lot of scopes for authentication. Hence, it a lot of work to add them all.

@devth Would you be interested on working on this?

I don't know anything about Mattermost codebase and I only just started using Mattermost itself. Just trying to use its OpenAPI spec to generate clients to consume the API. I'm happy to help test / provide feedback.

@hanzei - I'll take over this issue