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

[Feature]: TUS Resumable Uploads #93

Closed
paulocastellano opened this issue Jul 17, 2023 · 2 comments
Closed

[Feature]: TUS Resumable Uploads #93

paulocastellano opened this issue Jul 17, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@paulocastellano
Copy link

Description

It's possible to add TUS Resumable Uploads for large videos?

Example

No response

@paulocastellano paulocastellano added the enhancement New feature or request label Jul 17, 2023
@ToshY
Copy link
Owner

ToshY commented Jul 17, 2023

Hey @paulocastellano 👋

Thank you for your question. As I have no prior knowledge with TUS, I've started reading both the bunny.net documentation as well as tus.io faq.

It seems there is already an existing PHP implementation specifically for TUS: ankitpokhrel/tus-php. Looking through these docs and codebase it seems to me that it goes way beyond an API call (and requires additional functionality like caching).

Keep in mind that BunnyNet-PHP is "basically"1 just an API client for Bunny.net, and only has a handful of essential dependencies. The idea is to keep it that way, without adding more complexity than it actually needs to function properly.

That being said, I'm all ears if you have more knowledge on the subject or have other ideas that would make this easy to implement without the need of (many) additional dependencies.

Footnotes

  1. I wrote "basically" because I also included the TokenAuthentication class which is strictly speaking not part of the API; still a good (dependency-free) addition nonetheless.

@ToshY
Copy link
Owner

ToshY commented Jul 23, 2023

Hey @paulocastellano 👋

I haven't seen a response yet, so I will close the issue for now. You're more than welcome to respond if you want to further elaborate on your suggestion so we can re-open the issue.

@ToshY ToshY closed this as completed Jul 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants