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

Add TUS 2.0 #259

Merged
merged 2 commits into from
Jun 20, 2020
Merged

Add TUS 2.0 #259

merged 2 commits into from
Jun 20, 2020

Conversation

mstaack
Copy link
Contributor

@mstaack mstaack commented May 8, 2020

No description provided.

@jnbn
Copy link

jnbn commented May 15, 2020

I suggest using it as
"ankitpokhrel/tus-php": "^v1.0 || ^v2.0" for future and previous releases.

Copy link

@robertmindroi robertmindroi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will unlock the use for symfony/http-foundation ^5.0

@oscarnevarezleal
Copy link

This would be great to have. I'm getting an error due ankitpokhrel/tus-php@v1 is using a previous version of symfony/http-foundation that attempts to change the save handler using ini_set.

Since PHP 7.2 they dropped the ability to change the session save handler to "user" using ini_set()

@mstaack
Copy link
Contributor Author

mstaack commented May 25, 2020

@muglug do you mind having a look and merge?

or is there anyone else maintaining this library?

@apsg
Copy link

apsg commented Jun 20, 2020

Can anyone pls take a look into this?

@muglug muglug merged commit 169b8ea into vimeo:master Jun 20, 2020
@jnbn
Copy link

jnbn commented Jun 20, 2020

Can you please tag a release @muglug ?

@muglug
Copy link
Contributor

muglug commented Jun 20, 2020

Done, sorry for the wait

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.

6 participants