feat(WebServer): Add support for JavaScript modules#11782
feat(WebServer): Add support for JavaScript modules#11782me-no-dev merged 1 commit intoespressif:masterfrom
Conversation
👋 Hello cosstab, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Test Results 76 files 76 suites 13m 12s ⏱️ Results for commit 8e609cf. ♻️ This comment has been updated with latest results. |
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description of Change
Added media type for .mjs files on the WebServer library.
Previously, they were being served with the "application/octet-stream" MIME type, which web browsers wouldn't accept.
I chose "text/javascript" for the media type instead of "application/javascript" (which is used by the library for .js files) because the latter was deprecated on RFC 9239.
Test Scenarios
Tested on Arduino-esp32 core v3.2.1 using an ESP32 board.