Behaviour changes due to "Move user creation to a separate service"

[phil-davis]

After PR #35972 was merged to core master, password_policy tests started to fail due to some small changes in error reporting behaviour. See owncloud/password_policy#241 for the detailed test scenario changes needed.

1. When adding a user via the Provisioning API, if there is a PolicyException (e.g. the password does not meet some password policy requirement) then the OCS status message has the text Unable to create user due to exception: prepended to it - e.g.

Unable to create user due to exception: The password contains too few lowercase letters. At least 3 lowercase letters are required.

Previously the OCS status text was just:

The password contains too few lowercase letters. At least 3 lowercase letters are required.

Note: when changing a user's password via the Provisioning API the messages have not changed - the extra bit about the exception does not appear.

2. When adding a user using the occ command, then the text messages are unchanged. But they are coming on stdout rather than stderr.

Note: when changing a user password the error messages are still coming on stderr

What to do about each of these behaviour changes, and the inconsistencies between "add user" and "change password"?

[phil-davis]

@sharidas @micbar @DeepDiver1975 ?

I added to Server 10.3 project, because this needs to be discussed and either accepted or changed/fixed before release.

[PVince81]

Sounds like a regression.

@sharidas please make the status text look like it was before

[micbar]

@sharidas please add the previous exception https://github.com/owncloud/core/pull/35972/files#diff-a1f4d1e489770bca5e42e3d6572ba89bR128