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

feat: Upgrade AppSyncRTClient 1.8.0 and enable logs #519

Merged
merged 3 commits into from
Feb 25, 2022

Conversation

lawmicha
Copy link
Contributor

@lawmicha lawmicha commented Feb 16, 2022

Issue #, if available:

Description of changes:
Because we are using AppSyncRealTimeClient.logLevel which is introduced in 1.8, we should also set the minimum dependency on it to 1.8. This PR propagates the logLevel set when using AppSync SDK down to the logLevel of AppSyncRTClient.

Developers using AppSync SDK uses the SDK's logging mechansim AWSDDLog, ie.

AWSDDLog.sharedInstance.logLevel = .verbose
AWSDDLog.sharedInstance.add(AWSDDTTYLogger.sharedInstance)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@lawmicha lawmicha marked this pull request as ready for review February 16, 2022 20:39
Base automatically changed from feat/appsyncrt-1.7 to main February 21, 2022 18:10
@lawmicha lawmicha self-assigned this Feb 25, 2022
@lawmicha lawmicha force-pushed the feat/appsync-loglevel branch from 798ba34 to 62862a2 Compare February 25, 2022 19:07
@lawmicha lawmicha force-pushed the feat/appsync-loglevel branch from 62862a2 to 3147fae Compare February 25, 2022 19:16
@lawmicha
Copy link
Contributor Author

rebased and reran integration tests - all passing

@lawmicha lawmicha merged commit b3acd23 into main Feb 25, 2022
@lawmicha lawmicha deleted the feat/appsync-loglevel branch February 25, 2022 19:24
karim-alweheshy pushed a commit to chocoapp/aws-mobile-appsync-sdk-ios that referenced this pull request Mar 24, 2022
* pod repo update

* feat: Upgrade AppSyncRTClient 1.8.0 and enable logs

* Set subscription log level
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants