Missing deprecation of non-array list for keys and excludeKeys

[mtrezza]

New Issue Checklist

• I am not disclosing a vulnerability.
• I am not just asking a question.
• I have searched through existing issues.
• I can reproduce the issue with the latest version of Parse Server.

Issue Description

Issue #7245 deprecated comma separated string lists in favor of arrays, starting with Parse Server 5. The deprecation however is not implemented in code (via runtime deprecation) and documented in the deprecation table.

In order remove the deprecation in the future, as per policy it has to be officially communicated for a full major release (1 year). If the deprecation is officially added before Parse Server 5 release, the removal can happen earliest with Parse Server 6 (2023).

Steps to reproduce

n/a

Actual Outcome

n/a

Expected Outcome

n/a

Environment

n/a

Logs

n/a

[parse-github-assistant]

Thanks for opening this issue!

• 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.

[cbaker6]

I didn’t design the 7245 to deprecate a comma separated string because most of the client sdk’s are designed to that flaw. Instead, I designed the PR to work with an array of strings and the comma separated string which is why the documentation and my comments in the PR are written the way they are (also why I kept all of the old tests).

Deprecating will require PR’s in every client SDK in the future (except for the Swift SDK, I never designed it to work with the flaw) and essentially make newer versions of the respective SDKs not backwards compatible with servers <5.0.0. In the current state after my PR, all SDKs are backwards and forwards comparable. I suspect you will cause more harm than good by deprecating and it will probably be easier to keep both.

[mtrezza]

So we only deprecate the string once all SDKs are changed. I will flag this as "on hold". Unless there is another good reason to support strings in addition to arrays indefinitely.