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.

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

Simplify API #108

Merged
merged 50 commits into from
May 27, 2024
Merged

Simplify API #108

merged 50 commits into from
May 27, 2024

Conversation

djc
Copy link
Owner

@djc djc commented May 23, 2024

No description provided.

djc added 28 commits May 23, 2024 10:11
@djc
Externalize Token Arc
8f201b3
@djc
Modularize logic for CustomServiceAccount::from_json()
166ae3e
@djc
Modularize logic for CustomServiceAccount::from_file()
d7b7402
@djc
Modularize logic for CustomServiceAccount::from_env()
57c3f3f
@djc
Keep HTTP clients in ServiceAccount implementations
6add5ae
@djc
Store project ID in an Arc
571caae
@djc
Move HyperExt trait into types module
7d37728
@djc
Create newtype for HTTP client
30f4530
@djc
Rename HyperClient to HttpClient
adb6376
@djc
Use HttpClient constructor directly
5ee6de1
@djc
Fold HyperExt into HttpClient
462d0af
@djc
Reorder items in types module
b0fd7c0
@djc
Deduplicate HTTP retry logic
81443da
@djc
Import more hyper types
d21fba3
@djc
Move imports to the top of the file
bde8dfd
@djc
Move consts to the bottom of modules
4100ef2
@djc
Split fetch logic from storing for CustomServiceAccount
17bec15
@djc
Make ServiceAccount::get_token() async
d3257b5
@djc
Rename and reorder ServiceAccount::token() method
8630154
@djc
Use inner RwLock to avoid thundering herd problems
e043188
@djc
Inline refresh_token() logic
e8a95b4
@djc
Force fetching project ID once for MetadataServiceAccount
c6cde6e
@djc
Rename ServiceAccount to TokenProvider
b1ffb7d
@djc
Move TokenProvider crate into crate root
3f5ec79
@djc
Make TokenProvider trait public
443b5d1
@djc
Expose trait implementers directly
d98a0d2
@djc
Move Error type into crate root
c0636b2
@djc
Simplify lint configuration
5ea9a32 
Warnings are still denied in CI.
@djc djc merged commit be16aa4 into main May 27, 2024
6 checks passed
@djc djc deleted the provider branch May 27, 2024 11:07
@djc djc mentioned this pull request May 27, 2024
Open
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.

None yet
1 participant
@djc