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] mini plugin fails silently on meta-set #4582

Closed
Tracked by #4555
Bujuhu opened this issue Oct 19, 2022 · 0 comments · Fixed by #4729
Closed
Tracked by #4555

[FLOSS] mini plugin fails silently on meta-set #4582

Bujuhu opened this issue Oct 19, 2022 · 0 comments · Fixed by #4729
Assignees

Comments

@Bujuhu
Copy link
Contributor

Bujuhu commented Oct 19, 2022

As discussed in #4555

Steps to Reproduce the Problem

$ sudo kdb mount mini.ini user:/tests/mini mini
# RET: 0
$ kdb meta-set user:/tests/mini/key value value
# RET: 0  
$ kdb meta-get user:/tests/mini/key value
Metakey not found
# RET: 2

Expected Result

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

Actual Result

$ kdb meta-set user:/tests/mini/key value value fails silently and kdb meta-get user:/tests/mini/key value returns Metakey 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

While mini explicitly states in its documentation that it does not support metadata, it still doesn't have any warning if a user tries to set a meta key

static inline int writeFile (FILE * file, KeySet * keySet, Key * parentKey)

@Bujuhu Bujuhu changed the title [Floss H1] mini plugin fails silently on meta-set [Floss] mini plugin fails silently on meta-set Oct 19, 2022
@Bujuhu Bujuhu changed the title [Floss] mini plugin fails silently on meta-set [FLOSS] mini plugin fails silently on meta-set Oct 19, 2022
@Bujuhu Bujuhu self-assigned this Nov 29, 2022
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Nov 29, 2022
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Nov 29, 2022
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Dec 11, 2022
@Dynamichost96 Dynamichost96 self-assigned this Dec 12, 2022
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Dec 20, 2022
Bujuhu added a commit to Bujuhu/libelektra that referenced this issue Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants