Skip to content

added a way to disable SSL #1416

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

Closed
wants to merge 1 commit into from
Closed

added a way to disable SSL #1416

wants to merge 1 commit into from

Conversation

ettisan
Copy link

@ettisan ettisan commented Jan 12, 2016

Hi,

I was trying to slim down the binary size of one of my projects and saw that a lot of stuff is included in the binary even if it is never used (empty sketch needs >50% of my ESP module). This is particularly annoying since I want to use HTTP-OTA.

This PR allows to disable SSL using a preprocessor definition (currently not configureable in the UI).

For me this saves ~14% sketch size (since axtls is ommited).

I have not yet figured how to inspect the contents of the binary image but i guess there are a lot of things I do not require in all of my sketches (e.g. WPS, ...). I'm happy about any pointers.

Wolfgang

@igrr
Copy link
Member

igrr commented Jan 13, 2016

I would rather rely on the linker to remove unused stuff (through -gc-sections option). Need to investigate why something gets pulled from axTLS even when you don't use wificlientsecure.

Edit: OK, clear now. Need to think about refactoring httpclient a bit.

@ettisan
Copy link
Author

ettisan commented Jan 13, 2016

Maybe a little background:
I have a device running where the sketch takes about ~52% of the space. I want to build an update stub that only connects to the wifi and calls ESPhttpUpdate.update().
This way i can update the device to this stub (afterwards very little space is used). And this stub then updates to the actual new version (which may take a lot of space).

@igrr
Copy link
Member

igrr commented Apr 4, 2016

#1857 changes HTTPClient so that WiFiClientSecure is not pulled into the final program, if it is not required.

@igrr igrr closed this Apr 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants