8000 Retry cloud cache requests by ricsam · Pull Request #1 · ricsam/rushstack · GitHub
[go: up one dir, main page]
Skip to content

Retry cloud cache requests #1

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
Draft
wants to merge 10 commits into
base: master
Choose a base branch
from
Draft

Retry cloud cache requests #1

wants to merge 10 commits into from

Conversation

ricsam
Copy link
Owner
@ricsam ricsam commented Feb 22, 2022

Summary

Details

How it was tested

@ricsam ricsam force-pushed the retry-cloud-cache-requests branch 2 times, most recently from 6f40821 to 4f49740 Compare February 22, 2022 20:52
@ricsam ricsam force-pushed the retry-cloud-cache-requests branch from 4f49740 to b0c5ebf Compare February 23, 2022 10:07
@ricsam ricsam force-pushed the retry-cloud-cache-requests branch from f054e50 to 9df26d3 Compare February 24, 2022 21:31
ricsam and others added 4 commits March 3, 2022 09:38
@ricsam
handle 400, 401, 403 the same, where such response throws an error im…
28c9f02 
…mediately if credentials are provided. Without credentials the error is silenced, e.g. if someone builds rush locally without credentials
@iclanton
Some code cleanup.
aa436fc
@ricsam
warn user if 403 error when no credentials are set
721a434
@ricsam
rush change
1803eaf
@ricsam ricsam force-pushed the retry-cloud-cache-requests branch from 08f35b0 to 1803eaf Compare March 15, 2022 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ricsam @iclanton
0