Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

examples/aws: client-side signing #4463

Merged
merged 3 commits into from
Jul 20, 2023
Merged

examples/aws: client-side signing #4463

merged 3 commits into from
Jul 20, 2023

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented May 18, 2023

Add client-side signing to our AWS example.

@aduh95 aduh95 changed the title examples/aws: use multipart/form-data instead of application/json examples/aws: use application/x-www-form-urlencoded instead of application/json May 19, 2023
@arturi arturi requested a review from Acconut May 22, 2023 12:28
@aduh95 aduh95 marked this pull request as draft May 24, 2023 10:19
@socket-security
Copy link

socket-security bot commented May 24, 2023

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

@Acconut
Copy link
Member

Acconut commented May 24, 2023

I am a bit lost on this PR, to be honest. Does this PR rework the example to use temporary credentials? If so, how is this connected to using application/x-www-form-urlencoded now?

@socket-security
Copy link

socket-security bot commented Jun 8, 2023

No top level dependency changes detected. Learn more about Socket for GitHub ↗︎

@Murderlon
Copy link
Member

What is the status on this PR? I thought this only serves as a technical playground for us but not to merge as it's extremely complex to expect our users to implement.

@Murderlon Murderlon changed the title examples/aws: use application/x-www-form-urlencoded instead of application/json examples/aws: client-side signing Jun 21, 2023
@aduh95 aduh95 marked this pull request as ready for review July 13, 2023 17:15
@aduh95 aduh95 linked an issue Jul 13, 2023 that may be closed by this pull request
2 tasks
@aduh95 aduh95 requested a review from Murderlon July 19, 2023 09:56
@aduh95 aduh95 merged commit 961f11f into main Jul 20, 2023
19 checks passed
@aduh95 aduh95 deleted the no-preflight branch July 20, 2023 09:34
Murderlon added a commit that referenced this pull request Jul 20, 2023
* main:
  @uppy/status-bar: fix ETA when status bar is installed during upload (#4588)
  @uppy/provider-views: fix infinite folder loading  (#4590)
  examples/aws: client-side signing (#4463)
@github-actions github-actions bot mentioned this pull request Jul 20, 2023
github-actions bot added a commit that referenced this pull request Jul 20, 2023
| Package                | Version | Package                | Version |
| ---------------------- | ------- | ---------------------- | ------- |
| @uppy/aws-s3-multipart |   3.5.1 | @uppy/provider-views   |   3.4.1 |
| @uppy/companion-client |   3.2.2 | @uppy/status-bar       |   3.2.3 |
| @uppy/dashboard        |   3.5.0 | @uppy/utils            |   5.4.2 |
| @uppy/locales          |   3.2.4 | uppy                   |  3.13.0 |

- meta: Add i18n to CONTRIBUTING.md (Mikael Finstad / #4591)
- @uppy/provider-views: Add VirtualList to ProviderView (Merlijn Vos / #4566)
- @uppy/provider-views: fix race conditions with folder loading (Mikael Finstad / #4578)
- @uppy/status-bar: fix ETA when status bar is installed during upload (Antoine du Hamel / #4588)
- @uppy/provider-views: fix infinite folder loading  (Mikael Finstad / #4590)
- meta: examples/aws: client-side signing (Antoine du Hamel / #4463)
- meta: Bump word-wrap from 1.2.3 to 1.2.4 (dependabot[bot] / #4586)
- meta: e2e: increase `requestTimeout` to 16s (Antoine du Hamel / #4587)
- @uppy/locales: update zh_TW translation (5idereal / #4583)
- @uppy/aws-s3-multipart: fix crash on pause/resume (Merlijn Vos / #4581)
- @uppy/aws-s3-multipart: do not access `globalThis.crypto` on the top-level (Bryan J Swift / #4584)
Murderlon added a commit that referenced this pull request Jul 25, 2023
* main:
  @uppy/aws-s3-multipart: refresh file before calling user-defined functions (#4557)
  Release: uppy@3.13.0 (#4595)
  Add i18n to CONTRIBUTING.md (#4591)
  Add VirtualList to ProviderView (#4566)
  @uppy/provider-views: fix race conditions with folder loading (#4578)
  @uppy/status-bar: fix ETA when status bar is installed during upload (#4588)
  @uppy/provider-views: fix infinite folder loading  (#4590)
  examples/aws: client-side signing (#4463)
  Bump word-wrap from 1.2.3 to 1.2.4 (#4586)
  e2e: increase `requestTimeout` to 16s (#4587)
  @uppy/locales: update zh_TW translation (#4583)
  @uppy/aws-s3-multipart: fix crash on pause/resume (#4581)
  @uppy/aws-s3-multipart: do not access `globalThis.crypto` on the top-level (#4584)
  Release: uppy@3.12.0 (#4574)
  @uppy/transloadit: fix error message (#4572)
  @uppy/provider-views: add support for remote file paths (#4537)
  @uppy/transloadit: implement Server-sent event API (#4098)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AWS example doesn't demonstrate how to do signing on the client
3 participants