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

Added api/healthy command #121

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

GregoryLand
Copy link

  • Added api/healthy
  • Added tests for api/ready

Added api/healthy
Added tests for api/ready
@xaoc7
Copy link
Contributor

xaoc7 commented Aug 2, 2024

@GregoryLand Not sure if relevant for you but suggested changes are related to functionalities that are not part of a released version of Aptly Server.

@GregoryLand
Copy link
Author

Yes but I needed it. So commiting it to upstream where it could be pulled in one day seemed prudent. It also fails with an exception if the endpoint doesn't exist.

Corrected defect in misc.healthy with 404 codes.
Corrected defective unittest for misc.healthy for 404 codes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants