-
Notifications
You must be signed in to change notification settings - Fork 17
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
Cognito credential provider support #306
Open
xiazhvera
wants to merge
112
commits into
iot
Choose a base branch
from
cognito_support
base: iot
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+387
−74
Open
Changes from all commits
Commits
Show all changes
112 commits
Select commit
Hold shift + click to select a range
554c9b5
update crt error
xiazhvera e377204
fix lint
xiazhvera fdcc698
update error validation
xiazhvera 17022d6
Merge branch 'iot' of https://github.com/awslabs/aws-crt-swift into m…
xiazhvera 312ed71
add swift error space
xiazhvera c6fb2d1
fix lint
xiazhvera 24e4c21
add ios mqtt sample
xiazhvera abec93c
update readme
xiazhvera 6a67040
fix lint
xiazhvera 0bd46b9
clean up app
xiazhvera 7d6a6cb
move samples -> test
xiazhvera 5c2dbd3
move project data
xiazhvera 1607745
fix readme and .gitignore
xiazhvera 425260a
Merge branch 'iot' of https://github.com/awslabs/aws-crt-swift into m…
xiazhvera c0e4aa6
fix merge
xiazhvera d6a8a13
Merge branch 'iot' into mqtt_test_app
sbSteveK aad4d76
allow tls
sbSteveK e7e603d
fix logging in test app due to main merge changes
sbSteveK 8a7f52a
Merge branch 'mqtt_test_app' into secitem_bindings
sbSteveK f248f53
add setting of secitem options
sbSteveK 0406905
aws-c-io nw_secitem branch
sbSteveK d261d8a
latest aws-c-io nw_secitem
sbSteveK 08308e8
update aws-c-io
sbSteveK 54ff235
use AWS_USE_SECITEM define
sbSteveK e19519a
add eventloop types to other platforms
sbSteveK b338b5d
lint error and differetiate between macOS and iOS/tvOS
sbSteveK 9c2ba7f
add missing endif
sbSteveK 827c071
try excluding dispatch queue related source files
sbSteveK e0dfcad
CRT changes
sbSteveK b507465
aws-c-io changes
sbSteveK a50380c
update aws-c-io changes
sbSteveK fb1b501
remove test error
sbSteveK 1f3cee0
latest submodules
sbSteveK 81946d3
update aws-c-io
sbSteveK 78e5ac9
use latest aws-c-io
sbSteveK b9a6ed2
add defines in package.Swift
sbSteveK b2b21d1
latest aws-c-io
sbSteveK e6a9506
add defines to test and secitem options test
sbSteveK 610e716
defines for swift test target
sbSteveK 06451a0
add defines to tests in swift
sbSteveK 9b177d2
remove watchOS testing
sbSteveK 97ed422
pull latest aws-c-io changes
sbSteveK 9a50b8d
update aws-c-io again
sbSteveK 5e5bf33
update aws-c-io
sbSteveK 599ce74
Merge branch 'iot' into mqtt_test_app
sbSteveK 0960792
merge latest main
sbSteveK 52a458a
latest wip aws-c-io
sbSteveK e80b8a4
pull latest changes
sbSteveK 2af72b8
update aws-c-io
sbSteveK aac5d89
updated submodules
sbSteveK 1baa40b
update submodule
sbSteveK 986c937
pull in latest changes from merged nw_socket
sbSteveK 6e34fc8
latest aws-c-io
sbSteveK 9a55e43
make HostResolver init public
sbSteveK 468d6d6
update HostResolver to make init public
sbSteveK 7e54f81
Merge branch 'iot' into mqtt_test_app
sbSteveK 4539fa3
Merge branch 'iot' into mqtt_test_app
sbSteveK 6c5da21
Merge branch 'mqtt_test_app' into secitem_bindings
sbSteveK b3be803
add cognito credential provider support
xiazhvera e9d3149
Merge branch 'iot' of https://github.com/awslabs/aws-crt-swift into s…
xiazhvera 70e4dd6
Merge branch 'secitem_bindings' of https://github.com/awslabs/aws-crt…
xiazhvera 732e885
fix test name
xiazhvera af00a88
fix shutdown flag
xiazhvera 3e17395
skip the test if env is not set
xiazhvera d4c6b07
swift lint format
xiazhvera cddd848
test with new env var
xiazhvera 3ac6ec0
set use xcodebuild
xiazhvera 5b3d2a5
remove cognito test
xiazhvera 5eb25e9
waiting for shutdown credential provider
xiazhvera eb57164
add scope to make sure the provider released
xiazhvera d982864
setup ci for xcodebuild
xiazhvera 3b3ded9
Merge branch 'iot_ci_env_var' of https://github.com/awslabs/aws-crt-s…
xiazhvera 95487a4
[WIP]cognito test
xiazhvera 59ac4e4
WIP remove test vars
xiazhvera 4b84fcb
add cognito connecton test
xiazhvera 1895e9a
fix spacing...
xiazhvera e7612ba
change aws-c-io to nw_secitem_latest_socket
sbSteveK c66b36d
update aws-c-io
sbSteveK 137ebf0
Separate trust store override into 2 new funcs that take one or the o…
sbSteveK bd5e186
Add data CA override func
sbSteveK 7771fae
Merge branch 'secitem_bindings' of https://github.com/awslabs/aws-crt…
xiazhvera 7a63d6b
latest submodules and nw_secitem branch of aws-c-io
sbSteveK 2bf67e4
Merge branch 'secitem_bindings' of https://github.com/awslabs/aws-crt…
xiazhvera 1a5b8ed
merge with latest iot
sbSteveK 2632913
latest submodules
sbSteveK 7899729
Merge branch 'secitem_bindings' of https://github.com/awslabs/aws-crt…
xiazhvera 61931eb
revert aws-c-cal to the one being used in main
sbSteveK 62c87aa
Merge branch 'secitem_bindings' of https://github.com/awslabs/aws-crt…
xiazhvera 2986c96
remove test sample app
sbSteveK cd00d31
Merge branch 'secitem_bindings' of https://github.com/awslabs/aws-crt…
xiazhvera 65cffef
WIP: enable tests on ios and tvos, update cognito credential provider
xiazhvera 35957bc
fulfill the shutdown callback on failed tests
xiazhvera e2b816a
revert and disable secitem test
xiazhvera ff5a4de
test against nw_secitem_test
xiazhvera 52a2cc2
fix secitem compile error
xiazhvera 0e39690
test extend the timeout
xiazhvera 2005830
TEST: enable trace log for ios 13
xiazhvera 60bccb8
TEST: more prints
xiazhvera 5052f42
fix spacing
xiazhvera 1cc8aa8
revert spacing with cognito tests
xiazhvera c641970
more prints
xiazhvera 876b095
update base socket & log level set to error
xiazhvera 8df642b
fix aws-c-io
xiazhvera 95fda11
more prints and nul out the websocket transform?
xiazhvera 76473c6
extend the connection aiting time for ios13
xiazhvera da595de
Merge branch 'iot' of https://github.com/awslabs/aws-crt-swift into c…
xiazhvera 7f65a04
cleanup
xiazhvera 7ea75fd
remove macos-13 with ios
xiazhvera 5b0866b
remove macos-13 with ios
xiazhvera 6f7479a
remove macos-13 with ios
xiazhvera 327d133
update aws-c-io to nw_secitem
xiazhvera 75e130d
update test timeout time
xiazhvera File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
import Builder | ||
|
||
class CrtCiPrepXCodebuild(Builder.Action): | ||
def run(self, env): | ||
env.shell.setenv("TEST_RUNNER_AWS_TESTING_STS_ROLE_ARN", env.shell.get_secret("aws-c-auth-testing/sts-role-arn")) | ||
actions = [ | ||
Builder.SetupCrossCICrtEnvironment(use_xcodebuild=True) | ||
] | ||
return Builder.Script(actions, name='crt-ci-prep-xcodebuild') |
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
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
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
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't looked at the PR but can we avoid adding this action in crt-swift? We don't need to test the Cognito credential provider functionality in bindings; we can just create/destroy it for simple binding tests. The MQTT-specific tests should be in the IoT SDK.
This action makes it harder to run these tests locally. I think we will likely need this action for MQTT unit tests but we should try to avoid it if possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This script change is used to allow passing environment variables into xcodebuild tests. Regardless of the Cognito credential provider changes, we still need it for the unit tests that requires environment variables.