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

Add support for plugin ordering #710

Merged
merged 1 commit into from
Jul 12, 2022
Merged

Add support for plugin ordering #710

merged 1 commit into from
Jul 12, 2022

Conversation

rainest
Copy link
Contributor

@rainest rainest commented Jul 6, 2022

Add support for the new plugin ordering field. Draft pending Kong/go-kong#187 and a go-kong release.

@GGabriele
Copy link
Collaborator

Looks great!

Would you mind adding an integration tests for kong version >=3.0.0 ?

@codecov-commenter
Copy link

codecov-commenter commented Jul 7, 2022

Codecov Report

Merging #710 (d331c8b) into main (f483570) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #710      +/-   ##
==========================================
- Coverage   43.27%   43.24%   -0.03%     
==========================================
  Files          74       74              
  Lines        8970     8976       +6     
==========================================
  Hits         3882     3882              
- Misses       4718     4723       +5     
- Partials      370      371       +1     
Impacted Files Coverage Δ
file/types.go 62.38% <0.00%> (-1.17%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f483570...d331c8b. Read the comment docs.

@rainest rainest force-pushed the feat/plugin-order branch 4 times, most recently from 1e9015b to c3b12d8 Compare July 7, 2022 23:08
@rainest
Copy link
Contributor Author

rainest commented Jul 7, 2022

Tested along with adding the Enterprise nightly. Still draft pending a go-kong release. This breaks one of the existing tests--it looks like the 3.0 defaults add websocket protocols--so that's now limited to under 3.0.

@rainest rainest force-pushed the feat/plugin-order branch from 31bc3e2 to b3babf5 Compare July 11, 2022 17:27
@rainest rainest marked this pull request as ready for review July 12, 2022 17:14
@rainest rainest requested a review from a team July 12, 2022 17:14
@rainest rainest merged commit 41131b8 into main Jul 12, 2022
@rainest rainest deleted the feat/plugin-order branch July 12, 2022 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants