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

Merge V2 to main #1518

Merged
merged 6 commits into from
May 4, 2022
Merged

Merge V2 to main #1518

merged 6 commits into from
May 4, 2022

Commits on Apr 4, 2022

  1. Configuration menu
    Copy the full SHA
    0b509f2 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ea45242 View commit details
    Browse the repository at this point in the history

Commits on Apr 16, 2022

  1. Bugfix/async security check (#1512)

    * Add failing tests
    
    * Use for else construct
    Ruwann committed Apr 16, 2022
    Configuration menu
    Copy the full SHA
    c71b116 View commit details
    Browse the repository at this point in the history

Commits on Apr 27, 2022

  1. Merge v2 into main

    RobbeSneyders committed Apr 27, 2022
    Configuration menu
    Copy the full SHA
    ea020a0 View commit details
    Browse the repository at this point in the history
  2. openapi: remove JSON body second validation and type casting (#1170)

    * openapi: remove body preprocessing
    
    Body is already validated using jsonschema. There was also some type
    casting but it was wrong: e.g. not recurring deeply into dicts and lists,
    relying on existence of "type" in schema (which is not there e.g. if
    oneOf is used). Anyway, the only reason why types should be casted is
    converting integer values to float if the type is number. But this is in
    most cases irrelevant.
    
    Added an example, which did not work before this commit (echoed `{}`)
    e.g. for
    ```
    curl localhost:8080/api/foo -H 'content-type: application/json' -d
    '{"foo": 1}'
    ```
    but now the example works (echoes `{"foo": 1}`).
    
    * test with oneOf in the requestBody
    
    * remove oneof examples: superseded by tests
    
    Co-authored-by: Pavol Vargovcik <pavol.vargovcik@kiwi.com>
    p4l1ly and Pavol Vargovcik committed Apr 27, 2022
    Configuration menu
    Copy the full SHA
    792bc4d View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    94f8aae View commit details
    Browse the repository at this point in the history