Git Product home page Git Product logo

Comments (25)

cadeyrn avatar cadeyrn commented on June 18, 2024 4

I am busy with other stuff. I'll release a update for my other add-on New Tab Override soon then I'll return to the Bookmarks Organizer development. Thank you for your patience.

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024 2

I will work on an update after the release of Firefox 57. I don't have an timeline for a release date because I have to test a lot and investigate all the issues. But the plan is to stark working on this really soon.

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024 1

Hi,

thanks for the report. It's difficult to reproduce because I don't have this problem and because I don't have access to your Firefox profile. There is a debug option in the add-on's settings but the debug output will only be shown after all bookmarks are checked.

I need a solution to cancel a bookmark check after a few seconds and to log such events. Unfortunately it's not yet implemented. At the moment there is nothing what I can do before this enhancement is implemented. I'll try to find a solution soon but I don't have a timeline for this.

Once this is implemented it should no longer takes hours without result, in the worst case some bookmarks will be skipped. But it's better than the current state. And I hope that a log can provide some infos why a bookmark could not be checked.

from bookmarks-organizer.

PadaV4 avatar PadaV4 commented on June 18, 2024 1

For me the extension stopped at
Number of bookmarks: 3979
Checked:3978

In the end i narrowed it down to this bookmark i had.
evetaku.com/blog/2010/11/a-picture-drawn-by-an-eraser/
When i removed it, the extension finished checking the others pretty fast.

When i tried with only this bookmark existing(deleted all others), the extension actually did finish, but only after checking for a really long time. Here is the log i got.

[
{
"bookmark": {
"id": "RhVmhMGlDAbh",
"parentId": "QXLbmU3J1qjm",
"title": "A Picture Drawn by an Eraser | EveTaku",
"url": "http://evetaku.com/blog/2010/11/a-picture-drawn-by-an-eraser/",
"status": 902
},
"method": "HEAD",
"cause": "fetch-error",
"response": "NetworkError when attempting to fetch resource."
},
{
"bookmark": {
"id": "RhVmhMGlDAbh",
"parentId": "QXLbmU3J1qjm",
"title": "A Picture Drawn by an Eraser | EveTaku",
"url": "http://evetaku.com/blog/2010/11/a-picture-drawn-by-an-eraser/",
"status": 902
},
"method": "GET",
"cause": "fetch-error",
"response": "NetworkError when attempting to fetch resource."
}
]

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024 1

A short status update: I can't report any further progress because I was waiting for the Abort API. The Abort API landed in Firefox Nightly and will be enabled by default in Firefox 57. I hope this new API will help with the problem. This API is the only realistic way to solve this problem because it's not possible to abort a fetch() request without this API. This means that Firefox 57 will be required for a improved version.

from bookmarks-organizer.

mentalo- avatar mentalo- commented on June 18, 2024 1

It works perfecty with 59.0 ! Congrats !

from bookmarks-organizer.

Gitoffthelawn avatar Gitoffthelawn commented on June 18, 2024

+1 Same issue for me.

Thank you for explaining how the debug option works. I was wondering why nothing ever appears in the console (except for reflows).

from bookmarks-organizer.

rayman89 avatar rayman89 commented on June 18, 2024

@cadeyrn I have the same problem when checking for duplicates. I would suggest that you test on a profile with 2k+ different domain bookmarks. I think either the issue is in the amount of bookmarks or the time it spends waiting for an url to resolve.

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024

My profile has more than 5000 bookmarks. No urls have to be resolved to check for duplicates. URLs have only to be resolved for the check for broken bookmarks.

from bookmarks-organizer.

Gitoffthelawn avatar Gitoffthelawn commented on June 18, 2024

Here's what I get:

  • When checking for duplicates, it's fairly quick, and shows results. However, the overlay image with the three pulsing dots never disappears. Also, the duplicates are listed as errors, when they should probably be listed as warnings.

  • When checking for broken bookmarks, the extension never finishes. Interestingly, it always stalls with 3 bookmarks remaining, unless I reinstall the extension, in which case it stalls with 1 bookmark remaining. The extension never shows any results.

from bookmarks-organizer.

rayman89 avatar rayman89 commented on June 18, 2024

Can confirm that the duplicates "finishes" but stays with the logo and dots which doesn't allow you to do anything to the duplicates in my case.

In my case the search for broken bookmarks works properly. I have 2k+ bookmarks as well.

from bookmarks-organizer.

Gitoffthelawn avatar Gitoffthelawn commented on June 18, 2024

I wonder what causes the extension to stall for some installations and not others. BTW, I ran Places Maintenance before running Bookmarks Organizer, which should clean up any errors, and at the least, perform a SQL VACUUM. Unfortunately, that did not help.

from bookmarks-organizer.

StanfordAlan avatar StanfordAlan commented on June 18, 2024

I have this too on my Mac 10.12.6 running Firefox 54.0.1 (64-bit). I have 7570 bookmarks.

My experience is similar to that described by Gitoffthelawn.

Broken hangs at 89 checked though. Duplicates show 7570 checked, but it continues to check.

I've tried restarting and reinstalling the addon.

from bookmarks-organizer.

Gitoffthelawn avatar Gitoffthelawn commented on June 18, 2024

@cadeyrn What is the status of the fix for this?

from bookmarks-organizer.

Gitoffthelawn avatar Gitoffthelawn commented on June 18, 2024

@cadeyrn Thanks. Looking forward to the update.

from bookmarks-organizer.

Gitoffthelawn avatar Gitoffthelawn commented on June 18, 2024

Firefox 57 will officially be released in 2 days. Can this be fixed now?

Also, fixing the tag issue is important!

from bookmarks-organizer.

Jan02 avatar Jan02 commented on June 18, 2024

happens to me too: https://streamable.com/ewiss

from bookmarks-organizer.

AlphaHasi avatar AlphaHasi commented on June 18, 2024

Same issue her. I tried to find out, what bookmark causes the problem and activated "debug", but where can I find the debug info? Tried JS console and searched in profile folder but there is no file to find.

The browser secreen is empty beside the animated working-icon and the header with the stuuck info:
Anzahl Lesezeichen: 3291
Überprüft: 3289
Warnungen: 1281
Fehler: 230

Can you mabe just activate printing debug info into the console, so you print called URLs and returned code ore timeout info, so even when it breaks you would see, where it breaks?

from bookmarks-organizer.

TristisOris avatar TristisOris commented on June 18, 2024

could it happens on sites with redirect (cloudflare), captcha bot check, router authorization popup?
plugin works for me only 1 time in old FF release.

from bookmarks-organizer.

XhmikosR avatar XhmikosR commented on June 18, 2024

This is a duplicate of #23

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024

Closed as duplicate of #23.

from bookmarks-organizer.

mentalo- avatar mentalo- commented on June 18, 2024

So it won't be fixed ?

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024

It will be fixed once possible, please see the latest comment in #23. I already have a fix but I can't release the fix until Mozilla has fixed the Firefox crash.

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024

@mentalo- Mozilla fixed the crash in Firefox 60 and will probably also fix it for Firefox 59 (waiting for confirmation). I implemented my part of the fix in Bookmarks Organizer 2.0 which will require Firefox 59 or higher. In the case it won't be fixed in Firefox 59 I will provide different code pathes for Firefox 59 and 60+ and this issue will be fixed for users of Firefox 60 and higher.

The exact release date of Bookmarks Organizer 2.0 is still TBD but it should be this week.

from bookmarks-organizer.

cadeyrn avatar cadeyrn commented on June 18, 2024

Bookmarks Organizer 2.0 has been released:
https://addons.mozilla.org/en-US/firefox/addon/bookmarks-organizer/

from bookmarks-organizer.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.