Skip to content

Commit 040845d

Browse files
author
Eugene Cheung
authored
fix(sqs): update console URL (#503)
Fixes #502 --- _By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license_
1 parent 12cdccc commit 040845d

File tree

5 files changed

+16
-16
lines changed

5 files changed

+16
-16
lines changed

lib/common/url/AwsConsoleUrlFactory.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ export class AwsConsoleUrlFactory {
145145

146146
getSqsQueueUrl(queueUrl: string): string | undefined {
147147
const region = this.awsAccountRegion;
148-
const destinationUrl = `https://${region}.console.aws.amazon.com/sqs/v2/home?region=${region}#/queues/${queueUrl}`;
148+
const destinationUrl = `https://${region}.console.aws.amazon.com/sqs/v3/home?region=${region}#/queues/${queueUrl}`;
149149
return this.getAwsConsoleUrl(destinationUrl);
150150
}
151151

test/common/url/AwsConsoleUrlFactory.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ test("getSqsQueueUrl", () => {
7878
const factory = new AwsConsoleUrlFactory({ awsAccountId, awsAccountRegion });
7979

8080
const expected =
81-
"https://eu-west-1.console.aws.amazon.com/sqs/v2/home?region=eu-west-1#/queues/DummyQueueUrl";
81+
"https://eu-west-1.console.aws.amazon.com/sqs/v3/home?region=eu-west-1#/queues/DummyQueueUrl";
8282

8383
expect(stack.resolve(factory.getSqsQueueUrl("DummyQueueUrl"))).toEqual(
8484
expected

test/facade/__snapshots__/MonitoringAspect.test.ts.snap

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

test/monitoring/aws-sqs/__snapshots__/SqsQueueMonitoring.test.ts.snap

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

test/monitoring/aws-sqs/__snapshots__/SqsQueueMonitoringWithDlq.test.ts.snap

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)