From abcb3166f667068c254ce6037a89b60df9182c4c Mon Sep 17 00:00:00 2001
From: "Zaiming (Stone) Shi" <zmstone@gmail.com>
Date: Tue, 24 Oct 2023 14:46:01 +0200
Subject: [PATCH] fix: use binary string for raw default values

---
 apps/emqx_bridge_kafka/src/emqx_bridge_kafka.erl     |  2 +-
 apps/emqx_conf/src/emqx_conf_schema.erl              |  2 +-
 apps/emqx_ft/src/emqx_ft_schema.erl                  | 12 ++++++------
 .../src/emqx_opentelemetry.app.src                   |  2 +-
 apps/emqx_opentelemetry/src/emqx_otel_schema.erl     |  2 +-
 apps/emqx_s3/src/emqx_s3.app.src                     |  2 +-
 apps/emqx_s3/src/emqx_s3_schema.erl                  |  6 +++---
 7 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/apps/emqx_bridge_kafka/src/emqx_bridge_kafka.erl b/apps/emqx_bridge_kafka/src/emqx_bridge_kafka.erl
index 6b3f3cd64..5972ba323 100644
--- a/apps/emqx_bridge_kafka/src/emqx_bridge_kafka.erl
+++ b/apps/emqx_bridge_kafka/src/emqx_bridge_kafka.erl
@@ -444,7 +444,7 @@ fields(consumer_kafka_opts) ->
     [
         {max_batch_bytes,
             mk(emqx_schema:bytesize(), #{
-                default => "896KB", desc => ?DESC(consumer_max_batch_bytes)
+                default => <<"896KB">>, desc => ?DESC(consumer_max_batch_bytes)
             })},
         {max_rejoin_attempts,
             mk(non_neg_integer(), #{
diff --git a/apps/emqx_conf/src/emqx_conf_schema.erl b/apps/emqx_conf/src/emqx_conf_schema.erl
index 382d25de1..6f3964e17 100644
--- a/apps/emqx_conf/src/emqx_conf_schema.erl
+++ b/apps/emqx_conf/src/emqx_conf_schema.erl
@@ -953,7 +953,7 @@ fields("rpc") ->
             sc(
                 string(),
                 #{
-                    default => "0.0.0.0",
+                    default => <<"0.0.0.0">>,
                     desc => ?DESC(rpc_listen_address),
                     importance => ?IMPORTANCE_MEDIUM
                 }
diff --git a/apps/emqx_ft/src/emqx_ft_schema.erl b/apps/emqx_ft/src/emqx_ft_schema.erl
index dd21e9524..d9187f14e 100644
--- a/apps/emqx_ft/src/emqx_ft_schema.erl
+++ b/apps/emqx_ft/src/emqx_ft_schema.erl
@@ -77,7 +77,7 @@ fields(file_transfer) ->
                     desc => ?DESC("init_timeout"),
                     required => false,
                     importance => ?IMPORTANCE_LOW,
-                    default => "10s"
+                    default => <<"10s">>
                 }
             )},
         {store_segment_timeout,
@@ -87,7 +87,7 @@ fields(file_transfer) ->
                     desc => ?DESC("store_segment_timeout"),
                     required => false,
                     importance => ?IMPORTANCE_LOW,
-                    default => "5m"
+                    default => <<"5m">>
                 }
             )},
         {assemble_timeout,
@@ -97,7 +97,7 @@ fields(file_transfer) ->
                     desc => ?DESC("assemble_timeout"),
                     required => false,
                     importance => ?IMPORTANCE_LOW,
-                    default => "5m"
+                    default => <<"5m">>
                 }
             )},
         {storage,
@@ -208,7 +208,7 @@ fields(local_storage_segments_gc) ->
                 #{
                     desc => ?DESC("storage_gc_interval"),
                     required => false,
-                    default => "1h"
+                    default => <<"1h">>
                 }
             )},
         {maximum_segments_ttl,
@@ -218,7 +218,7 @@ fields(local_storage_segments_gc) ->
                 #{
                     desc => ?DESC("storage_gc_max_segments_ttl"),
                     required => false,
-                    default => "24h"
+                    default => <<"24h">>
                 }
             )},
         {minimum_segments_ttl,
@@ -228,7 +228,7 @@ fields(local_storage_segments_gc) ->
                 #{
                     desc => ?DESC("storage_gc_min_segments_ttl"),
                     required => false,
-                    default => "5m",
+                    default => <<"5m">>,
                     % NOTE
                     % This setting does not seem to be useful to an end-user.
                     hidden => true
diff --git a/apps/emqx_opentelemetry/src/emqx_opentelemetry.app.src b/apps/emqx_opentelemetry/src/emqx_opentelemetry.app.src
index d459fc107..3b429ea08 100644
--- a/apps/emqx_opentelemetry/src/emqx_opentelemetry.app.src
+++ b/apps/emqx_opentelemetry/src/emqx_opentelemetry.app.src
@@ -1,6 +1,6 @@
 {application, emqx_opentelemetry, [
     {description, "OpenTelemetry for EMQX Broker"},
-    {vsn, "0.1.2"},
+    {vsn, "0.1.3"},
     {registered, []},
     {mod, {emqx_otel_app, []}},
     {applications, [kernel, stdlib, emqx]},
diff --git a/apps/emqx_opentelemetry/src/emqx_otel_schema.erl b/apps/emqx_opentelemetry/src/emqx_otel_schema.erl
index 1479009a2..7fae5071e 100644
--- a/apps/emqx_opentelemetry/src/emqx_otel_schema.erl
+++ b/apps/emqx_opentelemetry/src/emqx_otel_schema.erl
@@ -62,7 +62,7 @@ fields("exporter") ->
                 emqx_schema:url(),
                 #{
                     mapping => "opentelemetry_exporter.otlp_endpoint",
-                    default => "http://localhost:4317",
+                    default => <<"http://localhost:4317">>,
                     desc => ?DESC(endpoint)
                 }
             )},
