PP-464: PBS Accounting Logs - Incorrect Escaping, Proposed Design posted


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!


This ticket is blocked on PP-745. Single node jobs are not sending json strings within single quotes.