From 29854264e3ba7e138375ec115fd93d971528ef03 Mon Sep 17 00:00:00 2001 From: Stuart Longland Date: Thu, 8 Jul 2021 07:57:48 +1000 Subject: [PATCH] =?UTF-8?q?broker=20tests:=20Change=20passwords=20for=20us?= =?UTF-8?q?er[1=E2=80=A63]?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit As pointed out, it's possible to accidentally mix them up if the passwords are the same. Obviously, the passwords used for these tests should _NOT_ be used in production. --- tests/plugins/passwd | 8 ++++---- tests/test_broker.py | 14 +++++++------- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/tests/plugins/passwd b/tests/plugins/passwd index 7749b33..c2dfe64 100644 --- a/tests/plugins/passwd +++ b/tests/plugins/passwd @@ -1,7 +1,7 @@ # Test password file user:$6$1sSXVMBVKMF$uDStq59YfiuFxVF1Gi/.i/Li7Vwf5iTwg8LovLKmCvM5FsRNJM.OPWHhXwI2.4AscLZXSPQVxpIlX6laUl9570 test_user:$6$.c9f9sAzs5YXX2de$GSdOi3iFwHJRCIJn1W63muDFQAL29yoFmU/TXcwDB42F2BZg3zcN5uKBM0.1PjwdMpWHRydbhXWSc3uWKSmKr. -# Password for these is "test" -user1:$6$QeikCQ5JjR$AS0ibQtNpfkLZY7fhpLqDC6KkVvYhwKQd1CM.WGjVQIoWIKxxTJGkyzNx4rWvp7FaoMG9kENULRHEI907I0fx1 -user2:$6$QeikCQ5JjR$AS0ibQtNpfkLZY7fhpLqDC6KkVvYhwKQd1CM.WGjVQIoWIKxxTJGkyzNx4rWvp7FaoMG9kENULRHEI907I0fx1 -user3:$6$QeikCQ5JjR$AS0ibQtNpfkLZY7fhpLqDC6KkVvYhwKQd1CM.WGjVQIoWIKxxTJGkyzNx4rWvp7FaoMG9kENULRHEI907I0fx1 +# Password for these is "${USER}password" +user1:$6$h.fV0zYsXI$8wKblqETpztRKcPD6OLWZc1mU4nW5yQ713R5ECs7EwJa7oas/yrhI2itUdhETI8BvmtfGy65ltAMap9gHkzdc1 +user2:$6$bUyF8v0mTo94$IJMa2BlCd6/mAM5N5s6heFB2ewC4j3CDkFb9mYIoarXg4OxiZVnLyh20IWHf6GY8i4sc5m2D9nWLrtbnIO5o8. +user3:$6$YOjKg1kYEeGibb$HlVa2EvQbF1ssSQs.lFnS1NhoTBQLG5YF7h0z4komAEvNJw6m4gay81MRp.lt4PSbcVrimcuRbidR9cRZkfBb/ diff --git a/tests/test_broker.py b/tests/test_broker.py index 2b5ebe6..7012e37 100644 --- a/tests/test_broker.py +++ b/tests/test_broker.py @@ -272,7 +272,7 @@ async def test_client_publish(broker, mock_plugin_manager): @pytest.mark.asyncio async def test_client_publish_acl_permitted(acl_broker): sub_client = MQTTClient() - ret = await sub_client.connect("mqtt://user2:test@127.0.0.1:1884/") + ret = await sub_client.connect("mqtt://user2:user2password@127.0.0.1:1884/") assert ret == 0 ret = await sub_client.subscribe( @@ -281,7 +281,7 @@ async def test_client_publish_acl_permitted(acl_broker): assert ret == [QOS_0] pub_client = MQTTClient() - ret = await pub_client.connect("mqtt://user1:test@127.0.0.1:1884/") + ret = await pub_client.connect("mqtt://user1:user1password@127.0.0.1:1884/") assert ret == 0 ret_message = await pub_client.publish("public/subtopic/test", b"data", QOS_0) @@ -299,7 +299,7 @@ async def test_client_publish_acl_permitted(acl_broker): @pytest.mark.asyncio async def test_client_publish_acl_forbidden(acl_broker): sub_client = MQTTClient() - ret = await sub_client.connect("mqtt://user2:test@127.0.0.1:1884/") + ret = await sub_client.connect("mqtt://user2:user2password@127.0.0.1:1884/") assert ret == 0 ret = await sub_client.subscribe( @@ -308,7 +308,7 @@ async def test_client_publish_acl_forbidden(acl_broker): assert ret == [QOS_0] pub_client = MQTTClient() - ret = await pub_client.connect("mqtt://user1:test@127.0.0.1:1884/") + ret = await pub_client.connect("mqtt://user1:user1password@127.0.0.1:1884/") assert ret == 0 ret_message = await pub_client.publish("public/forbidden/test", b"data", QOS_0) @@ -326,11 +326,11 @@ async def test_client_publish_acl_forbidden(acl_broker): @pytest.mark.asyncio async def test_client_publish_acl_permitted_sub_forbidden(acl_broker): sub_client1 = MQTTClient() - ret = await sub_client1.connect("mqtt://user2:test@127.0.0.1:1884/") + ret = await sub_client1.connect("mqtt://user2:user2password@127.0.0.1:1884/") assert ret == 0 sub_client2 = MQTTClient() - ret = await sub_client2.connect("mqtt://user3:test@127.0.0.1:1884/") + ret = await sub_client2.connect("mqtt://user3:user3password@127.0.0.1:1884/") assert ret == 0 ret = await sub_client1.subscribe( @@ -344,7 +344,7 @@ async def test_client_publish_acl_permitted_sub_forbidden(acl_broker): assert ret == [0x80] pub_client = MQTTClient() - ret = await pub_client.connect("mqtt://user1:test@127.0.0.1:1884/") + ret = await pub_client.connect("mqtt://user1:user1password@127.0.0.1:1884/") assert ret == 0 ret_message = await pub_client.publish("public/subtopic/test", b"data", QOS_0)