Skip to content
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

Add "server_cert" parameter to firmware_update command. #1043

Merged
merged 1 commit into from
Jul 29, 2021

Conversation

h2zero
Copy link
Collaborator

@h2zero h2zero commented Jul 29, 2021

When updating from the MQTT command from a SSL enabled server this adds an option to provide
the server SSL certificate with the command.

This is useful if connecting to different servers or if the configured certificate has expired
and requires updating to not lose the ability to update via OTA.

The certificate provided will be saved to flash memory for use on subsequent restarts.

Description:

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

@h2zero h2zero force-pushed the update-ota-cert-mqtt branch from bf64197 to 86ec283 Compare July 29, 2021 02:18
When updating from the MQTT command from a SSL enabled server this adds an option to provide
the server SSL certificate with the command.

This is useful if connecting to different servers or if the configured certificate has expired
and requires updating to not lose the ability to update via OTA.

The certificate provided will be saved to flash memory for use on subsequent restarts.
@h2zero h2zero force-pushed the update-ota-cert-mqtt branch from 86ec283 to 354af64 Compare July 29, 2021 03:16
@1technophile 1technophile added this to the v0.9.8 milestone Jul 29, 2021
@1technophile
Copy link
Owner

Thanks

@1technophile 1technophile merged commit 2dedabc into 1technophile:development Jul 29, 2021
@h2zero h2zero deleted the update-ota-cert-mqtt branch July 30, 2021 01:03
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