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

Initial attempt at direct s3 remote cache (with AuthHeadersProvider) #9137

Closed
wants to merge 1 commit into from

Conversation

borkaehw
Copy link
Contributor

@borkaehw borkaehw commented Aug 9, 2019

This is the split-up of #4889 with AuthHeadersProvider. AuthHeaderRequest.java is required in this PR but the new file is created in #9138. It needs #9138 to get merged first or we could move AuthHeaderRequest.java to here.


Removes the directy dependency of the remote and buildeventservice
packages on the authentication libraries by using the new AuthHeadersProvider.

The authandtls package is split into the "authencation/google" and
"grpc" package.

This is a step towards merging #6810.

This change is a no-op.

Removes the directy dependency of the remote and buildeventservice
packages on the authentication libraries by using the new AuthHeadersProvider.

The authandtls package is split into the "authencation/google" and
"grpc" package.

This is a step towards merging bazelbuild#6810.

This change is a no-op.
@borkaehw borkaehw changed the title remote: migrate to AuthHeadersProvider Initial attempt at direct s3 remote cache (with AuthHeadersProvider) Aug 9, 2019
@aiuto aiuto requested a review from buchgr August 21, 2019 13:11
@borkaehw
Copy link
Contributor Author

borkaehw commented Sep 5, 2019

Close this based on discussion in #4889

@borkaehw borkaehw closed this Sep 5, 2019
@borkaehw borkaehw deleted the s3-cache-w-auth branch December 6, 2019 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants