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

fix(template) fix iterator in stream shdict #7078

Merged
merged 2 commits into from
Apr 29, 2021
Merged

Conversation

fffonion
Copy link
Contributor

Summary

kong.pdk.node iterrates through ngx.shared magic table to
get list of shdict metrics, implement the __pairs metamethod so
it's happy.

Full changelog

  • fix(template) fix iterrator in stream shdict

Issues resolved

See also Kong/kong-plugin-prometheus#118

`kong.pdk.node` iterrates through ngx.shared magic table to
get list of shdict metrics, implement the __pairs metamethod so
it's happy.
@fffonion fffonion changed the title fix(template) fix iterrator in stream shdict fix(template) fix iterator in stream shdict Apr 22, 2021
Copy link
Member

@dndx dndx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comment, otherwise LGTM.

kong/templates/nginx_kong_stream.lua Outdated Show resolved Hide resolved
Co-authored-by: Datong Sun <datong.sun@konghq.com>
@fffonion fffonion merged commit 875ba15 into master Apr 29, 2021
@fffonion fffonion deleted the fix/stream-shdict-pairs branch April 29, 2021 08:57
gszr pushed a commit that referenced this pull request May 10, 2021
`kong.pdk.node` iterrates through ngx.shared magic table to
get list of shdict metrics, implement the __pairs metamethod so
it's happy.

Co-authored-by: Datong Sun <datong.sun@konghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core/streams Refers to the streams subsystem core/templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants