-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix compile error with core 3.0.0 for Range Extender driver xdrv_58_range_extender.ino #22205
Conversation
ported xdrv_58_range_extender.ino to work with framework-arduinoespressif32 @ 3.0.5. Tested to work correctly on ESP32 with and without NAPT, but prevoius full features could not be reached. |
@pharaocurt Hi, thx for the PR. You can remove the "if" queries. Tasmota Arduino framework has this sdkconfig lwip settings.
|
…k-arduinoespressif32 @ 3.1.0+sha.22a3b096 is available with CONFIG_LWIP_IPV4_NAPT_PORTMAP=y
Verified working of feature with framework-arduinoespressif32 @ 3.1.0+sha.22a3b096, portmap_add is available. |
Verified also configuration of NAPT port_add with MACaddress. Sorry for previous wrong info. |
I did a build for an ESP32-C3 with this fix. No compile errors, and I was able to have a tablet connecting and using Internet via the range extender. It is a simple setup with NAPT, not using routing. However, I also tried to connect a Tasmota device, this used to work, but is now failing with repeated |
…ange_extender.ino (arendst#22205) * final ported range_extender * removed #define USE_WIFI_RANGE_EXTENDER_PORTADD, because new framework-arduinoespressif32 @ 3.1.0+sha.22a3b096 is available with CONFIG_LWIP_IPV4_NAPT_PORTMAP=y
…ange_extender.ino (arendst#22205) * final ported range_extender * removed #define USE_WIFI_RANGE_EXTENDER_PORTADD, because new framework-arduinoespressif32 @ 3.1.0+sha.22a3b096 is available with CONFIG_LWIP_IPV4_NAPT_PORTMAP=y
Description:
Related issue (if applicable): fixes #
Checklist:
NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass