Skip to content

Commit 6fe4156

Browse files
committed
Fix: PR feedback
1 parent 05b6f8a commit 6fe4156

File tree

3 files changed

+79
-14
lines changed

3 files changed

+79
-14
lines changed

lib/storage/CHANGELOG.md

Lines changed: 78 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,80 +7,151 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
77

88
**Note:** Version bump only for package @aws-sdk/lib-storage
99

10+
11+
12+
13+
1014
## [3.4.1](https://github.com/aws/aws-sdk-js-v3/compare/v3.4.0...v3.4.1) (2021-01-29)
1115

16+
1217
### Bug Fixes
1318

14-
- **clients:** use TS 3.4 compatible types for TS 3.9 ([#1978](https://github.com/aws/aws-sdk-js-v3/issues/1978)) ([8bced5c](https://github.com/aws/aws-sdk-js-v3/commit/8bced5c32b9dbc68f1065054d796cb0b8b87bcc4))
19+
* **clients:** use TS 3.4 compatible types for TS 3.9 ([#1978](https://github.com/aws/aws-sdk-js-v3/issues/1978)) ([8bced5c](https://github.com/aws/aws-sdk-js-v3/commit/8bced5c32b9dbc68f1065054d796cb0b8b87bcc4))
20+
21+
22+
23+
1524

1625
# [3.4.0](https://github.com/aws/aws-sdk-js-v3/compare/v3.3.0...v3.4.0) (2021-01-28)
1726

27+
1828
### Features
1929

20-
- use downlevel-dts to generate TS 3.4 compatible types ([#1943](https://github.com/aws/aws-sdk-js-v3/issues/1943)) ([63ad215](https://github.com/aws/aws-sdk-js-v3/commit/63ad2151c8bb7be32ea8838a9b0974806ed3906b))
30+
* use downlevel-dts to generate TS 3.4 compatible types ([#1943](https://github.com/aws/aws-sdk-js-v3/issues/1943)) ([63ad215](https://github.com/aws/aws-sdk-js-v3/commit/63ad2151c8bb7be32ea8838a9b0974806ed3906b))
31+
32+
33+
34+
2135

2236
# [3.3.0](https://github.com/aws/aws-sdk-js-v3/compare/v3.2.0...v3.3.0) (2021-01-14)
2337

2438
**Note:** Version bump only for package @aws-sdk/lib-storage
2539

40+
41+
42+
43+
2644
# [3.2.0](https://github.com/aws/aws-sdk-js-v3/compare/v3.1.0...v3.2.0) (2021-01-09)
2745

46+
2847
### Bug Fixes
2948

30-
- **lib-storage:** chunk from readable only when defined ([#1886](https://github.com/aws/aws-sdk-js-v3/issues/1886)) ([4cdc08a](https://github.com/aws/aws-sdk-js-v3/commit/4cdc08aef3dc57d1d200f4d6d9e9683426354374))
49+
* **lib-storage:** chunk from readable only when defined ([#1886](https://github.com/aws/aws-sdk-js-v3/issues/1886)) ([4cdc08a](https://github.com/aws/aws-sdk-js-v3/commit/4cdc08aef3dc57d1d200f4d6d9e9683426354374))
50+
51+
52+
53+
3154

3255
# [3.1.0](https://github.com/aws/aws-sdk-js-v3/compare/v3.0.0...v3.1.0) (2020-12-23)
3356

57+
3458
### Bug Fixes
3559

36-
- **lib-storage:** cleanup stream listeners to prevent memory leak ([3d36682](https://github.com/aws/aws-sdk-js-v3/commit/3d3668256467c5fa7a00e19c6c42667b5b3a77d4))
60+
* **lib-storage:** cleanup stream listeners to prevent memory leak ([3d36682](https://github.com/aws/aws-sdk-js-v3/commit/3d3668256467c5fa7a00e19c6c42667b5b3a77d4))
61+
62+
63+
64+
3765

3866
# [3.0.0](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.10...v3.0.0) (2020-12-15)
3967

68+
4069
### Features
4170

42-
- bump version to 3.0.0 ([#1793](https://github.com/aws/aws-sdk-js-v3/issues/1793)) ([d8475f8](https://github.com/aws/aws-sdk-js-v3/commit/d8475f8d972d28fbc15cd7e23abfe18f9eab0644))
71+
* bump version to 3.0.0 ([#1793](https://github.com/aws/aws-sdk-js-v3/issues/1793)) ([d8475f8](https://github.com/aws/aws-sdk-js-v3/commit/d8475f8d972d28fbc15cd7e23abfe18f9eab0644))
72+
73+
74+
75+
4376

4477
# [1.0.0-rc.10](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.9...v1.0.0-rc.10) (2020-12-15)
4578

4679
**Note:** Version bump only for package @aws-sdk/lib-storage
4780

81+
82+
83+
84+
4885
# [1.0.0-rc.9](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.8...v1.0.0-rc.9) (2020-12-11)
4986

5087
**Note:** Version bump only for package @aws-sdk/lib-storage
5188

89+
90+
91+
92+
5293
# [1.0.0-rc.8](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.7...v1.0.0-rc.8) (2020-12-05)
5394

5495
**Note:** Version bump only for package @aws-sdk/lib-storage
5596

97+
98+
99+
100+
56101
# [1.0.0-rc.7](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.6...v1.0.0-rc.7) (2020-11-20)
57102

58103
**Note:** Version bump only for package @aws-sdk/lib-storage
59104

105+
106+
107+
108+
60109
# [1.0.0-rc.6](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.5...v1.0.0-rc.6) (2020-11-13)
61110

62111
**Note:** Version bump only for package @aws-sdk/lib-storage
63112

113+
114+
115+
116+
64117
# [1.0.0-rc.5](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.4...v1.0.0-rc.5) (2020-11-09)
65118

66119
**Note:** Version bump only for package @aws-sdk/lib-storage
67120

121+
122+
123+
124+
68125
# [1.0.0-rc.4](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.3...v1.0.0-rc.4) (2020-10-31)
69126

70127
**Note:** Version bump only for package @aws-sdk/lib-storage
71128

129+
130+
131+
132+
72133
# [1.0.0-rc.3](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.2...v1.0.0-rc.3) (2020-10-27)
73134

74135
**Note:** Version bump only for package @aws-sdk/lib-storage
75136

137+
138+
139+
140+
76141
# [1.0.0-rc.2](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-rc.1...v1.0.0-rc.2) (2020-10-22)
77142

143+
78144
### Bug Fixes
79145

80-
- **storage:** add version and downloads badges ([#1599](https://github.com/aws/aws-sdk-js-v3/issues/1599)) ([230d030](https://github.com/aws/aws-sdk-js-v3/commit/230d0305801c298f93d3c534884833bf5cfa3f3f))
146+
* **storage:** add version and downloads badges ([#1599](https://github.com/aws/aws-sdk-js-v3/issues/1599)) ([230d030](https://github.com/aws/aws-sdk-js-v3/commit/230d0305801c298f93d3c534884833bf5cfa3f3f))
147+
148+
149+
150+
81151

82152
# [1.0.0-rc.1](https://github.com/aws/aws-sdk-js-v3/compare/v1.0.0-gamma.11...v1.0.0-rc.1) (2020-10-19)
83153

154+
84155
### Features
85156

86-
- **storage:** adding lib/storage upload ([#1547](https://github.com/aws/aws-sdk-js-v3/issues/1547)) ([b07e719](https://github.com/aws/aws-sdk-js-v3/commit/b07e71986817d7781e12d9b687532c58d5ea6ba0))
157+
* **storage:** adding lib/storage upload ([#1547](https://github.com/aws/aws-sdk-js-v3/issues/1547)) ([b07e719](https://github.com/aws/aws-sdk-js-v3/commit/b07e71986817d7781e12d9b687532c58d5ea6ba0))

lib/storage/src/Upload.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,8 +154,6 @@ export class Upload extends EventEmitter {
154154
const completeMultipartUpload = await this.client.send(
155155
new CompleteMultipartUploadCommand({
156156
...this.params,
157-
Bucket: this.params.Bucket,
158-
Key: this.params.Key,
159157
UploadId: this.uploadId,
160158
MultipartUpload: {
161159
Parts: this.uploadedParts,

lib/storage/src/bytelength.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,6 @@
1-
const toBuffer = (data: any) => {
2-
return typeof Buffer.from === "function" && Buffer.from !== Uint8Array.from ? Buffer.from(data) : new Buffer(data);
3-
};
4-
51
export const byteLength = (input: any) => {
62
if (input === null || input === undefined) return 0;
7-
if (typeof input === "string") input = toBuffer(input);
3+
if (typeof input === "string") input = Buffer.from(input);
84
if (typeof input.byteLength === "number") {
95
return input.byteLength;
106
} else if (typeof input.length === "number") {

0 commit comments

Comments
 (0)