remove fmt.Print statement from decomposedfs

[wkloucek]

fixes owncloud/ocis#2290

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[wkloucek]

@butonic what should we do here? There is no logging available. Can we just remove the Print statements?

[butonic]

The correct implementation would check

1. if the symlink already exists
2. if it points to the correct node (if not overwrite the existing symlink)
3. if the symlink cannot be created ... log an error ... hah the trick is

   reva/pkg/auth/manager/ldap/ldap.go

   Line 113 in b1d57b9

   logger.New().Warn().Msg("userfilter is deprecated, use a loginfilter like `(&(objectclass=posixAccount)(|(cn={{login}}))(mail={{login}}))`")

it will use the default log configuration, but at least it is a structured log message.

and the left over case really is an error the admin should tackle, because the correct symlink cannot be created.

[wkloucek]

The correct implementation would check

1. if the symlink already exists
2. if it points to the correct node (if not overwrite the existing symlink)
3. if the symlink cannot be created ... log an error ... hah the trick is

   reva/pkg/auth/manager/ldap/ldap.go

   Line 113 in b1d57b9

   logger.New().Warn().Msg("userfilter is deprecated, use a loginfilter like `(&(objectclass=posixAccount)(|(cn={{login}}))(mail={{login}}))`")

it will use the default log configuration, but at least it is a structured log message.

and the left over case really is an error the admin should tackle, because the correct symlink cannot be created.

Doesn't sound like just replacing the Print with a Log. Could you please take care?

[butonic]

closing in favor of #1992