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

downloading favicon shows progress bar twice #2265

Closed
muchqs opened this issue Sep 3, 2018 · 10 comments · Fixed by #2309
Closed

downloading favicon shows progress bar twice #2265

muchqs opened this issue Sep 3, 2018 · 10 comments · Fixed by #2309

Comments

@muchqs
Copy link

muchqs commented Sep 3, 2018

When I click on the download favicon button, it either says failed to download or download instantly, but in either case a progress bar window then shows up and get stuck at 0% or 1% and doesn't go away on its own and I have to click cancel. Happening for every website.

@droidmonkey
Copy link
Member

Please don't delete the issue template. Paste your debug info from the about dialog.

@muchqs
Copy link
Author

muchqs commented Sep 3, 2018

sorry about that.

KeePassXC - Version 2.3.4
Revision: 6fe821c

Libraries:

  • Qt 5.11.1
  • libgcrypt 1.8.3

Operating system: Windows 10 (10.0)
CPU architecture: x86_64
Kernel: winnt 10.0.17134

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • Legacy Browser Integration (KeePassHTTP)
  • SSH Agent
  • YubiKey

Doesn't matter if favicon is successfully downloaded or not, a progress bar window pops up as if the download is starting but just shows 0% and never goes away. Also sometimes when I click "download favicon", the progress bar shows but shows 0% and hangs, but then I click "cancel" the favicon is downloaded(???). Then after I click ok to close the edit window, the download favicon progress bar pops up again and hangs at 0%.

@alvarogmj
Copy link

This happens for me also

KeePassXC - Version 2.3.4
Revision: 6fe821c

Libraries:

  • Qt 5.9.3
  • libgcrypt 1.8.2

Operating system: OS X El Capitan (10.11)
CPU architecture: x86_64
Kernel: darwin 15.6.0

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • Legacy Browser Integration (KeePassHTTP)
  • SSH Agent
  • YubiKey

@muchqs
Copy link
Author

muchqs commented Sep 6, 2018

Do you have that "google as fall-back" option checked? I think that is causing this issue. But if i uncheck the option I can't download any icon. Every website just returns the error "unable to fetch icon".

@alvarogmj
Copy link

No, I don't have it checked, and the download works for me. I only get "unable to fetch icon" when the icon is not at /favicon.ico, I guess that is expected :)

@whisdol
Copy link

whisdol commented Sep 8, 2018

When the "Google as fallback" option is checked, I even get three popups:
image
The popups appear for me in this order for namecheap.com, where the favicon download fails:

  1. Download progress from namecheap.com
  2. Download progress from namecheap.com
  3. Download progress from google.com
  4. All popups are closed, an error message is displayed
  5. Download progress from namecheap.com
  6. Download progress from namecheap.com
  7. Download progress from google.com

@ek0weav
Copy link

ek0weav commented Sep 10, 2018

Same issue on my platform.

KeePassXC - Version 2.3.4
Revision: 6fe821c

Libraries:

  • Qt 5.5.1
  • libgcrypt 1.8.1

Operating system: Linux Mint 18.3
CPU architecture: x86_64
Kernel: linux 4.10.0-38-generic

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • Legacy Browser Integration (KeePassHTTP)
  • SSH Agent
  • YubiKey

@tlnagy
Copy link

tlnagy commented Sep 11, 2018

As @Olfrix points out, this isn't isolated to windows. I can confirm this is occurring on Linux as well. Sometimes I get multiple popup windows, other times I get a Download Progress popup that is stuck at 1%. I also have the Google as a fallback option checked, otherwise the favicons fail to download more >50% of the time.

image

KeePassXC - Version 2.3.4
Revision: 2cc52a8
Distribution: Snap

Libraries:

  • Qt 5.9.5
  • libgcrypt 1.8.1

Operating system: Ubuntu Core 18
CPU architecture: x86_64
Kernel: linux 4.15.0-33-generic

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • Legacy Browser Integration (KeePassHTTP)
  • SSH Agent
  • YubiKey

@droidmonkey
Copy link
Member

I thought I had fixed this when I also fixed the associated crash this caused. Unfortunately, looks like there is still a race condition that occurs. The dialogs should have an update timeout that autocloses them.

@kneitinger
Copy link
Contributor

@droidmonkey it seems that setMinimumDuration() used in the fix for the crash only delays the time until the progress bar of the cancelled fetch appears. If I increase it to 4000, it appears after 4 seconds.

kneitinger added a commit to kneitinger/keepassxc that referenced this issue Sep 19, 2018
Name the UrlFetchProgressDialog() with the corresponding URL in order to
be identified by name by its parent when the failed request is handeled
in EditWidgetIcons::fetchFinished(). fetchFinished() retrieves the
relevant UrlFetchProgressDialog() and calls close() on it.

Fixes: keepassxreboot#2265
kneitinger added a commit to kneitinger/keepassxc that referenced this issue Sep 20, 2018
Name the UrlFetchProgressDialog() with the corresponding URL in order to
be identified by name by its parent when the failed request is handeled
in EditWidgetIcons::fetchFinished(). fetchFinished() retrieves the
relevant UrlFetchProgressDialog() and calls close() on it.

Fixes: keepassxreboot#2265
droidmonkey pushed a commit to kneitinger/keepassxc that referenced this issue Sep 20, 2018
Name the UrlFetchProgressDialog() with the corresponding URL in order to
be identified by name by its parent when the failed request is handeled
in EditWidgetIcons::fetchFinished(). fetchFinished() retrieves the
relevant UrlFetchProgressDialog() and calls close() on it.

Fixes: keepassxreboot#2265
droidmonkey pushed a commit that referenced this issue Sep 21, 2018
…er (#2309)

* Replace Google with DuckDuckGo for optional fallback favicon fetch URL
Modify the work initially done in #36, and most recently modified in #1786,
to use DuckDuckGo's https://icons.duckduckgo.com/ip3/www.example.com.ico
favicon endpoint.

Fixes #2258

* Close failed favicon fetch progress bars

Name the UrlFetchProgressDialog() with the corresponding URL in order to
be identified by name by its parent when the failed request is handeled
in EditWidgetIcons::fetchFinished(). fetchFinished() retrieves the
relevant UrlFetchProgressDialog() and calls close() on it.

Fixes: #2265
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants