Comment 2 for bug 1604148

Revision history for this message
Brad Morgan (morgabra) wrote :

Couple followups:

1) Does Ironic even want to support these characters in attribute keys? If no, this can be closed.

2) The tests here are probably unwanted, as they really just test the jsonpatch lib, but I left them in so you could see the use case that this fixes.

3) This is sort of a poor user experience if someone manages to get anything in the DB with these characters, even if this patch merges. (You'll need to know the magic '~0', '~1' to escape them.) What can we do about that?