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.

Sign up for GitHub

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

Jump to bottom

fix health check for rest api #1793

Merged
merged 1 commit into from Mar 27, 2024
Merged

fix health check for rest api #1793

merged 1 commit into from Mar 27, 2024

Conversation

pxp928
Copy link
Collaborator

@pxp928 pxp928 commented Mar 27, 2024

Description of the PR

fix health check for rest api

PR Checklist

  • All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
  • All new changes are covered by tests
  • If GraphQL schema is changed, make generate has been run
  • If OpenAPI spec is changed, make generate has been run
  • If collectsub protobuf has been changed, make proto has been run
  • All CI checks are passing (tests and formatting)
  • All dependent PRs have already been merged

@pxp928
fix health check for rest api
d9a854a 
Signed-off-by: pxp928 <parth.psu@gmail.com>
@pxp928 pxp928 requested review from mihaimaruseac and jeffmendoza and removed request for mihaimaruseac March 27, 2024 19:11
@kodiakhq kodiakhq bot merged commit ef1c2c9 into guacsec:main Mar 27, 2024
8 checks passed
arorasoham9 pushed a commit to arorasoham9/guac that referenced this pull request Apr 1, 2024
@pxp928 @arorasoham9
fix health check for rest api (guacsec#1793)
2ec0e09 
Signed-off-by: pxp928 <parth.psu@gmail.com>
arorasoham9 pushed a commit to arorasoham9/guac that referenced this pull request Apr 4, 2024
@pxp928 @arorasoham9
fix health check for rest api (guacsec#1793)
ae87655 
Signed-off-by: pxp928 <parth.psu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mihaimaruseac mihaimaruseac mihaimaruseac approved these changes

@jeffmendoza jeffmendoza jeffmendoza approved these changes

Assignees
No one assigned
Labels
size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pxp928 @mihaimaruseac @jeffmendoza