starred/twofactor_gateway-nextcloud

Fork 0

mirror of https://github.com/nextcloud/twofactor_gateway.git synced 2026-04-25 09:05:55 +03:00

[GH-ISSUE #302] sms77io not selectable #70

New issue

Open

opened 2026-02-26 05:32:50 +03:00 by kerem · 9 comments

kerem commented

2026-02-26 05:32:50 +03:00

Owner

Originally created by @itverwaltung on GitHub (Dec 12, 2019).
Original GitHub issue: https://github.com/nextcloud/twofactor_gateway/issues/302

In version 15 smsio is supported as a twofactor gateway provider.

www-data@svrhnextcloud:/var/www/nextcloud$ php occ twofactorauth:gateway:configure sms
Please choose a SMS provider (websms, playsms, clockworksms, puzzelsms, ecallsms, voipms, huawei_e3531, spryng):

sms77io is not listed... How can i use this provider?

www-data@svrhnextcloud:/var/www/nextcloud$ php occ app:list | grep twofactor_gateway

twofactor_gateway: 0.15.0

So version 15 is installed ...

Thank you for any advice..

Alex

Originally created by @itverwaltung on GitHub (Dec 12, 2019). Original GitHub issue: https://github.com/nextcloud/twofactor_gateway/issues/302 In version 15 smsio is supported as a twofactor gateway provider. www-data@svrhnextcloud:/var/www/nextcloud$ php occ twofactorauth:gateway:configure sms Please choose a SMS provider (websms, playsms, clockworksms, puzzelsms, ecallsms, voipms, huawei_e3531, spryng): sms77io is not listed... How can i use this provider? www-data@svrhnextcloud:/var/www/nextcloud$ php occ app:list | grep twofactor_gateway - twofactor_gateway: 0.15.0 So version 15 is installed ... Thank you for any advice.. Alex

kerem added the

help wanted

1. to develop

labels

2026-02-26 05:32:50 +03:00

kerem commented

2026-02-26 05:32:50 +03:00

Author

Owner

@ChristophWurst commented on GitHub (Dec 12, 2019):

Hi Alex,

enter 'sms77' ;)

It's just missing in the help text at github.com/nextcloud/twofactor_gateway@c7a23e6eb7/lib/Command/Configure.php (L108). Mind submitting a pull request to add it? That would be awesome!

Cheers and thanks for the feedback.

@ChristophWurst commented on GitHub (Dec 12, 2019): Hi Alex, enter 'sms77' ;) It's just missing in the help text at https://github.com/nextcloud/twofactor_gateway/blob/c7a23e6eb77fa5a44465d2f870a8e418e687b1ce/lib/Command/Configure.php#L108. Mind submitting a pull request to add it? That would be awesome! Cheers and thanks for the feedback.

kerem commented

2026-02-26 05:32:50 +03:00

Author

Owner

@itverwaltung commented on GitHub (Dec 12, 2019):

Hi Christoph,

i did as you said, but no luck:

www-data@svrhnextcloud:/var/www/nextcloud$ php occ twofactorauth:gateway:configure sms
Please choose a SMS provider (websms, playsms, clockworksms, puzzelsms, ecallsms, voipms, huawei_e3531, spryng): sms77

In ProviderFactory.php line 59:

Provider does not exist

twofactorauth:gateway:configure

www-data@svrhnextcloud:/var/www/nextcloud$

Any further hints?
Thank you!

Kind regards

Alex

@itverwaltung commented on GitHub (Dec 12, 2019): Hi Christoph, i did as you said, but no luck: www-data@svrhnextcloud:/var/www/nextcloud$ php occ twofactorauth:gateway:configure sms Please choose a SMS provider (websms, playsms, clockworksms, puzzelsms, ecallsms, voipms, huawei_e3531, spryng): sms77 In ProviderFactory.php line 59: Provider <sms77> does not exist twofactorauth:gateway:configure <gateway> www-data@svrhnextcloud:/var/www/nextcloud$ Any further hints? Thank you! Kind regards Alex

kerem commented

2026-02-26 05:32:51 +03:00

Author

Owner

@ChristophWurst commented on GitHub (Dec 12, 2019):

Correct. There is another bug 🙈

Please submit another PR that changes github.com/nextcloud/twofactor_gateway@c7a23e6eb7/lib/Command/Configure.php (L239) to 'sms77io'. Then it should work.

