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

Multiple Image-Upload broken when SessionHandlerRedis is installed #5

Closed
tiefenb opened this issue Jan 19, 2019 · 3 comments
Closed

Comments

@tiefenb
Copy link

tiefenb commented Jan 19, 2019

I got a very strange bug in my system:

One week after switching from SessionHandlerDB to SessionHandlerRedis, the costumer reported that uploading multiple images in parallel are broken. After searching the bug some hours in other modules, core and some hooks which I use, I begann to deactivate each third-party module to test the uploading. After deactivating SessionHandlerRedis the upload was working correct.

Do you have any idea why SessionHandlerRedis causes this bug?

What exactly happens is:
I upload 1 image (abc.jpg) with 2000KB and 1 image with (xyz.jpg) with 200 KB.
The first row in the database-table what I see is the row with xyz.jpg and after the other image is finished with the upload the xyz.jpg row is deleted and only the abc.jpg is saved.

Processwire: 3.0.122
PHP: 7.0

I will try to reproduce this with a fresh and empty PW installation and SessionHandlerRedis soon.

@tiefenb
Copy link
Author

tiefenb commented Jan 19, 2019

Now I have tested it with a clean installation of PW 3.0.124 (site-default), PHP 7.0.31 and SessionHandlerRedis.

after installing SessionHandlerRedis the paralell uploading doesn't work any more :-(

@tiefenb
Copy link
Author

tiefenb commented Jan 19, 2019

I have zipped my installation and you can download it here: https://wetransfer.com/downloads/0d89567ece797bc755514df5a2c3fa7220190119210112/7fcbc5926afa1e5c07864f681f98271220190119210112/238d77

if you want to run it with docker go into the root directory of pw:

docker-compose build
docker-compose up

sql dump is also in the root: pwtest_dump.sql

@tiefenb
Copy link
Author

tiefenb commented Jan 22, 2019

okay this bug appear also with native redis AND memcached as php session handler. I will close this issue because I think this is a general processwire or PHP bug: processwire/processwire-issues#798

@tiefenb tiefenb closed this as completed Jan 22, 2019
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

No branches or pull requests

1 participant