Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: allow disabling swagger ui and disable it by default #1026

Merged
merged 3 commits into from
Dec 11, 2024

Conversation

fmartingr
Copy link
Member

Adds a new variable to enable the Swagger UI SHIORI_HTTP_SERVE_SWAGGER.

Swagger UI is now disabled by default.

Closes #1025

@fmartingr fmartingr self-assigned this Dec 11, 2024
Copy link

codecov bot commented Dec 11, 2024

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 34.71%. Comparing base (4aa0f51) to head (d84a169).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
internal/http/server.go 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1026      +/-   ##
==========================================
- Coverage   34.72%   34.71%   -0.02%     
==========================================
  Files          61       61              
  Lines        5324     5326       +2     
==========================================
  Hits         1849     1849              
- Misses       3251     3253       +2     
  Partials      224      224              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fmartingr fmartingr merged commit 87bc7a8 into master Dec 11, 2024
15 of 17 checks passed
@fmartingr fmartingr deleted the feat/swagger-route-config branch December 11, 2024 18:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Config flag to enable disable swagger ui
1 participant