I am fine with the changes as long as they do not cause issues if a admin encapsulates a json string in a custom resource. If this has been tested and works as expected then I am fine with the proposed changes.
I havent started with the implementation as yet. I wanted an agreement before I proceed.
To summarize when escaping/quoting will take place:
- Attributes/values created by PBS will not be modified
- Attributes/values submitted by user will be escaped/quoted
- Attributes that already have surrounding quotes will not have another layer of quotes added
Sounds good to me!