Skip to content

Commit

Permalink
Merge pull request #9406 from lafirest/fix/max_topic_level
Browse files Browse the repository at this point in the history
fix: change the defualt value of `max_topic_levels` to 128
  • Loading branch information
lafirest committed Nov 22, 2022
2 parents 5cd31e4 + 461aa7f commit f44e70a
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion apps/emqx/src/emqx_schema.erl
Expand Up @@ -399,7 +399,7 @@ fields("mqtt") ->
sc(
range(1, 65535),
#{
default => 65535,
default => 128,
desc => ?DESC(mqtt_max_topic_levels)
}
)},
Expand Down
1 change: 1 addition & 0 deletions changes/v5.0.11-en.md
Expand Up @@ -19,6 +19,7 @@

- Improve node name generation rules to avoid potential atom table overflow risk [#9387](https://github.com/emqx/emqx/pull/9387).

- Set the default value for the maximum level of a topic to 128 [#9406](https://github.com/emqx/emqx/pull/9406).

## Bug fixes

Expand Down
2 changes: 2 additions & 0 deletions changes/v5.0.11-zh.md
Expand Up @@ -17,6 +17,8 @@

- 改进了节点名称生成规则,以避免潜在的原子表溢出风险 [#9387](https://github.com/emqx/emqx/pull/9387)

- 将主题的最大层级限制的默认值设置为128 [#9406](https://github.com/emqx/emqx/pull/9406)

## 修复

- 修复 helm chart 的 `ssl.existingName` 选项不起作用 [#9307](https://github.com/emqx/emqx/issues/9307)
Expand Down

0 comments on commit f44e70a

Please sign in to comment.