OCC command exits with 0 for the not existing user

owncloud / twofactor_totp

🔑 Second factor TOTP (Google Authenticator) provider for ownCloud

GNU Affero General Public License v3.0

9 stars 9 forks source link

Open skshetry opened 5 years ago

skshetry commented 5 years ago

As per the title, if we invoke occ command:

./occ twofactor_totp:set-secret-verification-status -u not-existing-user false

then, the occ command exits with 0 code.

P.S. Also, the error are printed to standard output, should that not be to stderr?

phil-davis commented 5 years ago

This might be very easy to fix. I did a similar thing last week in https://github.com/owncloud/ransomware_protection/pull/161 - so look there and maybe do similar.