Skip to content

Commit 7fdac5e

Browse files
committed
Add routine to get the OTA URL.
1 parent 55728db commit 7fdac5e

File tree

1 file changed

+11
-13
lines changed

1 file changed

+11
-13
lines changed

Firmware/RTK_Surveyor/menuFirmware.ino

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -479,6 +479,12 @@ void getFirmwareVersion(char * buffer, int bufferLength, bool includeDate)
479479
formatFirmwareVersion(FIRMWARE_VERSION_MAJOR, FIRMWARE_VERSION_MINOR, buffer, bufferLength, includeDate);
480480
}
481481

482+
const char * otaGetUrl()
483+
{
484+
// Select the URL for the over-the-air (OTA) updates
485+
return enableRCFirmware ? OTA_RC_FIRMWARE_JSON_URL : OTA_FIRMWARE_JSON_URL;
486+
}
487+
482488
// Returns true if we successfully got the versionAvailable
483489
// Modifies versionAvailable with OTA getVersion response
484490
// Connects to WiFi as needed
@@ -494,18 +500,12 @@ bool otaCheckVersion(char *versionAvailable, uint8_t versionAvailableLength)
494500
getFirmwareVersion(versionString, sizeof(versionString), enableRCFirmware);
495501
systemPrintf("Current firmware version: v%s\r\n", versionString);
496502

497-
if (enableRCFirmware == false)
498-
systemPrintf("Checking to see if an update is available from %s\r\n", OTA_FIRMWARE_JSON_URL);
499-
else
500-
systemPrintf("Checking to see if an update is available from %s\r\n", OTA_RC_FIRMWARE_JSON_URL);
503+
const char * url = otaGetUrl();
504+
systemPrintf("Checking to see if an update is available from %s\r\n", url);
501505

502506
ESP32OTAPull ota;
503507

504-
int response;
505-
if (enableRCFirmware == false)
506-
response = ota.CheckForOTAUpdate(OTA_FIRMWARE_JSON_URL, versionString, ESP32OTAPull::DONT_DO_UPDATE);
507-
else
508-
response = ota.CheckForOTAUpdate(OTA_RC_FIRMWARE_JSON_URL, versionString, ESP32OTAPull::DONT_DO_UPDATE);
508+
int response = ota.CheckForOTAUpdate(url, versionString, ESP32OTAPull::DONT_DO_UPDATE);
509509

510510
// We don't care if the library thinks the available firmware is newer, we just need a successful JSON parse
511511
if (response == ESP32OTAPull::UPDATE_AVAILABLE || response == ESP32OTAPull::NO_UPDATE_AVAILABLE)
@@ -562,17 +562,15 @@ void otaUpdate()
562562
ESP32OTAPull ota;
563563

564564
int response;
565-
const char * url = enableRCFirmware ? OTA_RC_FIRMWARE_JSON_URL : OTA_FIRMWARE_JSON_URL;
565+
const char * url = otaGetUrl();
566566
response = ota.CheckForOTAUpdate(url, &versionString[1], ESP32OTAPull::DONT_DO_UPDATE);
567567

568568
if (response == ESP32OTAPull::UPDATE_AVAILABLE)
569569
{
570570
systemPrintln("Installing new firmware");
571571
ota.SetCallback(otaPullCallback);
572572
if (enableRCFirmware == false)
573-
ota.CheckForOTAUpdate(OTA_FIRMWARE_JSON_URL, versionString); // Install new firmware, no reset
574-
else
575-
ota.CheckForOTAUpdate(OTA_RC_FIRMWARE_JSON_URL, versionString); // Install new firmware, no reset
573+
ota.CheckForOTAUpdate(url, versionString); // Install new firmware, no reset
576574

577575
if (apConfigFirmwareUpdateInProcess)
578576
{

0 commit comments

Comments
 (0)