Skip to content
This repository was archived by the owner on Aug 4, 2022. It is now read-only.

Ping AOS whenever an AOSClient is constructed#124

Merged
AndreasHassing merged 3 commits intomicrosoft:mainfrom
Antonstockmarr:PingAosOnConstruct
Jan 10, 2022
Merged

Ping AOS whenever an AOSClient is constructed#124
AndreasHassing merged 3 commits intomicrosoft:mainfrom
Antonstockmarr:PingAosOnConstruct

Conversation

@Antonstockmarr
Copy link
Copy Markdown
Contributor

@Antonstockmarr Antonstockmarr commented Nov 23, 2021

Make sure that the AOS is warm whenever a new client is created. Improve readability of errors caused by timeout.
#patch
resolves #106
image

Make sure that the AOS is warm whenever a new client is created.  Improve readability of errors caused by timeout.
#patch
@Antonstockmarr Antonstockmarr changed the title Ping AOS whenever a AOSClient is constructed Ping AOS whenever an AOSClient is constructed Nov 23, 2021
@AndreasHassing
Copy link
Copy Markdown
Contributor

@Antonstockmarr is this good to go - tested and validated?

@Antonstockmarr
Copy link
Copy Markdown
Contributor Author

@Antonstockmarr is this good to go - tested and validated?

Yes, tested, validated and ready to go in.

This change will make sure that the AOS is responsive before making any requests, and if the connection times out, the error will say so explicitly.

Copy link
Copy Markdown
Contributor

@AndreasHassing AndreasHassing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! :shipit:

@AndreasHassing AndreasHassing merged commit 9a40703 into microsoft:main Jan 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ping AOS before making requests

3 participants