yuanbiao
/
emqx
1
0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #8728 from lafirest/fix/subs_api 

fix(mgmt): fix `$queue` topic name error in this API return
This commit is contained in:
lafirest 2022-08-16 11:12:52 +08:00 committed by GitHub
parent 4fa961caeb 64b2cf25cc
commit c1adf04c9a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGES-5.0.md
View File

@ -3,6 +3,7 @@
## Bug fixes ## Bug fixes
* Remove the needless `will_msg` field from the client API. [#8721](https://github.com/emqx/emqx/pull/8721) * Remove the needless `will_msg` field from the client API. [#8721](https://github.com/emqx/emqx/pull/8721)
* Fix `$queue` topic name error in management API return
# 5.0.5 # 5.0.5

2
apps/emqx_management/src/emqx_mgmt_api_subscriptions.erl
View File

@ -177,6 +177,8 @@ format({_Subscriber, Topic, Options}) ->
maps:with([qos, nl, rap, rh], Options) maps:with([qos, nl, rap, rh], Options)
). ).
get_topic(Topic, #{share := <<"$queue">> = Group}) ->
filename:join([Group, Topic]);
get_topic(Topic, #{share := Group}) -> get_topic(Topic, #{share := Group}) ->
filename:join([<<"$share">>, Group, Topic]); filename:join([<<"$share">>, Group, Topic]);
get_topic(Topic, _) -> get_topic(Topic, _) ->