Skip to content

Commit f1b5bfb

Browse files
committed
Not to use try cath
1 parent 0fb587d commit f1b5bfb

File tree

2 files changed

+8
-14
lines changed

2 files changed

+8
-14
lines changed

packages/polly-request-presigner/src/getSignedUrls.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ describe("getSignedUrl", () => {
9999
mockPresign.mockReturnValue(mockPresigned);
100100
const client = new PollyClient(clientParams);
101101
const command = new SynthesizeSpeechCommand({
102-
Text: "hello world, this is alex",
102+
Text: "hello world",
103103
OutputFormat: "mp3",
104104
VoiceId: "Kimberly",
105105
});

packages/polly-request-presigner/src/getSignedUrls.ts

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -51,19 +51,13 @@ export const getSignedUrl = async (
5151
} as any;
5252
};
5353
const middlewareName = "presignInterceptMiddleware";
54-
try {
55-
client.middlewareStack.addRelativeTo(presignInterceptMiddleware, {
56-
name: middlewareName,
57-
relation: "before",
58-
toMiddleware: "awsAuthMiddleware",
59-
});
60-
} catch (e) {
61-
if (e.message!.includes(`Duplicated middleware name '${middlewareName}'`)) {
62-
// Swallow if the interceptor is already added. See https://github.com/aws/aws-sdk-js-v3/issues/2417
63-
} else {
64-
throw e;
65-
}
66-
}
54+
55+
client.middlewareStack.addRelativeTo(presignInterceptMiddleware, {
56+
name: middlewareName,
57+
relation: "before",
58+
toMiddleware: "awsAuthMiddleware",
59+
override: true,
60+
});
6761

6862
let presigned: HttpRequest;
6963
try {

0 commit comments

Comments
 (0)