(ref github.com/nextcloud/twofactor_gateway@c7a23e6eb7/lib/Service/Gateway/SMS/Provider/Sms77Io.php (L33))

@ChristophWurst commented on GitHub (Dec 12, 2019): Correct. There is another bug :see_no_evil: Please submit another PR that changes https://github.com/nextcloud/twofactor_gateway/blob/c7a23e6eb77fa5a44465d2f870a8e418e687b1ce/lib/Command/Configure.php#L239 to 'sms77io'. Then it should work. (ref https://github.com/nextcloud/twofactor_gateway/blob/c7a23e6eb77fa5a44465d2f870a8e418e687b1ce/lib/Service/Gateway/SMS/Provider/Sms77Io.php#L33)

kerem commented

2026-02-26 05:32:51 +03:00

Author

Owner

@itverwaltung commented on GitHub (Dec 13, 2019):

I tried to make a fork, a branch, edited the file, commited, and made a pull request. First time, have mercy ...

@itverwaltung commented on GitHub (Dec 13, 2019): I tried to make a fork, a branch, edited the file, commited, and made a pull request. First time, have mercy ...

kerem commented

2026-02-26 05:32:51 +03:00

Author

Owner

@ChristophWurst commented on GitHub (Dec 13, 2019):

No worries. We all started there :)

The DCO check failed. You might have to rebase the commits locally. Let me knew if you need help. You'll also find useful information at https://help.github.com/en/github

@ChristophWurst commented on GitHub (Dec 13, 2019): No worries. We all started there :) The DCO check failed. You might have to rebase the commits locally. Let me knew if you need help. You'll also find useful information at https://help.github.com/en/github

kerem commented

2026-02-26 05:32:51 +03:00

Author

Owner

@nursoda commented on GitHub (Jan 18, 2020):

Christoph, seriously? We are users (or admins) but not all of us are devs. For us it's hard enough to post decent bug reports. But requesting multiple PR from an oviously non-dev where it would have been two tiny code changes you already looked up seems insane to me.

@nursoda commented on GitHub (Jan 18, 2020): Christoph, seriously? We are users (or admins) but not all of us are devs. For us it's hard enough to post decent bug reports. But requesting multiple PR from an oviously non-dev where it would have been two tiny code changes you already looked up seems insane to me.

kerem commented

2026-02-26 05:32:51 +03:00

Author

Owner

@ChristophWurst commented on GitHub (Jan 20, 2020):

Wow, wait a sec. This app is a community effort. There is no obligation for anyone to maintain this project.

I've been very permissive towards the many community PRs that add support for edge case gateways. There is no way for me to test them, nor maintain it in a way that I would know if they work.

If you need enterprise support for this app you can check out https://nextcloud.com/enterprise/. But otherwise this is and will remain a community app.

Doing the maintenance of managing PRs, triaging issues and helping with problems is the most I can do. The change is minimal. But it takes me as much time as it would take you.

@ChristophWurst commented on GitHub (Jan 20, 2020): Wow, wait a sec. This app is a community effort. There is no obligation for anyone to maintain this project. I've been very permissive towards the many community PRs that add support for edge case gateways. There is no way for me to test them, nor maintain it in a way that I would know if they work. If you need enterprise support for this app you can check out https://nextcloud.com/enterprise/. But otherwise this is and will remain a community app. Doing the maintenance of managing PRs, triaging issues and helping with problems is the most I can do. The change is minimal. But it takes me as much time as it would take you.

kerem commented

2026-02-26 05:32:51 +03:00

Author

Owner

@ChristophWurst commented on GitHub (Jan 20, 2020):

And you don't have to code yourself if you're not a programmer. Just ask or hire someone to do it if it's important.

@ChristophWurst commented on GitHub (Jan 20, 2020): And you don't have to code yourself if you're not a programmer. Just ask or hire someone to do it if it's important.

kerem commented

2026-02-26 05:32:51 +03:00

Author

Owner

@nursoda commented on GitHub (Jan 20, 2020):

Fair enough. Thanks for committing. I confirm that sms77.io works fine :-)

@nursoda commented on GitHub (Jan 20, 2020): Fair enough. Thanks for committing. I confirm that sms77.io works fine :-)

kerem referenced this issue

2026-02-26 05:33:52 +03:00

[PR #70] [CLOSED] Bump vue-loader from 14.2.2 to 15.2.6 #186

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/twofactor_gateway-nextcloud#70

No description provided.

Rows
Columns