diff --git a/apps/emqx_s3/src/emqx_s3.app.src b/apps/emqx_s3/src/emqx_s3.app.src
index 0e3cd9220..ba94f66e1 100644
--- a/apps/emqx_s3/src/emqx_s3.app.src
+++ b/apps/emqx_s3/src/emqx_s3.app.src
@@ -1,6 +1,6 @@
 {application, emqx_s3, [
     {description, "EMQX S3"},
-    {vsn, "5.0.9"},
+    {vsn, "5.0.10"},
     {modules, []},
     {registered, [emqx_s3_sup]},
     {applications, [
diff --git a/apps/emqx_s3/src/emqx_s3_schema.erl b/apps/emqx_s3/src/emqx_s3_schema.erl
index 7d6badaf5..db37c6e2d 100644
--- a/apps/emqx_s3/src/emqx_s3_schema.erl
+++ b/apps/emqx_s3/src/emqx_s3_schema.erl
@@ -74,7 +74,7 @@ fields(s3) ->
                 %% not used in a `receive ... after' block, just timestamp comparison
                 emqx_schema:duration_s(),
                 #{
-                    default => "1h",
+                    default => <<"1h">>,
                     desc => ?DESC("url_expire_time"),
                     required => false
                 }
@@ -83,7 +83,7 @@ fields(s3) ->
             mk(
                 emqx_schema:bytesize(),
                 #{
-                    default => "5mb",
+                    default => <<"5mb">>,
                     desc => ?DESC("min_part_size"),
                     required => true,
                     validator => fun part_size_validator/1
@@ -93,7 +93,7 @@ fields(s3) ->
             mk(
                 emqx_schema:bytesize(),
                 #{
-                    default => "5gb",
+                    default => <<"5gb">>,
                     desc => ?DESC("max_part_size"),
                     required => true,
                     validator => fun part_size_validator/1