Skip to content

Commit

Permalink
@uppy/companion: capitalize POST (#5198)
Browse files Browse the repository at this point in the history
  • Loading branch information
mifi authored and aduh95 committed May 30, 2024
1 parent e09b428 commit e54d7fa
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions docs/guides/migration-guides.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ These cover all the major Uppy versions and how to migrate to them.
provider, you should not be affected.
- The static `getExtraConfig` property has been renamed to
`getExtraGrantConfig`.
- Endpoint `GET /s3/params` now returns `{ method: "POST" }` instead of
`{ method: "post" }`. This will not affect most people.

### `@uppy/companion-client`

Expand Down
2 changes: 1 addition & 1 deletion packages/@uppy/companion/src/server/controllers/s3.js
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ module.exports = function s3 (config) {
Key: key,
}).then(data => {
res.json({
method: 'post', // TODO: switch to the uppercase 'POST' in the next major
method: 'POST',
url: data.url,
fields: data.fields,
expires: config.expires,
Expand Down

0 comments on commit e54d7fa

Please sign in to comment.