Skip to content

fix: add health endpoint to openapi#891

Merged
luislhl merged 1 commit intomasterfrom
fix/add-health-endpoint-to-openapi
Dec 8, 2023
Merged

fix: add health endpoint to openapi#891
luislhl merged 1 commit intomasterfrom
fix/add-health-endpoint-to-openapi

Conversation

@luislhl
Copy link
Contributor

@luislhl luislhl commented Dec 8, 2023

Motivation

We need the /health endpoint to be included in the OpenAPI definitions that generate the Nginx config

Acceptance Criteria

  • The Nginx config should contain the /health endpoint when generated

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@luislhl luislhl requested a review from glevco December 8, 2023 12:55
@luislhl luislhl self-assigned this Dec 8, 2023
jansegre
jansegre previously approved these changes Dec 8, 2023
@luislhl luislhl force-pushed the fix/add-health-endpoint-to-openapi branch from 491005a to 20752b6 Compare December 8, 2023 12:59
@codecov
Copy link

codecov bot commented Dec 8, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4a97f92) 85.32% compared to head (20752b6) 85.35%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #891      +/-   ##
==========================================
+ Coverage   85.32%   85.35%   +0.03%     
==========================================
  Files         283      283              
  Lines       22416    22416              
  Branches     3397     3397              
==========================================
+ Hits        19126    19134       +8     
+ Misses       2604     2600       -4     
+ Partials      686      682       -4     

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

@luislhl luislhl merged commit 628f3ad into master Dec 8, 2023
@luislhl luislhl deleted the fix/add-health-endpoint-to-openapi branch December 8, 2023 14:06
@jansegre jansegre mentioned this pull request Dec 11, 2023
2 tasks
This was referenced Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants