Skip to content
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

[FLOSS] simpleini plugin fails silently on meta-set #4583

Closed
Tracked by #4555
Bujuhu opened this issue Oct 19, 2022 · 4 comments
Closed
Tracked by #4555

[FLOSS] simpleini plugin fails silently on meta-set #4583

Bujuhu opened this issue Oct 19, 2022 · 4 comments

Comments

@Bujuhu
Copy link
Contributor

Bujuhu commented Oct 19, 2022

As discussed in #4555

Steps to Reproduce the Problem

$ sudo kdb mount ini user:/tests/ini ccode simpleini
# RET: 0
$ kdb meta-set user:/tests/ini/key key value
# RET: 0  
$ kdb meta-get user:/tests/ini/key key
Key not found
# RET: 1

Expected Result

Should print an error on kdb meta-set user:/tests/ini/key value value and return a non 0 exit code

Actual Result

$ kdb meta-set user:/tests/ini/key key value fails silently and kdb meta-get user:/tests/ini/key value returns Key not found

System Information

Operating System:
Arch Linux, Kernel 5.19.13-arch1-1

kdb and plugins version

# kdb --version
KDB_VERSION: 0.9.11
SO_VERSION: 5

Relevant Source Code

Like mini, simpleini states in its documentation that it does not support metadata as well, but still won't reject metadata user input

int elektraSimpleiniSet (Plugin * handle ELEKTRA_UNUSED, KeySet * returned, Key * parentKey)

@Bujuhu Bujuhu changed the title [Floss H1] simpleini plugin fails silently on meta-set [Floss] simpleini plugin fails silently on meta-set Oct 19, 2022
@Bujuhu Bujuhu changed the title [Floss] simpleini plugin fails silently on meta-set [FLOSS] simpleini plugin fails silently on meta-set Oct 19, 2022
@kodebach
Copy link
Member

It should be kdb meta-get user:/tests/ini/key key in the example.

@Bujuhu
Copy link
Contributor Author

Bujuhu commented Oct 19, 2022

Fixed it. Thank you!

@Dynamichost96 Dynamichost96 self-assigned this Dec 12, 2022
Copy link

I mark this stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping by writing a message here or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@github-actions github-actions bot added the stale label Dec 13, 2023
Copy link

I closed this now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants