Skip to content

Added api/ready command#120

Open
GregoryLand wants to merge 1 commit into
gopythongo:masterfrom
GregoryLand:add-api-ready
Open

Added api/ready command#120
GregoryLand wants to merge 1 commit into
gopythongo:masterfrom
GregoryLand:add-api-ready

Conversation

@GregoryLand

@GregoryLand GregoryLand commented Jul 27, 2024

Copy link
Copy Markdown
  • Added api/ready
  • Added tests for api/ready
  • Added fix for BaseApiClient.do_get. 503 status codes were getting turned into ApltyAPIExceptions. The new api/ready uses 503 as its status code when aptly is not ready.

* Added api/ready
* Added tests for api/ready
* Added fix for BaseApiClient.do_get. 503 status codes were getting
  turned ApltyAPIExceptions.  The new api/ready uses 503 as its status
  code when aptly is not ready.
@xaoc7

xaoc7 commented Aug 2, 2024

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
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.

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