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

fix(CI): don't use any authz sources when testing

This commit is contained in:
Shawn 2022-06-09 18:45:31 +08:00
parent f159e081fa
commit f18eab402a
3 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.ci/docker-compose-file/scripts/run-emqx.sh
View File

@ -22,6 +22,8 @@ esac
echo "HOCON_ENV_OVERRIDE_PREFIX=EMQX_" echo "HOCON_ENV_OVERRIDE_PREFIX=EMQX_"
echo "EMQX_ZONES__DEFAULT__MQTT__RETRY_INTERVAL=2s" echo "EMQX_ZONES__DEFAULT__MQTT__RETRY_INTERVAL=2s"
echo "EMQX_ZONES__DEFAULT__MQTT__MAX_TOPIC_ALIAS=10" echo "EMQX_ZONES__DEFAULT__MQTT__MAX_TOPIC_ALIAS=10"
echo "EMQX_AUTHORIZATION__SOURCES=[]"
echo "EMQX_AUTHORIZATION__NO_MATCH=allow"
} >> .ci/docker-compose-file/conf.cluster.env } >> .ci/docker-compose-file/conf.cluster.env
is_node_up() { is_node_up() {

2
.github/workflows/run_fvt_tests.yaml vendored
View File

@ -186,6 +186,8 @@ jobs:
--set image.pullPolicy=Never \ --set image.pullPolicy=Never \
--set emqxConfig.EMQX_ZONES__DEFAULT__MQTT__RETRY_INTERVAL=2s \ --set emqxConfig.EMQX_ZONES__DEFAULT__MQTT__RETRY_INTERVAL=2s \
--set emqxConfig.EMQX_ZONES__DEFAULT__MQTT__MAX_TOPIC_ALIAS=10 \ --set emqxConfig.EMQX_ZONES__DEFAULT__MQTT__MAX_TOPIC_ALIAS=10 \
--set emqxConfig.EMQX_AUTHORIZATION__SOURCES=[] \
--set emqxConfig.EMQX_AUTHORIZATION__NO_MATCH=allow \
deploy/charts/emqx \ deploy/charts/emqx \
--debug --debug

2
apps/emqx_authz/src/emqx_authz.erl
View File

@ -329,7 +329,7 @@ authorize(
'client.check_authz_complete', 'client.check_authz_complete',
[Client, PubSub, Topic, deny, AuthzSource] [Client, PubSub, Topic, deny, AuthzSource]
), ),
?SLOG(info, #{ ?SLOG(warning, #{
msg => "authorization_permission_denied", msg => "authorization_permission_denied",
username => Username, username => Username,
ipaddr => IpAddress, ipaddr => IpAddress,