Revert "Add include directive for mbedtls error handling in build flags"

[cpatulea]

This reverts commit 391928e.

Fixes compile error under pioarduino:

: fatal error: mbedtls/error.h: No such file or directory

🤝 Attestations

• I have tested that my proposed changes behave as described.
• I have tested that my proposed changes do not cause any obvious regressions on the following devices:
  • Heltec (Lora32) V3
  • LilyGo T-Deck
  • LilyGo T-Beam
  • RAK WisBlock 4631
  • Seeed Studio T-1000E tracker card
  • Other (please specify below)

[cpatulea]

From build checks I now see the compile error which the original commit was intended to fix:

/pio/workspace/libdeps/heltec-ht62-esp32c3-sx1262/esp32_https_server/src/HTTPSConnection.cpp: In member function 'bool httpsserver::HTTPSConnection::setup(mbedtls_ssl_config*)':
/pio/workspace/libdeps/heltec-ht62-esp32c3-sx1262/esp32_https_server/src/HTTPSConnection.cpp:29:5: error: 'mbedtls_strerror' was not declared in this scope
     mbedtls_strerror(res, buf, sizeof(buf));
     ^~~~~~~~~~~~~~~~

Will work on a fix.

[mverch67]

Will work on a fix.

I think the include "mbedtls/error.h" belongs to here:
https://github.com/meshtastic/esp32_https_server/blob/b78f12c86ea65c3ca08968840ff554ff7ed69b60/src/HTTPSConnection.cpp#L3

[cpatulea]

Yes you're right. That's the fix I'm working on.

[cpatulea]

meshtastic/esp32_https_server#11

[cpatulea]

esp32_https_server version has been bumped, but only in master. Waiting for that to be merged into develop, then I'll be able to re-open this PR.

[thebentern]

esp32_https_server version has been bumped, but only in master. Waiting for that to be merged into develop, then I'll be able to re-open this PR.

@cpatulea I have backmerged master into develop this morning

[cpatulea]

@thebentern perfect, thank you. This PR is now ready for review.

[Dadud]

Pull Request

A fix has been submitted: PR #10127

Please review and merge if the changes look good. Thanks!

[Copilot]

Pull request overview

Reverts a previously added global compiler -include mbedtls/error.h build flag for ESP32 targets to fix PlatformIO Arduino (pioarduino) builds that fail because that header is not available in that framework configuration.

Changes:

• Removes the -include mbedtls/error.h directive from ESP32 common build_flags.