Skip to content

Commit 7a8eb73

Browse files
authored
feat(logging): Add Cloudwatch Logging Plugin (#2518)
1 parent 5662e82 commit 7a8eb73

File tree

110 files changed

+2386
-113
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

110 files changed

+2386
-113
lines changed

aws-api/src/main/java/com/amplifyframework/api/aws/AppSyncGraphQLOperation.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
import com.amplifyframework.api.graphql.GraphQLResponse;
2929
import com.amplifyframework.core.Amplify;
3030
import com.amplifyframework.core.Consumer;
31+
import com.amplifyframework.core.category.CategoryType;
3132
import com.amplifyframework.logging.Logger;
3233

3334
import java.io.IOException;
@@ -50,7 +51,7 @@
5051
* @param <R> Casted type of GraphQL result data
5152
*/
5253
public final class AppSyncGraphQLOperation<R> extends GraphQLOperation<R> {
53-
private static final Logger LOG = Amplify.Logging.forNamespace("amplify:aws-api");
54+
private static final Logger LOG = Amplify.Logging.logger(CategoryType.API, "amplify:aws-api");
5455
private static final String CONTENT_TYPE = "application/json";
5556
private static final int START_OF_CLIENT_ERROR_CODE = 400;
5657
private static final int END_OF_CLIENT_ERROR_CODE = 499;

aws-api/src/main/java/com/amplifyframework/api/aws/MultiAuthAppSyncGraphQLOperation.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
import com.amplifyframework.api.graphql.GraphQLResponse;
2929
import com.amplifyframework.core.Amplify;
3030
import com.amplifyframework.core.Consumer;
31+
import com.amplifyframework.core.category.CategoryType;
3132
import com.amplifyframework.core.model.auth.AuthorizationTypeIterator;
3233
import com.amplifyframework.datastore.appsync.AppSyncExtensions;
3334
import com.amplifyframework.logging.Logger;
@@ -53,7 +54,7 @@
5354
* @param <R> Casted type of GraphQL result data
5455
*/
5556
public final class MultiAuthAppSyncGraphQLOperation<R> extends GraphQLOperation<R> {
56-
private static final Logger LOG = Amplify.Logging.forNamespace("amplify:aws-api");
57+
private static final Logger LOG = Amplify.Logging.logger(CategoryType.API, "amplify:aws-api");
5758
private static final String CONTENT_TYPE = "application/json";
5859

5960
private final String endpoint;

aws-api/src/main/java/com/amplifyframework/api/aws/MutiAuthSubscriptionOperation.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
import com.amplifyframework.core.Action;
2828
import com.amplifyframework.core.Amplify;
2929
import com.amplifyframework.core.Consumer;
30+
import com.amplifyframework.core.category.CategoryType;
3031
import com.amplifyframework.core.model.auth.AuthorizationTypeIterator;
3132
import com.amplifyframework.datastore.appsync.AppSyncExtensions;
3233
import com.amplifyframework.logging.Logger;
@@ -38,7 +39,7 @@
3839
import java.util.concurrent.atomic.AtomicBoolean;
3940

4041
final class MutiAuthSubscriptionOperation<T> extends GraphQLOperation<T> {
41-
private static final Logger LOG = Amplify.Logging.forNamespace("amplify:aws-api");
42+
private static final Logger LOG = Amplify.Logging.logger(CategoryType.API, "amplify:aws-api");
4243

4344
private final SubscriptionEndpoint subscriptionEndpoint;
4445
private final ExecutorService executorService;

aws-api/src/main/java/com/amplifyframework/api/aws/SubscriptionEndpoint.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
import com.amplifyframework.core.Action;
3030
import com.amplifyframework.core.Amplify;
3131
import com.amplifyframework.core.Consumer;
32+
import com.amplifyframework.core.category.CategoryType;
3233
import com.amplifyframework.logging.Logger;
3334
import com.amplifyframework.util.UserAgent;
3435

@@ -60,7 +61,7 @@
6061
* and multiple GraphQL subscriptions that work on top of it.
6162
*/
6263
final class SubscriptionEndpoint {
63-
private static final Logger LOG = Amplify.Logging.forNamespace("amplify:aws-api");
64+
private static final Logger LOG = Amplify.Logging.logger(CategoryType.API, "amplify:aws-api");
6465
private static final int CONNECTION_ACKNOWLEDGEMENT_TIMEOUT = 30 /* seconds */;
6566
private static final int NORMAL_CLOSURE_STATUS = 1000;
6667
private static final String UNAUTHORIZED_EXCEPTION = "UnauthorizedException";

aws-api/src/main/java/com/amplifyframework/api/aws/SubscriptionOperation.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import com.amplifyframework.core.Action;
2525
import com.amplifyframework.core.Amplify;
2626
import com.amplifyframework.core.Consumer;
27+
import com.amplifyframework.core.category.CategoryType;
2728
import com.amplifyframework.logging.Logger;
2829

2930
import java.util.Objects;
@@ -32,7 +33,7 @@
3233
import java.util.concurrent.atomic.AtomicBoolean;
3334

3435
final class SubscriptionOperation<T> extends GraphQLOperation<T> {
35-
private static final Logger LOG = Amplify.Logging.forNamespace("amplify:aws-api");
36+
private static final Logger LOG = Amplify.Logging.logger(CategoryType.API, "amplify:aws-api");
3637

3738
private final SubscriptionEndpoint subscriptionEndpoint;
3839
private final ExecutorService executorService;

aws-api/src/main/java/com/amplifyframework/api/aws/auth/ApiRequestDecoratorFactory.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
import com.amplifyframework.api.graphql.GraphQLRequest;
3333
import com.amplifyframework.auth.CognitoCredentialsProvider;
3434
import com.amplifyframework.core.Amplify;
35+
import com.amplifyframework.core.category.CategoryType;
3536
import com.amplifyframework.logging.Logger;
3637

3738
import java.util.Objects;
@@ -43,7 +44,7 @@
4344
* Factory class that creates instances of different implementations of {@link RequestDecorator}s.
4445
*/
4546
public final class ApiRequestDecoratorFactory {
46-
private static final Logger LOG = Amplify.Logging.forNamespace("amplify:aws-api");
47+
private static final Logger LOG = Amplify.Logging.logger(CategoryType.API, "amplify:aws-api");
4748
private static final String AUTH_DEPENDENCY_PLUGIN_KEY = "awsCognitoAuthPlugin";
4849
private static final String APP_SYNC_SERVICE_NAME = "appsync";
4950
private static final String API_GATEWAY_SERVICE_NAME = "execute-api";

aws-auth-cognito/src/main/java/com/amplifyframework/auth/cognito/AWSCognitoAuthPlugin.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ import com.amplifyframework.auth.result.AuthUpdateAttributeResult
5757
import com.amplifyframework.core.Action
5858
import com.amplifyframework.core.Amplify
5959
import com.amplifyframework.core.Consumer
60+
import com.amplifyframework.core.category.CategoryType
6061
import com.amplifyframework.statemachine.codegen.data.AuthConfiguration
6162
import kotlinx.coroutines.CoroutineScope
6263
import kotlinx.coroutines.CoroutineStart
@@ -77,7 +78,7 @@ class AWSCognitoAuthPlugin : AuthPlugin<AWSCognitoAuthService>() {
7778
}
7879

7980
private val logger =
80-
Amplify.Logging.forNamespace(AWS_COGNITO_AUTH_LOG_NAMESPACE.format(this::class.java.simpleName))
81+
Amplify.Logging.logger(CategoryType.AUTH, AWS_COGNITO_AUTH_LOG_NAMESPACE.format(this::class.java.simpleName))
8182

8283
@VisibleForTesting
8384
internal lateinit var realPlugin: RealAWSCognitoAuthPlugin

aws-datastore/src/androidTest/java/com/amplifyframework/datastore/MultiAuthSyncEngineInstrumentationTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,10 @@
113113
* verify behavior when in multi-auth mode.
114114
*/
115115
public final class MultiAuthSyncEngineInstrumentationTest {
116-
private static final Logger LOG = Amplify.Logging.forNamespace("MultiAuthSyncEngineInstrumentationTest");
116+
private static final Logger LOG = Amplify.Logging.logger(
117+
CategoryType.DATASTORE,
118+
"MultiAuthSyncEngineInstrumentationTest"
119+
);
117120
private static final int TIMEOUT_SECONDS = 20;
118121
private static final String AUDIENCE = "integtest";
119122
private static final String GOOGLE_ISS_CLAIM = "https://accounts.google.com";

aws-datastore/src/androidTest/java/com/amplifyframework/datastore/MultiAuthSyncEngineNoAuthInstrumentationTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,10 @@
7979
* verify behavior when in multi-auth mode.
8080
*/
8181
public final class MultiAuthSyncEngineNoAuthInstrumentationTest {
82-
private static final Logger LOG = Amplify.Logging.forNamespace("MultiAuthSyncEngineInstrumentationTest");
82+
private static final Logger LOG = Amplify.Logging.logger(
83+
CategoryType.DATASTORE,
84+
"MultiAuthSyncEngineInstrumentationTest"
85+
);
8386
private static final int TIMEOUT_SECONDS = 20;
8487
private static final String GOOGLE_ISS_CLAIM = "https://accounts.google.com";
8588

aws-datastore/src/androidTest/java/com/amplifyframework/datastore/storage/sqlite/ModelUpgradeSQLiteInstrumentedTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import com.amplifyframework.AmplifyException;
2222
import com.amplifyframework.core.Amplify;
2323
import com.amplifyframework.core.Consumer;
24+
import com.amplifyframework.core.category.CategoryType;
2425
import com.amplifyframework.core.model.ModelSchema;
2526
import com.amplifyframework.core.model.SchemaRegistry;
2627
import com.amplifyframework.datastore.DataStoreConfiguration;
@@ -114,7 +115,7 @@ public void modelVersionStoredCorrectlyBeforeAndAfterUpgrade() throws AmplifyExc
114115
.syncInterval(2L, TimeUnit.MINUTES)
115116
.build());
116117
} catch (DataStoreException exception) {
117-
Amplify.Logging.forNamespace("amplify:aws-datastore").warn(exception.toString());
118+
Amplify.Logging.logger(CategoryType.DATASTORE, "amplify:aws-datastore").warn(exception.toString());
118119
}
119120
}
120121
);
@@ -152,7 +153,7 @@ public void modelVersionStoredCorrectlyBeforeAndAfterUpgrade() throws AmplifyExc
152153
.syncInterval(2L, TimeUnit.MINUTES)
153154
.build());
154155
} catch (DataStoreException exception) {
155-
Amplify.Logging.forNamespace("amplify:aws-datastore").warn(exception.toString());
156+
Amplify.Logging.logger(CategoryType.DATASTORE, "amplify:aws-datastore").warn(exception.toString());
156157
}
157158
}
158159
);

0 commit comments

Comments
 (0)