Comments (6)
Do you guys have any thoughts here?
It would be great if Symfony were to fully implement HEAD
requests, but there have been many arguments for it in the past, and they were all shot down. I don't know what the chances are of Symfony doing so, but I put in the feature request regardless.
from tus-php.
Changing HEAD request to GET is definitely not desired. However, lets verify the changes needed for HEAD request to implement it in Symfony Request/Response. But we have to identify its impact on the project. Waiting for thoughts from @ankitpokhrel
from tus-php.
File download in GET /files/<hash>
request is not a part of the tus specification. I would propose to move current file download endpoint to GET /files/<hash>/get
and treat GET /files/<hash>
same as HEAD request.
One thing to note here is that official tus server implementation treats GET /files/<hash>
request as file download request.
I am open for other ideas.
from tus-php.
I love the idea of adding /get
or any other path component to indicate download.
I've not seen any traction in Symfony or Drupal camps to implement HEAD
, I'm surprised it has gone this way for years, but no one had a valid use case.
from tus-php.
@ankitpokhrel I think we should try to implement what current tus-servers have implemented for the download request.
The solution that you proposed could work as custom solution for download extending on top of official tus server implementation.
I think we should implement the custom solution that are already in use by most tus-servers.
from tus-php.
Changed in #87
from tus-php.
Related Issues (20)
- File based cache - tus_php.server.cache empty - 410 Error HOT 5
- Continue file upload from different browser and device HOT 3
- How to protect single file link? HOT 2
- CakePHP integration does not work HOT 2
- How can I do Cloudflare Integration? HOT 2
- Location header set to http when deployed behind reverse proxy HOT 2
- Unable to upload files - PATCH GONE HOT 1
- Getting Started Documentation
- Allow PHP 8+ for version 1.x.x
- Checksum implementation does not match TUS specification
- Enable to update to PHP 8.1 due to version constraint
- Documentation, Please!
- validating file mime type
- Headers can be totally configurable HOT 1
- Please kindly explain this code to me HOT 1
- Password protected Redis
- Possible bug or configuration issue? HOT 3
- Unable to create resource.
- PHP 8.1+ support HOT 3
- Laravel Octane support
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from tus-php.