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

Backport of Fix multiple OpenAPI generation issues with new AST-based generator into release/1.12.x #19718

Commits on Mar 23, 2023

  1. Fix multiple OpenAPI generation issues with new AST-based generator (#…

    …18554)
    
    * Regexp metacharacter `.` should be escaped when used literally
    
    The paths including `/.well-known/` in the Vault API could currently
    technically be invoked with any random character in place of the dot.
    
    * Replace implementation of OpenAPI path translator with regexp AST-based one
    
    * Add changelog
    
    * Typo fix from PR review - thanks!
    
    Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com>
    
    * Add comment based on review feedback
    
    * Change style of error handling as suggested in code review
    
    * Make a further tweak to the handling of the error case
    
    * Add more tests, testing cases which fail with the previous implementation
    
    * Resolve issue with a test, and improve comment
    
    ---------
    
    Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com>
    maxb and averche committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    9c4249b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    1d09ecd View commit details
    Browse the repository at this point in the history
  3. Merge branch 'release/1.12.x' into backport/openapi-generation-regexp…

    …-syntax/vastly-useful-dingo
    averche authored Mar 23, 2023
    Configuration menu
    Copy the full SHA
    0404691 View commit details
    Browse the repository at this point in the history