mirror of
https://github.com/nextcloud/twofactor_gateway.git
synced 2026-04-25 17:15:53 +03:00
[GH-ISSUE #522] No PHP version check on last version (0.20.0) #111
Labels
No labels
0. to triage
1. to develop
3. to review
blocked
bug
discussion
duplicate
enhancement
enhancement
gateway:signal
gateway:signal
gateway:signal
gateway:sms
gateway:telegram
hacktoberfest
help wanted
invalid
needs info
php
pull-request
question
technical debt
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/twofactor_gateway-nextcloud#111
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @hmag on GitHub (Jul 4, 2022).
Original GitHub issue: https://github.com/nextcloud/twofactor_gateway/issues/522
Having a NextCloud 20.0.6 running on a Debian 10 (PHP version 7.3.31-1
deb10u1), I had installed twofactor_gateway version 0.19.0.deb10u1." : not possible to login, my sync applications failed, etc...I was proposed by NextCloud to update it to the latest version, which I did.
Then I was stuck with the message "Your Composer dependencies require a PHP version ">= 7.4.0". You are running 7.3.31-1
After checking, I performed a "occ app:disable twofactor_gateway" to disable the application and now everything is fine... except I don't have anymore the gateway.
So it seems like PHP 7.4 is min is required but this is not checked before installing the app.
Best regards,
Hervé
@vitormattos commented on GitHub (Jul 5, 2022):
Hi @hmag, Nextcloud 20 isn't more supported by this app:
https://github.com/nextcloud/twofactor_gateway/blob/master/appinfo/info.xml#L22
@hmag commented on GitHub (Jul 5, 2022):
Hi,
Sorry, I meant Nextcloud 23.0.6 (which is supported by nextcloud with php 7.3 and is supposed to be the same by the app which requires php 7.4), my apologies for the confusion.
Best regards
@vitormattos commented on GitHub (Jul 6, 2022):
Thanks for this information.
I remembered that we already have an issue that will solve this problem: #484, because of this, I will maintain this issue closed and follow with the necessary changes in #484
Best regards.