Feature #23493
closedconfig: strip/escape single-quotes in values when setting them via conf file/assimilate conf
0%
Description
At the moment, the config parsing state machine does not account for single-quotes as potential value enclosures, as it does for double-quotes.
This can raise issues similar to what was observed in #23345.
Filing the ticket for future reference: we should strip all single-quotes from a value EXCEPT when they are escaped.
E.g.,
foo = 'bar' 'baz'
should result in
foo = bar baz
Whereas,
foo = 'bar\' \'baz'
should result in
foo = bar' 'baz
Also, the following would be considered a malformed value:
foo = 'bar\' \'baz\'
for lack of closing single-quote.
Updated by Greg Farnum about 6 years ago
- Tracker changed from Bug to Feature
- Category changed from Correctness/Safety to Administration/Usability
- Component(RADOS) Monitor added
Updated by Joao Eduardo Luis about 6 years ago
- Related to Bug #23345: `ceph osd erasure-code-profile set` crashes the monitors on vstart clusters added
Updated by Brad Hubbard over 5 years ago
Joao,
Could you take a look at https://github.com/ceph/ceph/pull/20610 and see whether you consider it something that could be expanded to cover the more general case?
Updated by Kefu Chai almost 5 years ago
- Pull request ID set to 28634
i think https://github.com/ceph/ceph/pull/28634 is a small step in the right direction.
foo = 'bar' 'baz'
is not supported yet though.
Updated by Laura Flores over 1 year ago
- Translation missing: en.field_tag_list set to low-hanging-fruit