Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

[FLOSS] yajl plugin fails silently on meta-set #4581

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

[FLOSS] yajl plugin fails silently on meta-set #4581

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

Comments

@Bujuhu
Copy link
Contributor

Bujuhu commented Oct 19, 2022

As discussed in #4555

Steps to Reproduce the Problem

$ sudo kdb mount config.json user:/tests/yajl yajl
# RET: 0
$ kdb meta-set user:/tests/yajl/roots 'a' 'b'
# RET: 0  
$ kdb meta-get user:/tests/yajl/roots 'a'
Key not found
# RET: 2

Expected Result

Should either print an error on kdb meta-set user:/tests/yajl/roots 'a' 'b' and return a non 0 exit code or
kdb meta-get user:/tests/yajl/roots 'a' should return b

Actual Result

$ kdb meta-set user:/tests/yajl/roots 'a' 'b' fails silently and kdb meta-get user:/tests/yajl/roots 'a' returns a key not found error

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

yajl only checks for the array and type meta keys. All others are ignored

const char * meta = keyString (keyGetMeta (cur, "array"));

const Key * type = keyGetMeta (cur, "type");

@Bujuhu Bujuhu changed the title [Floss H1] yajl plugin fails silently on meta-set [Floss] yajl plugin fails silently on meta-set Oct 19, 2022
@Bujuhu Bujuhu changed the title [Floss] yajl plugin fails silently on meta-set [FLOSS] yajl 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 12, 2022
Dynamichost96 pushed a commit to Dynamichost96/libelektra that referenced this issue Jan 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant