Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Schema does not define code method for verification #260

Closed
4 of 6 tasks
DASPRiD opened this issue Mar 21, 2023 · 1 comment · Fixed by ory/kratos#3213
Closed
4 of 6 tasks

Schema does not define code method for verification #260

DASPRiD opened this issue Mar 21, 2023 · 1 comment · Fixed by ory/kratos#3213
Labels
bug Something is not working.

Comments

@DASPRiD
Copy link

DASPRiD commented Mar 21, 2023

Preflight checklist

Describe the bug

At the time of writing (schema version 1.1.23) the schema body for updateVerificationFlowBody is set to a discriminator which only supports the link type, pointing to the updateVerificationFlowWithLinkMethod component.

There is already a updateVerificationFlowWithCodeMethodBody defined, but it is not referenced.

This causes issues for me, as I'm using an SDK client generated from a more recent OpenAPI generator version, which actually validates the input I give it against the schema. Thus I'm unable to submit any verification requests.

Reproducing the bug

  1. Generate a JavaScript SDK with the current version of OpenAPI Generator or use https://www.npmjs.com/package/@dasprid/ory-client-fetch
  2. Try to submit a verification flow with method set to code
  3. See the SDK rejecting the body.

Relevant log output

No response

Relevant configuration

selfservice:
  flows:
    recovery:
      enabled: true
      ui_url: http://localhost:4000/recovery
      use: code

Version

1.1.23

On which operating system are you observing this issue?

Linux

In which environment are you deploying?

Docker Compose

Additional Context

No response

@jonas-jonas
Copy link
Member

Thanks for the report, will be fixed with ory/kratos#3213

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants