-
Notifications
You must be signed in to change notification settings - Fork 54
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
something wrong with decode json kv.value #126
Comments
I just figure this out by looking into the source code, require this when new etcd instance, not sure is the correct way to handle this. |
@Yiyiyimu |
Hi @findmark sorry I lost track on this issue, hope it's not too late 😬 Actually, the accepted option for lua-resty-etcd/t/v3/serializer.t Lines 58 to 61 in 16a5733
And yes the documentation is a bit confusing 🤦 If you got time, welcome draft a PR to refine it~ |
* Fix api7#126 * cause to conform to v3.md Signed-off-by: u5surf <[email protected]>
* Fix api7#126 * cause to conform to v3.md Signed-off-by: u5surf <[email protected]>
* Fix api7#126 * cause to conform to v3.md Signed-off-by: u5surf <[email protected]>
* Fix api7#126 * cause to conform to v3.md Signed-off-by: u5surf <[email protected]>
hi, I use the newest version but I got the following issue:
2021/03/24 09:25:19 [error] 6112#54059: *13 lua entry thread aborted: runtime error: /usr/local/share/lua/5.1/resty/etcd/v3.lua:487: attempt to index field 'serializer' (a nil value)
I believe serializer default is json as the doc mention in v3.md. Nvm, I set it to json manually, but I got error message instead , I believe is the same issue.
2021/03/24 09:20:19 [error] 5976#50240: *13 lua entry thread aborted: runtime error: /usr/local/share/lua/5.1/resty/etcd/v3.lua:487: attempt to call field 'deserialize' (a nil value)
Could you help look into this? Thanks in advance
The text was updated successfully, but these errors were encountered: