Add iOS CI back in #599
Closed
Add iOS CI back in #599
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* make Credential Provider tests async * Make all AWSClient tests async * Make endpoint, service, middleware logging tests async * Make paginate, payload and waiter tests async * Last non async call
* Remove SotoCrypto * Remove @available/format
* Remove eventLoop from execute commands * Remove commented out code
* Remove EventLoop paginate * Remove Async from PaginateTests * Remove @available(macOS...
* Remove EventLoop version of waiters * Speed up waiter tests
* Add ExpiringValue actor * Replace EventLoop endpoint discovery * Rename AWSClient+EndpointDiscovery+async * Add tests for ExpiringValue, add comments * more comments * Remove Date.now * Add logging for endpoint discovery * Update Sources/SotoCore/AWSClient+EndpointDiscovery.swift Co-authored-by: Tim Condon <[email protected]> * Add comment about endpoint renewal --------- Co-authored-by: Tim Condon <[email protected]>
* Remove AWSClient.execute calls with EventLoop * Fix STSAssumeRole after removing EventLoop execute * Fix AWSClient waiter code * Remove commented out code * Remove EL credential functions * Merge async files with parent implementations * Delete AWSClient+Waiter+async.swift * Fix up paginate code * Remove EventLoop from execute closure * Changes requested in PR * Update Sources/SotoCore/Waiters/AWSClient+Waiter.swift Co-authored-by: Tim Condon <[email protected]> --------- Co-authored-by: Tim Condon <[email protected]>
* Make credential providers async * Fix tests Also add support for initialising an ExpiringValue with an updating closure. Needed to add another state to get this working. * getTaskProviderTask -> getCredentialProviderTask * Format copyright 2023 * validation changes * Fix issues with credential providers Add error state for ExpiringValue where previous getValue returned an error * Don't throw error during credential provider selection If credential provider fails to provide a credential don't throw an error when you are shutting it down * Shutdown test for rotating credential provider And fixed a bug * Added testDeferredCredentialProviderSetupShutdown
* Use async HTTPClient calls Currently streaming is not working * Comments * Extract HTTPBody out so it can be used in request * Flush body, when not expecting anything * Remove eventLoop from function comment headers
* Add support for returning streamed responses * Remove old response streaming fiunctions * Remove unused function * Comment update
* Add support for request streaming * Remove commented out code
* S3 signed requests * Remove old streaming code and AWSPayload * comments * Add todo comment * bufferSequence -> asyncSequence * compile fix * Make HTTPBody(asyncSequence:length:) public * Add helper init for AWSResponseError * Make ByteBufferAsyncSequence public * if response body is buffer then print as string * Compile fix
* Rename HTTPBody to AWSHTTPBody * Add AWSResponse to Codable userInfo * Add decode functions for AWSResponse * ResponseContainer for decoding * Fix tests after userInfo changes * Remove DictionaryDecoder Now we pass the response via uesrInfo to the decoder and decode header, body and payload in the init(from) function, we can use the JSONDecoder for decoding responses. * Fix tests * Re-organise root element code * Fix after merge * Update Sources/SotoCore/Encoder/ResponseContainer.swift Co-authored-by: Tim Condon <[email protected]> --------- Co-authored-by: Tim Condon <[email protected]>
* Add EventStream type * Split event stream code into two files * If response is chunked assume it is raw * Fix test errors * Fix a couple strict concurrency issues I missed * Add tests for EventStreams * Changes from PR review
* reorder execute * process response in execute * Collapse AWSHTTPResponse and AWSResponse into one * Collpase AWSRequest and AWSHTTPRequest into one * Add HTTP to request and response type names * Remove new middleware until we need it No need to include in this PR * Remove re-org to make PR review easier * Changes from PR comments * Another fix
* import SotoXML as implementation only * formatting
* Re-org request and response middleware So these are applied in same function * MiddlewareStack * Multiple AWSClient inits * Add Error handling middleware * Add retry middleware * Add fix up for S3 notFound errors * Add endpoint discovery middleware * Endpoint middleware fixes * Add streaming parameter to stop collation of body * Add fixup for getBucketLocation * @preconcurrency import struct Foundation.CharacterSet * formatting * Update Sources/SotoCore/AWSClient.swift Co-authored-by: Tim Condon <[email protected]> * comments --------- Co-authored-by: Tim Condon <[email protected]>
* Tracing middleware with basic attributes * Add test for tracing * Update TracingMiddleware.swift
* Add RequestEncodingContainer, pass to encoders * Fixup checksum tests * Get rid of all uses of Mirror, fix Payloads * Fix requests with payload and headers * Move URL building to RequestEncoderContainer * Changes to get services woking * Add EventPayload * Fix up tests * Conform AWSEventStream to decodable * Make RequestContainer code inlinable * Rename decode functions to align with RequestContainer * comments * Remove unused code * Add SingleValueEncodingContainer.encodeNil * add file header * Add more tests * More tests
Given we have removed SotoCrypto maybe we should remove the tests
* Use JSON date decoding strategy .iso8601 * Use ISO8601DateFormatter in DateCoders * Use ISO8601DateFormatter in Encoders/Decoders * Add test for header date encoding Ensure header dates use http timedate format
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 7.x.x #599 +/- ##
==========================================
+ Coverage 77.14% 80.43% +3.29%
==========================================
Files 87 77 -10
Lines 8557 6465 -2092
==========================================
- Hits 6601 5200 -1401
+ Misses 1956 1265 -691 ☔ View full report in Codecov by Sentry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.