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


#21

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.


#22

@jon,

I havent started with the implementation as yet. I wanted an agreement before I proceed.

Thanks,
Prakash


#23

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!


#24

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