-
Notifications
You must be signed in to change notification settings - Fork 293
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
Error changing password as an admin for a user (v0.91) #1089
Comments
What php version are you using? It seems to work fine on my 8.1.1 installation. |
Hi @ByteHamster , I do use the docker version "https://hub.docker.com/r/ckulka/baikal" of Baikal. Guess maybe here is the problem? Best regards |
I Guess it is "Release 0.9.1 with PHP 8.0 ckulka/baikal-docker#52" and maybe that is the problem here? |
The Docker image of 0.9.1 comes with PHP 8.1.1 as well: $ docker run --rm -it ckulka/baikal:0.9.1 php --version
PHP 8.1.1 (cli) (built: Dec 21 2021 19:41:31) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.1, Copyright (c) Zend Technologies
$ docker run --rm -it ckulka/baikal:0.9.1 php --re pdo | head -1
Extension [ <persistent> extension #20 PDO version 8.1.1 ] {
$ docker run --rm -it ckulka/baikal:0.9.1 php --re pdo_mysql | head -1
Extension [ <persistent> extension #33 pdo_mysql version 8.1.1 ] { @forestvogo, can you please try Edit: the images were published as |
@ckulka 👍 Will do and get back to you soon with the result. |
@ckulka : Forget my last message (already deleted it). With the experimental Image changing the password was a success: Does that help you to analyse further? |
I'd say it helped, yes: as far as I can tell from your two attemps, your issue only appears in PHP 8.1 but not with PHP 8.0, so at least you have a working instance now. As far as "why" goes, I'm not familiar enough with PHP to know what other differences or dependencies could cause the different behaviour between PHP 8.1.1 in the container that fails (the details I posted above) and PHP 8.1.1 that @ByteHamster (and others probably too) have success with. If I had to guess, I think that sabre-io/vobject#561 could fix the issue - like you already alluded to in #1089 (comment), the error messages look the same. |
Weird. I can reproduce the same stack trace on another screen (calendar settings page) but still not on the user settings page. The code in #1099 fixes the error message on my calendar settings page and I am pretty sure it will also fix the problem on your user settings page. |
Baikal version: 0.91
Expected behaviour: Adminstrator should be able to set a new password for the user
Current behaviour: Error message is shown as follows:
Steps to reproduce:
The text was updated successfully, but these errors were encountered: