@@ -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