Fixes the audio.com upload flow

libraries/lib-cloud-audiocom/ServiceConfig.cpp

@@ -46,11 +46,17 @@ std::string ServiceConfig::GetAPIUrl(std::string_view apiURI) const
 std::string ServiceConfig::GetFinishUploadPage(
    std::string_view audioID, std::string_view token) const
 {
-   return "http://audio.com/audacity/upload?audioId=" + std::string(audioID) +
+   return "https://audio.com/audacity/upload?audioId=" + std::string(audioID) +
           "&token=" + std::string(token) +
           "&clientId=" + std::string(GetOAuthClientID());
 }
 
+std::string ServiceConfig::GetAudioURL(
+   std::string_view userSlug, std::string_view audioSlug) const
+{
+   return "https://audio.com/" + std::string(userSlug) + "/audio/" + std::string(audioSlug) + "/edit";
+}
+
 std::chrono::milliseconds ServiceConfig::GetProgressCallbackTimeout() const
 {
    return std::chrono::seconds(3);

libraries/lib-cloud-audiocom/ServiceConfig.h

@@ -35,6 +35,8 @@ class CLOUD_AUDIOCOM_API ServiceConfig final
    std::string GetAPIUrl(std::string_view apiURI) const;
    //! Helper to construct the page URL for the anonymous upload last stage
    std::string GetFinishUploadPage(std::string_view audioID, std::string_view token) const;
+   //! Helper to construct the page URL for the authorised upload
+   std::string GetAudioURL(std::string_view userSlug, std::string_view audioSlug) const;
    //! Timeout between progress callbacks
    std::chrono::milliseconds GetProgressCallbackTimeout() const;
    //! Preferred audio format

libraries/lib-cloud-audiocom/UploadService.cpp

@@ -206,6 +206,7 @@ struct AudiocomUploadOperation final :
 
    std::string mAudioID;
    std::string mUploadToken;
+   std::string mUserName;
 
    std::string mAudioSlug;
 
@@ -264,10 +265,14 @@ struct AudiocomUploadOperation final :
 
       if (mCompletedCallback)
       {
+         const auto uploadURL =
+            mAuthToken.empty() ?
+               mServiceConfig.GetFinishUploadPage(mAudioID, mUploadToken) :
+               mServiceConfig.GetAudioURL(mUserName, mAudioSlug);
 
          mCompletedCallback(
             { UploadOperationCompleted::Result::Success,
-              UploadSuccessfulPayload { mAudioID, mAudioSlug } });
+              UploadSuccessfulPayload { mAudioID, mAudioSlug, mUploadToken, uploadURL } });
       }
 
       mProgressCallback = {};
@@ -395,6 +400,8 @@ struct AudiocomUploadOperation final :
 
          if (extra.HasMember("token"))
             mUploadToken = extra["token"].GetString();
+
+         mUserName = extra["audio"]["username"].GetString();
       }
 
       const auto encType = document.HasMember("enctype") ?
@@ -579,7 +586,7 @@ UploadOperationHandle UploadService::Upload(
       mServiceConfig, fileName, projectName, isPublic,
       std::move(completedCallback), std::move(progressCallback));
 
-   mOAuthService.ValidateAuth([operation](std::string_view authToken)
+   mOAuthService.ValidateAuth([operation, this](std::string_view authToken)
                               { operation->InitiateUpload(authToken); });
 
    return UploadOperationHandle { operation };

libraries/lib-cloud-audiocom/UploadService.h

@@ -43,6 +43,10 @@ struct CLOUD_AUDIOCOM_API UploadSuccessfulPayload final
    std::string audioId;
    //! "Slug" to be used for shareable URL construction
    std::string audioSlug;
+   //! Upload token, if any
+   std::string uploadToken;
+   //! URL to the uploaded audio
+   std::string audioUrl;
 };
 
 //! Message that is sent when upload is finished.

src/cloud/audiocom/ShareAudioDialog.cpp

@@ -436,28 +436,8 @@ void ShareAudioDialog::StartUploadProcess()
 void ShareAudioDialog::HandleUploadSucceeded(
    const UploadSuccessfulPayload& payload)
 {
-   mProgressPanel.timePanel->Hide();
-   mProgressPanel.title->SetLabel(XO("Upload complete!").Translation());
-   mProgressPanel.info->Show();
-
-   mProgressPanel.info->SetLabel(
-      "By pressing continue, you will be taken to audio.com and given a shareable link.");
-   mProgressPanel.info->Wrap(mProgressPanel.root->GetSize().GetWidth());
-
-   mContinueAction = [this, slug = std::string(payload.audioSlug)]()
-   {
-      EndModal(wxID_CLOSE);
-      auto url = wxString::Format(
-         "https://audio.com/%s/%s/edit", GetUserService().GetUserSlug(),
-         audacity::ToWXString(slug));
-
-      OpenInDefaultBrowser(url);
-   };
-
-   mContinueButton->Show();
-
-   Layout();
-   Fit();
+   EndModal(wxID_CLOSE);
+   OpenInDefaultBrowser(wxString { payload.audioUrl });
 }
 
 void ShareAudioDialog::HandleUploadFailed(const UploadFailedPayload& payload)
@@ -466,12 +446,7 @@ void ShareAudioDialog::HandleUploadFailed(const UploadFailedPayload& payload)
 
    TranslatableString message;
 
-   if (payload.status == 401)
-   {
-      message = XO(
-         "We are unable to upload this file. Please try again and make sure to link to your audio.com account before uploading.");
-   }
-   else
+   if (!payload.message.empty())
    {
       auto details = payload.message;
 
@@ -480,6 +455,11 @@ void ShareAudioDialog::HandleUploadFailed(const UploadFailedPayload& payload)
 
       message = XO("Error: %s").Format(details);
    }
+   else
+   {
+      message = XO(
+         "We are unable to upload this file. Please try again and make sure to link to your audio.com account before uploading.");
+   }
 
    BasicUI::ShowErrorDialog(
       {}, XO("Upload error"),