Skip to content

Commit 8fe3c2b

Browse files
committed
fix: resolve bugs with user service
1 parent 432bd33 commit 8fe3c2b

16 files changed

Lines changed: 24 additions & 22 deletions

File tree

src/user-management-service/lib/constructs/lambdaFunction.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ export class InstrumentedLambdaFunction extends Construct {
3737
functionName: `CDK-${props.sharedProps.serviceName}-${id}-${props.sharedProps.environment}`,
3838
manifestPath: props.manifestPath,
3939
memorySize: 256,
40+
timeout: Duration.seconds(30),
4041
environment: {
4142
DEPLOYED_AT: new Date().toISOString(),
4243
DD_OTLP_CONFIG_RECEIVER_PROTOCOLS_HTTP_ENDPOINT: "localhost:4318",

src/user-management-service/src/user-management/core/src/adapters.rs

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -84,13 +84,14 @@ impl DynamoDbRepository {
8484
AttributeValue::N(details.order_count.to_string()),
8585
);
8686

87-
let res = if details.last_active.is_some() {
88-
put_item_builder.clone().item(
89-
LAST_ACTIVE_KEY,
90-
AttributeValue::S(details.last_active.unwrap().to_string()),
91-
);
92-
93-
put_item_builder.send().await
87+
let res = if let Some(last_active) = details.last_active {
88+
put_item_builder
89+
.item(
90+
LAST_ACTIVE_KEY,
91+
AttributeValue::S(last_active.to_string()),
92+
)
93+
.send()
94+
.await
9495
} else {
9596
put_item_builder.send().await
9697
};
@@ -546,7 +547,7 @@ impl Repository for DynamoDbRepository {
546547
.delete_item()
547548
.table_name(&self.table_name)
548549
.key(PARTITION_KEY, AttributeValue::S(format!("CODE#{}", code)))
549-
.key(SORT_KEY, AttributeValue::S(format!("CODE#{}", code)))
550+
.key(SORT_KEY, AttributeValue::S("METADATA".to_string()))
550551
.send()
551552
.await;
552553

src/user-management-service/src/user-management/lambdas/create_user/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ async fn main() -> Result<(), Error> {
3434
}
3535
};
3636

37-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
37+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
3838

3939
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
4040
let config = aws_config::load_from_env().await;

src/user-management-service/src/user-management/lambdas/get_user_details/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ async fn main() -> Result<(), Error> {
7777
}
7878
};
7979

80-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
80+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
8181

8282
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
8383
let config = aws_config::load_from_env().await;

src/user-management-service/src/user-management/lambdas/handle_order_completed_for_user/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ async fn main() -> Result<(), Error> {
2525
}
2626
};
2727

28-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
28+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
2929
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
3030
let config = aws_config::load_from_env().await;
3131
let dynamodb_client = aws_sdk_dynamodb::Client::new(&config);

src/user-management-service/src/user-management/lambdas/login/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ async fn main() -> Result<(), Error> {
6060
}
6161
};
6262

63-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
63+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
6464
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
6565
let config = aws_config::load_from_env().await;
6666
let dynamodb_client = aws_sdk_dynamodb::Client::new(&config);

src/user-management-service/src/user-management/lambdas/oauth_authorize/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ async fn main() -> Result<(), Error> {
227227
}
228228
};
229229

230-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
230+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
231231
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
232232
let config = aws_config::load_from_env().await;
233233
let dynamodb_client = aws_sdk_dynamodb::Client::new(&config);

src/user-management-service/src/user-management/lambdas/oauth_authorize_callback/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ async fn main() -> Result<(), Error> {
111111
}
112112
};
113113

114-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
114+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
115115
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
116116
let config = aws_config::load_from_env().await;
117117
let dynamodb_client = aws_sdk_dynamodb::Client::new(&config);

src/user-management-service/src/user-management/lambdas/oauth_client_delete/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ async fn main() -> Result<(), Error> {
6262
}
6363
};
6464

65-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
65+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
6666
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
6767
let config = aws_config::load_from_env().await;
6868
let dynamodb_client = aws_sdk_dynamodb::Client::new(&config);

src/user-management-service/src/user-management/lambdas/oauth_client_get/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ async fn main() -> Result<(), Error> {
6262
}
6363
};
6464

65-
let _ = TRACER_PROVIDER.set(otel_providers.unwrap().0);
65+
if let Some(providers) = otel_providers { let _ = TRACER_PROVIDER.set(providers.0); }
6666
let table_name = env::var("TABLE_NAME").expect("TABLE_NAME is not set");
6767
let config = aws_config::load_from_env().await;
6868
let dynamodb_client = aws_sdk_dynamodb::Client::new(&config);

0 commit comments

Comments
 (0)