-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Docker: PHP-Redis extension is missing #10910
Comments
👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can. |
It isn't installed on this version, it should have been added on v6, but I just checked composer file on master and it isn't there.. |
We’d happily take a PR for this. Redis is not commonly used with Snipe-IT, so I’m not exactly shocked it wasn’t added to the docker image. There are not many use cases I can think of that would require Redis here. |
Again, we'd welcome a PR for it. |
@snipe @asassoye @stavros-k For alpine image FROM snipe/snipe-it:latest-alpine
RUN apk add --no-cache \
php7-redis this for ubuntu FROM snipe/snipe-it:latest
RUN apt-get update -qqy \
&& apt-get install -qqy --no-install-recommends \
php7.4-redis |
@stavros-k @asassoye |
Debug mode
Describe the bug
Hello,
When i configure Redis in the configuration, i have this error:
LogicException Please make sure the PHP Redis extension is installed and enabled.
with the https://hub.docker.com/r/snipe/snipe-it docker image with tag v5.4.1 (and also alpine version)When i install the php-redis package on the container everything works fine and the sessions and cache are stored in Redis.
Reproduction steps
Config:
Expected behavior
Snipe-it is using Redis for cache and for sessions
Screenshots
Snipe-IT Version
v5.4.1
Operating System
Docker
Web Server
Docker (Apache)
PHP Version
7.4.3
The text was updated successfully, but these errors were encountered: