New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ini plugin: separator char in value string not working #1721

Closed
BernhardDenner opened this Issue Dec 9, 2017 · 0 comments

Comments

Projects
None yet
3 participants
@BernhardDenner
Contributor

BernhardDenner commented Dec 9, 2017

When passing a value containing the separator character “=”, the behavior of the “ini”
storage plugin is unaccountable. The complete value enclosed in double quotes is written
to the file. However, an additional property, consisting of the separator character enclosed
in double quotes, with an empty value is added too. Also, adding the separator character
to a property name leads to an unexpected behavior. A separator character
within a section name is handled correctly.

Steps to Reproduce the Problem

All steps were done in Docker with Ubuntu 16.04 based image with elektra current master (f6f7952).

kdb mount /root/test.ini system/tests/ini ini
cat > test.ini  <<EOF
[section1]
hello = world
EOF

kdb set system/tests/ini/section1/hello 'a + b = b + a'
#> Set string to "a + b = b + a"

kdb ls system/tests/ini
#> system/tests/ini/section1
#> system/tests/ini/section1/=

kdb get system/tests/ini/section1/hello
#> Did not find key

cat test.ini 
#> [section1]
#> hello = "a + b = b + a"

@BernhardDenner BernhardDenner referenced this issue Dec 9, 2017

Open

bugs in storage plugins #1718

0 of 2 tasks complete

@markus2330 markus2330 added this to the 0.8.21 milestone Dec 9, 2017

@markus2330 markus2330 modified the milestones: 0.8.21, 0.8.22 Dec 22, 2017

@sanssecours sanssecours self-assigned this Feb 13, 2018

sanssecours added a commit to sanssecours/elektra that referenced this issue Feb 16, 2018

INI: Parse line with multiple `=` differently
The new strategy is **far from ideal**, since it does not allow us to
save certain keys properly. Anyway, at least it should now be possibly
to save keys and values containing delimiter characters (`=`).

This commit closes ElektraInitiative#1721.

sanssecours added a commit to sanssecours/elektra that referenced this issue Feb 17, 2018

INI: Parse line with multiple `=` differently
The new strategy is **far from ideal**, since it does not allow us to
save certain keys properly. Anyway, at least it should now be possibly
to save keys and values containing delimiter characters (`=`).

This commit closes ElektraInitiative#1721.

sanssecours added a commit to sanssecours/elektra that referenced this issue Feb 17, 2018

INI: Parse line with multiple `=` differently
The new strategy is **far from ideal**, since it does not allow us to
save certain keys properly. Anyway, at least it should now be possibly
to save keys and values containing delimiter characters (`=`).

This commit closes ElektraInitiative#1721.

sanssecours added a commit to sanssecours/elektra that referenced this issue Feb 17, 2018

INI: Parse line with multiple `=` differently
The new strategy is **far from ideal**, since it does not allow us to
save certain keys properly. Anyway, at least it should now be possibly
to save keys and values containing delimiter characters (`=`).

This commit closes ElektraInitiative#1721.

sanssecours added a commit to sanssecours/elektra that referenced this issue Feb 17, 2018

INI: Parse line with multiple `=` differently
The new strategy is **far from ideal**, since it does not allow us to
save certain keys properly. Anyway, at least it should now be possibly
to save keys and values containing delimiter characters (`=`).

This commit closes ElektraInitiative#1721.

@sanssecours sanssecours referenced this issue Feb 17, 2018

Merged

Various Small Fixes #1818

4 of 4 tasks complete

@markus2330 markus2330 modified the milestones: 0.8.22, 0.8.23 Feb 24, 2018

sanssecours added a commit to sanssecours/elektra that referenced this issue Mar 2, 2018

INI: Parse line with multiple `=` differently
The new strategy is **far from ideal**, since it does not allow us to
save certain keys properly. Anyway, at least it should now be possibly
to save keys and values containing delimiter characters (`=`).

This commit closes ElektraInitiative#1721.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment