fix(richdocuments): Mark some parameters of richdocuments WopiMapper as sensitive

[nickvergessen]

Found in our own logs

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[rullzer]

I do get the quick fix. However for future proofing this (and quicker updates). It would be great if there is a way for apps to register their own sensitive parameters. That way it would scale better and you don't have to wait for a server update each time.

[blizzz]

I do get the quick fix. However for future proofing this (and quicker updates). It would be great if there is a way for apps to register their own sensitive parameters. That way it would scale better and you don't have to wait for a server update each time.

We have that: https://github.com/nextcloud/server/blob/master/lib/public/AppFramework/Bootstrap/IRegistrationContext.php#L352

[nickvergessen]

I do get the quick fix. However for future proofing this (and quicker updates). It would be great if there is a way for apps to register their own sensitive parameters. That way it would scale better and you don't have to wait for a server update each time.

Well once PHP 8.2 is the minimum, the correct way is using the native PHP attribute (which I added in nextcloud/richdocuments#3042 )

I was also not aware of the IRegisterContext function, so yeah will use that one instead.