-
Notifications
You must be signed in to change notification settings - Fork 92
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
Fix expired sas token on getting job result #593
Conversation
@ArthurKamalov, can you try to merge the |
azure-quantum/tests/unit/recordings/test_job_get_results_with_expired_sas_token.yaml
Dismissed
Show dismissed
Hide dismissed
@kikomiss, I've made the following changes that were causing non-deterministic behavior in the test case:
I tested in two different machines that both recording mode and playback mode are working. |
This PR adds verification of the sas token's expiration time, to refresh it, if necessary.
fixes #553