diff --git a/.github/workflows/modem__target-test.yml b/.github/workflows/modem__target-test.yml index 5681a5e05..4de01e6cb 100644 --- a/.github/workflows/modem__target-test.yml +++ b/.github/workflows/modem__target-test.yml @@ -15,7 +15,7 @@ jobs: matrix: idf_ver: ["latest"] idf_target: ["esp32c3"] - test: [ { app: pppd, path: test/target }, { app: sim800_c3, path: examples/pppos_client }, { app: sim800_cmux, path: examples/simple_cmux_client } ] + test: [ { app: pppd, path: test/target }, { app: pppd_chap_auth, path: test/target }, { app: sim800_c3, path: examples/pppos_client }, { app: sim800_cmux, path: examples/simple_cmux_client } ] include: - idf_ver: "latest" idf_target: "esp32s2" @@ -58,7 +58,7 @@ jobs: matrix: idf_ver: ["latest"] idf_target: ["esp32c3"] - test: [ { app: pppd, path: test/target }, { app: sim800_c3, path: examples/pppos_client }, { app: sim800_cmux, path: examples/simple_cmux_client } ] + test: [ { app: pppd, path: test/target }, { app: pppd_chap_auth, path: test/target }, { app: sim800_c3, path: examples/pppos_client }, { app: sim800_cmux, path: examples/simple_cmux_client } ] include: - idf_ver: "latest" idf_target: "esp32s2" diff --git a/components/esp_modem/test/target/main/Kconfig.projbuild b/components/esp_modem/test/target/main/Kconfig.projbuild index 8b32e0252..871f75485 100644 --- a/components/esp_modem/test/target/main/Kconfig.projbuild +++ b/components/esp_modem/test/target/main/Kconfig.projbuild @@ -27,12 +27,23 @@ menu "Test App Configuration" help Pin number of UART RX. - config TEST_APP_TCP_PORT - int "Port of test" - range 0 65535 - default 2222 + config TEST_APP_AUTH + bool "Use PPP authentication" + select LWIP_PPP_CHAP_SUPPORT + default n help - The remote port to which the client will connects to - once the PPP connection established + Set to true for the PPP client to use authentication + + config TEST_APP_AUTH_USERNAME + string "Set username for authentication" + default "myclient" + help + Username to authenticate the PPP connection. + + config TEST_APP_AUTH_PASSWORD + string "Set password for authentication" + default "mypassword" + help + Password to authenticate the PPP connection. endmenu diff --git a/components/esp_modem/test/target/main/pppd_test.cpp b/components/esp_modem/test/target/main/pppd_test.cpp index 06195215d..4d4860977 100644 --- a/components/esp_modem/test/target/main/pppd_test.cpp +++ b/components/esp_modem/test/target/main/pppd_test.cpp @@ -94,6 +94,10 @@ extern "C" void app_main(void) ESP_ERROR_CHECK(esp_event_handler_register(IP_EVENT, ESP_EVENT_ANY_ID, on_modem_event, nullptr)); ESP_ERROR_CHECK(esp_event_handler_register(NETIF_PPP_STATUS, ESP_EVENT_ANY_ID, &on_ppp_changed, nullptr)); +#if CONFIG_TEST_APP_AUTH + esp_netif_ppp_set_auth(ppp_netif, NETIF_PPP_AUTHTYPE_CHAP, CONFIG_TEST_APP_AUTH_USERNAME, CONFIG_TEST_APP_AUTH_PASSWORD); +#endif + modem_start_network(); Catch::Session session; int numFailed = session.run(); diff --git a/components/esp_modem/test/target/pytest_pppd.py b/components/esp_modem/test/target/pytest_pppd.py index 707936a54..d675e3c65 100644 --- a/components/esp_modem/test/target/pytest_pppd.py +++ b/components/esp_modem/test/target/pytest_pppd.py @@ -1,4 +1,4 @@ -# SPDX-FileCopyrightText: 2022 Espressif Systems (Shanghai) CO LTD +# SPDX-FileCopyrightText: 2022-2024 Espressif Systems (Shanghai) CO LTD # SPDX-License-Identifier: Unlicense OR CC0-1.0 from __future__ import print_function, unicode_literals @@ -9,14 +9,20 @@ from threading import Event, Thread import netifaces -def run_server(server_stop, port, server_ip, client_ip): +def run_server(server_stop, port, server_ip, client_ip, auth, auth_user, auth_password): print('Starting PPP server on port: {}'.format(port)) try: arg_list = [ 'sudo', 'pppd', port, '115200', - '{}:{}'.format(server_ip, client_ip), 'modem', 'local', 'noauth', + '{}:{}'.format(server_ip, client_ip), 'modem', 'local', 'debug', 'nocrtscts', 'nodetach', '+ipv6' ] + if auth: + arg_list.extend(['auth', '+chap']) + subprocess.run(['sudo', 'rm', '/etc/ppp/chap-secrets']) + subprocess.run(f"echo '{auth_user} * {auth_password} *' | sudo tee -a /etc/ppp/chap-secrets", shell=True) + else: + arg_list.append('noauth') p = subprocess.Popen(arg_list, stdout=subprocess.PIPE, bufsize=1) while not server_stop.is_set(): if p.poll() is not None: @@ -51,6 +57,9 @@ def test_examples_protocol_pppos_connect(dut): try: server_ip = dut.app.sdkconfig.get('TEST_APP_PPP_SERVER_IP') client_ip = dut.app.sdkconfig.get('TEST_APP_PPP_CLIENT_IP') + auth = dut.app.sdkconfig.get('TEST_APP_AUTH') + auth_user = dut.app.sdkconfig.get('TEST_APP_AUTH_USERNAME') + auth_password = dut.app.sdkconfig.get('TEST_APP_AUTH_PASSWORD') except Exception: print( 'ENV_TEST_FAILURE: Some mandatory configuration not found in sdkconfig' @@ -63,7 +72,7 @@ def test_examples_protocol_pppos_connect(dut): # Start the PPP server server_stop = Event() t = Thread(target=run_server, - args=(server_stop, port, server_ip, client_ip)) + args=(server_stop, port, server_ip, client_ip, auth, auth_user, auth_password)) t.start() try: ppp_server_timeout = time.time() + 30 diff --git a/components/esp_modem/test/target/sdkconfig.ci.pppd_chap_auth b/components/esp_modem/test/target/sdkconfig.ci.pppd_chap_auth new file mode 100644 index 000000000..23b587e9c --- /dev/null +++ b/components/esp_modem/test/target/sdkconfig.ci.pppd_chap_auth @@ -0,0 +1,4 @@ +CONFIG_COMPILER_CXX_EXCEPTIONS=y +CONFIG_ESP_MAIN_TASK_STACK_SIZE=4096 +CONFIG_LWIP_PPP_SUPPORT=y +CONFIG_TEST_APP_AUTH=y diff --git a/components/esp_modem/test/target/sdkconfig.defaults b/components/esp_modem/test/target/sdkconfig.defaults index 4af92e67d..28ae669b9 100644 --- a/components/esp_modem/test/target/sdkconfig.defaults +++ b/components/esp_modem/test/target/sdkconfig.defaults @@ -1,4 +1,3 @@ CONFIG_COMPILER_CXX_EXCEPTIONS=y -CONFIG_CXX_EXCEPTIONS=y -CONFIG_PPP_SUPPORT=y CONFIG_ESP_MAIN_TASK_STACK_SIZE=4096 +CONFIG_LWIP_PPP_SUPPORT=y