Skip to content

Adding Swagger

Jason requested to merge adding-swagger into master

All Submissions:

  • [YES] Have you followed our code review guidelines?
  • [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
  • [YES] I have updated the documentation accordingly.
  • [NA] I have added tests to cover my changes.
  • [YES] All new and existing tests passed.
  • [YES] My code follows the code style of this project.
  • [NO] I ran lint checks locally prior to submission.

What is the current behavior?

  • No swagger for file service
  • Azure Istio auth policy has wrong path for allowing swagger resources without authentication
  • Azure chart has the wrong default tag(it is "lastest", when it should be "latest")

What is the new behavior?

  • Swagger can be found at the /api/file/v2/swagger endpoint for the service.
  • Azure Istio auth policy has the correct path for swagger resources without authentication
  • Azure chart has latest as default tag

Does this introduce a breaking change?

  • [NO]

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

Other information

This change was tested locally and on the Azure dev environment.

Edited by Jason

Merge request reports