Skip to content

Commit

Permalink
fix(propagation): remove redundant warning messages (#13220)
Browse files Browse the repository at this point in the history
The propagation module logs some redundant warning messages that are
already printed, when necessary, during schema validation, with logic
that also checks the "old default" value. This commit removes the
unnecessary messages.
  • Loading branch information
samugi authored Jun 17, 2024
1 parent 3c0aa60 commit 8b6186c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 9 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
message: "**OpenTelemetry / Zipkin**: remove redundant deprecation warnings"
type: bugfix
scope: Plugin
9 changes: 0 additions & 9 deletions kong/tracing/propagation/init.lua
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,6 @@ local function get_plugin_params(config)

if (config.default_header_type or null) ~= null then
propagation_config.default_format = config.default_header_type

kong.log.warn(
"the default_header_type parameter is deprecated, please update your "
.. "configuration to use the propagation.default_format, "
.. "propagation.extract and propagation.inject options instead")
end

if (config.header_type or null) ~= null then
Expand Down Expand Up @@ -75,10 +70,6 @@ local function get_plugin_params(config)
config.header_type
}
end

kong.log.warn(
"the header_type parameter is deprecated, please update your "
.. "configuration to use propagation.extract and propagation.inject instead")
end

return propagation_config
Expand Down

1 comment on commit 8b6186c

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

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

Bazel Build

Docker image available kong/kong:8b6186c134ad4b32925cb78f64656c385e3a01d0
Artifacts available https://github.com/Kong/kong/actions/runs/9550620458

Please sign in to comment.