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(wasm): execute filter plugins in a consistent order #13946

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

flrgh
Copy link
Contributor

@flrgh flrgh commented Nov 28, 2024

Summary

Before this change, execution order of filter plugins was subject to ordering returned by the underling DB implementation of kong.db.plugins:each().

This adds an extra step that sorts all discovered filter plugins by name so that execution order is more consistent.

Checklist

  • The Pull Request has tests
  • A changelog file has been created
  • There is a user-facing docs PR

Issue reference

KAG-5851

Before this change, execution order of filter plugins was subject to
ordering returned by the underling DB implementation of
`kong.db.plugins:each()`.

This adds an extra step that sorts all discovered filter plugins by name
so that execution order is more consistent.
@flrgh flrgh added this to the 3.9.0 milestone Nov 28, 2024
@github-actions github-actions bot added core/proxy cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Nov 28, 2024
@github-actions github-actions bot added the core/wasm Everything relevant to [proxy-]wasm label Nov 28, 2024
@flrgh flrgh merged commit 0d59bd0 into master Dec 2, 2024
47 checks passed
@flrgh flrgh deleted the fix/wasm-filter-plugins-order branch December 2, 2024 17:23
github-actions bot pushed a commit that referenced this pull request Dec 2, 2024
Before this change, execution order of filter plugins was subject to
ordering returned by the underling DB implementation of
`kong.db.plugins:each()`.

This adds an extra step that sorts all discovered filter plugins by name
so that execution order is more consistent.

(cherry picked from commit 0d59bd0)
@team-gateway-bot
Copy link
Collaborator

Successfully created cherry-pick PR for master:

@team-gateway-bot
Copy link
Collaborator

Successfully created backport PR for release/3.9.x:

ProBrian pushed a commit that referenced this pull request Dec 13, 2024
Before this change, execution order of filter plugins was subject to
ordering returned by the underling DB implementation of
`kong.db.plugins:each()`.

This adds an extra step that sorts all discovered filter plugins by name
so that execution order is more consistent.
lhanjian pushed a commit that referenced this pull request Dec 23, 2024
)

Before this change, execution order of filter plugins was subject to
ordering returned by the underling DB implementation of
`kong.db.plugins:each()`.

This adds an extra step that sorts all discovered filter plugins by name
so that execution order is more consistent.

(cherry picked from commit 0d59bd0)

Co-authored-by: Michael Martin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release/3.9.x cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee core/proxy core/wasm Everything relevant to [proxy-]wasm size/L skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants