Skip to content

Conversation

@ks-yim
Copy link
Contributor

@ks-yim ks-yim commented Sep 5, 2023

Modification:

A new channel access token type, stateless channel access token, was added and now brought to public as described in
https://developers.line.biz/en/reference/messaging-api/#issue-stateless-channel-access-token

This commit adds the OpenAPI definition for the newly added token type.

Modification:

A new channel access token type, `stateless channel access token`, was
added and now brought to public as described in
https://developers.line.biz/en/reference/messaging-api/#issue-stateless-channel-access-token

This commit adds the OpenAPI definition for the newly added token type.
Copy link
Contributor

@Yang-33 Yang-33 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! I left one minor comment but LGTM!

application/x-www-form-urlencoded:
schema:
oneOf:
- $ref: '#/components/schemas/IssueChannelTokenByJWTAssertionRequest'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably these names can be
IssueStatelessChannelTokenByJWTAssertionRequest
and
IssueStatelessChannelTokenByClientSecretRequest
because we don't use same schema basically...?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed the schema names as per your comment, thanks!

@Yang-33
Copy link
Contributor

Yang-33 commented Sep 6, 2023

Thank you very much to update this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants