Project

General

Profile

Actions

Feature #23493

closed

config: strip/escape single-quotes in values when setting them via conf file/assimilate conf

Added by Joao Eduardo Luis about 6 years ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Administration/Usability
Target version:
-
% Done:

0%

Source:
Tags:
low-hanging-fruit
Backport:
Reviewed:
Affected Versions:
Component(RADOS):
Monitor
Pull request ID:

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.


Related issues 1 (0 open1 closed)

Related to RADOS - Bug #23345: `ceph osd erasure-code-profile set` crashes the monitors on vstart clustersResolvedJoao Eduardo Luis03/13/2018

Actions
Actions

Also available in: Atom PDF