Skip to content

Commit

Permalink
fix: Extended log download timeframe. Improved messages. [backport re…
Browse files Browse the repository at this point in the history
…lease-5.6.0] (#5600)

fix: Extended log download timeframe. Improved messages. (#5599)

Extended the time frame where the spinning cog will stay displayed to 5 minutes.
Improved messaging to the user.

Signed-off-by: MMaiero <[email protected]>
(cherry picked from commit 246a8a3)

Co-authored-by: Matteo Maiero <[email protected]>
  • Loading branch information
eclipse-kura-bot and MMaiero authored Dec 3, 2024
1 parent 8633fc1 commit 3353d22
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -95,10 +95,12 @@ interface LogTabUiUiBinder extends UiBinder<Widget, LogTabUi> {
private final String nonce = Integer.toString(Random.nextInt());

private static final int DOWNLOAD_COMPLETE_WAIT_TIMEOUT = 5000;
private static final int DOWNLOAD_COMPLETION_RETRIES= 60;

private Timer waitDownloadCompleted = new Timer() {

// safety parameter, 9 = 45secs
private short retryLimit = 9;
private short retryLimit = DOWNLOAD_COMPLETION_RETRIES;
private String cookieName;

@Override
Expand All @@ -121,7 +123,7 @@ public void run() {
private void hideModalAndStop() {
EntryClassUi.hideWaitModal();
Cookies.removeCookie(cookieName, "/");
this.retryLimit = 9;
this.retryLimit = DOWNLOAD_COMPLETION_RETRIES;
this.cancel();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -888,7 +888,7 @@ pwdMaxLength=New password can be at most 255 characters long
pwdWhitespaceCharacters=New password cannot contain whitespace characters
pwdEmpty=New password cannot be empty

logDownload=Click this button to download all system logs in a single zip file
logDownload=Click this button to download all system logs in a single zip file. The process may take up to 5 minutes.

configurationChangeEventNotification=Another user has modified configuration for {0}. You might see an inconsistent state and a page refresh might be needed to show the latest changes.
configurationChangeEventNotificationGeneric=Another user has modified some configuration. You might see an inconsistent state and a page refresh might be needed to show the latest changes.
Expand Down

0 comments on commit 3353d22

Please sign in to comment.