-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] master from Kong:master #69
Open
pull
wants to merge
4,426
commits into
Mu-L:master
Choose a base branch
from
Kong:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+286,426
−57,644
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
f089a5b
to
62d113d
Compare
* fix(core): `node_id` in configuration is deprecated in this PR, which means it is not recommended to be specified manually. A warning message is printed in the error log if the `node_id` is recognized in the configuration. FTI-6221
…ons (#13863) Changed error log level from ERROR to WARN for better clarity and to avoid alarming users unnecessarily when control plane connections break. This adjustment enhances user experience under connection interruptions. Fix:[FTI-6238](https://konghq.atlassian.net/browse/FTI-6238) Signed-off-by: tzssangglass <[email protected]>
* feat(proxy-wasm): add host properties 'kong.service_name' and 'kong.route_name' * feat(prometheus): Proxy-Wasm metrics
This is just for local development, engineers can easily cache some targets during the development instead of rebuild everything.
ATC-Router requires rust version >= 1.81.0, so we bump to 1.82.0
### Summary This release contains multiple improvements, including a better way of validating expressions and interpreter performance optimizations. ### Changelog https://github.com/Kong/atc-router/releases/tag/v1.7.0 ### Full Changelog Kong/atc-router@v1.6.2...v1.7.0 KAG-5873 --------- Co-authored-by: Qi <[email protected]>
`kong_config.resurrect_ttl` is a invalid value, actually it should be `kong_config.db_resurrect_ttl`
Signed-off-by: Aapo Talvensaari <[email protected]>
…14266) * chore(tests): stabilise flaky tests of response ratelimiting * chore(tests): fix changelog * chore(tests): fix temp test tag
### Summary 1. flatten validation error for sync deltas 2. call RPC notify to API `kong.sync.v2.notify_validation_error` (spec: Kong/openrpc_specfiles#18) 3. fix bug that CP will crash if DP calls a non-existent RPC notify: KAG-6381 KAG-6351, KAG-5898, KAG-6381
### Summary In #13168 I made this already faster, but this time I managed to make it a way faster. Signed-off-by: Aapo Talvensaari <[email protected]>
Signed-off-by: Aapo Talvensaari <[email protected]>
I merged the PR Kong/kong-ee#10652 prematurely. This adds a couple of log messages back for testing. Signed-off-by: Aapo Talvensaari <[email protected]>
Signed-off-by: Aapo Talvensaari <[email protected]>
…%20 instead of + (#14277) ### Summary The `+` encoding is more correct (in query strings), but it seems to cause some problems with some users. As a short-term solution I propose that we just convert `+` to `%20`. This is more in-lines what Nginx and OpenResty seems to be doing as well. Signed-off-by: Aapo Talvensaari <[email protected]>
…st (#14155) FTI-6314 --------- Co-authored-by: Zachary Hu <[email protected]>
### Summary See: https://github.com/PCRE2Project/pcre2/blob/pcre2-10.45/NEWS Signed-off-by: Aapo Talvensaari <[email protected]>
Precondition of KAG-6296
This module could throw errors if certain validations fail. While this is useful for type validation, it can be risky when the error logic depends on more complex conditions. This commit removes the error thrown when a span's end time is earlier than its start time. Although this shouldn't happen under normal circumstances, the end time calculation depends on multiple factors, including values from Nginx, making it not trivial to ensure it is always valid. In such cases, logging an error is more appropriate than failing the request.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )