8000 Comparing spec-first:main...rickpr:main · spec-first/connexion · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: spec-first/connexion
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: rickpr/connexion
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
  • 7 commits
  • 19 files changed
  • 5 contributors

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 authored 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. 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 authored Apr 27, 2022
    Configuration menu
    Copy the full SHA
    792bc4d View commit details
    Browse the repository at this point in the history

Commits on May 4, 2022

  1. Configuration menu
    Copy the full SHA
    cff0310 View commit details
    Browse the repository at this point in the history

Commits on May 6, 2022

  1. Fix OpenAPI parameters containing other parameters

    OpenAPI parameters can be unspecified and sometimes contain other
    parameters. The current behavior is to assume it's a bracket parameter
    and nest the containing parameter within the contained parameter, which
    breaks the schema as the original parameter now seems to contain a
    nested object.
    
    We can avert this by checking for the presence of a '[' in the
    parameter.
    Ricardo Piro-Rael committed May 6, 2022
    Configuration menu
    Copy the full SHA
    cc381d1 View commit details
    Browse the repository at this point in the history

Commits on May 9, 2022

  1. Configuration menu
    Copy the full SHA
    05d64da View commit details
    Browse the repository at this point in the history
Loading
0