multithreaded-download-manager icon indicating copy to clipboard operation
multithreaded-download-manager copied to clipboard

Firefox developer 103, MDM V3.2b4 - Still showing "cannot access internal storage"

Open XxUnkn0wnxX opened this issue 2 years ago • 12 comments

Screen Shot 2022-07-05 at 10 51 49 am is there a way to hide that message? as the instructions for that are a bit dated here: https://github.com/jingyu9575/multithreaded-download-manager/wiki/About-%22Cannot-access-internal-storage%22

there is no "Accept Cookies From Websites" option anymore in firefox. & the MDM works fine as is. I just have that annoying red popup I cannot get rid of, clicking on the white X button does nothing.

XxUnkn0wnxX avatar Jul 05 '22 00:07 XxUnkn0wnxX

Same problem.

Remonli avatar Jul 20 '22 17:07 Remonli

I reinstalled it and it works again.

linsui avatar Jul 22 '22 13:07 linsui

103.0b9 dev edition - still the same problem...

Evengard avatar Jul 26 '22 04:07 Evengard

Stable is on 103 now. The same problem also exists there.

Onurtag avatar Jul 29 '22 02:07 Onurtag

Same issue

Anutrix avatar Jul 30 '22 10:07 Anutrix

Same issue on Firefox v103.0.2 (64bit)

t0rzz avatar Aug 09 '22 19:08 t0rzz

希望作者能早点修复这个问题。

aiprog avatar Aug 12 '22 01:08 aiprog

@XxUnkn0wnxX please change the title to reflect the actual problem. Internal storage error message appears on Firefox stable too.

t0rzz avatar Aug 12 '22 01:08 t0rzz

Sorry for the delay.

This is only a display problem. The extension can be used even if this error shows. (unless "Always use private browsing mode" is on, then the instructions needs to be done.)

I can fix it, but right now the extension has a more severe problem #179, which can cause downloads to be corrupted. Unfortunately it is a Firefox bug and can only be fixed by updating to Firefox 104 (to be released on 2022-08-23).

If I fix this issue and update the extension now, #179 will still corrupt the files.

I'm thinking about waiting until 2022-08-23 to update the extension, so that both problems are fixed. Meanwhile, if you are not affected by #179 (not seeing file corruption), please go to the extension options → InterfaceCustom CSS, and add this code:

:root[data-path="panel/panel"] #no-storage-access {
    display: none;
}

Then the error will be hidden.

What do you think?

@aiprog Click here for the comment in Chinese 点此展开中文 抱歉现在才回复。

这个只是显示问题,即使有这个错误,扩展也可以使用(但如果启用了“一律使用隐私浏览模式”,扩展就无法使用了,这时就需要按链接里的方法操作了)

这个错误可以修复,但目前扩展还有这个更严重的问题 #179,可能导致下载的文件不正确。#179是火狐的bug,只能更新到火狐104解决(8月23日发布)。如果先修复这个显示问题,更新扩展,#179仍然会导致文件损坏。

我想等到8月23日再更新,这样两个问题一并解决。目前如果没遇到#179(下载的文件是正常的),建议到扩展选项→界面→自定义CSS,添加下面的代码:

:root[data-path="panel/panel"] #no-storage-access {
    display: none;
}

错误信息就可以隐藏了。

有什么建议吗?

jingyu9575 avatar Aug 12 '22 14:08 jingyu9575

Yes, I also have same problem as #179 , now I use aria2c instead, let's wait for firefox 104.

Anyway, @jingyu9575 thanks for your great job of the past years ,this ext is really very helpful to firefox users.

Remonli avatar Aug 12 '22 14:08 Remonli

L

Sorry for the delay.

This is only a display problem. The extension can be used even if this error shows. (unless "Always use private browsing mode" is on, then the instructions needs to be done.)

I can fix it, but right now the extension has a more severe problem #179, which can cause downloads to be corrupted. Unfortunately it is a Firefox bug and can only be fixed by updating to Firefox 104 (to be released on 2022-08-23).

If I fix this issue and update the extension now, #179 will still corrupt the files.

I'm thinking about waiting until 2022-08-23 to update the extension, so that both problems are fixed. Meanwhile, if you are not affected by #179 (not seeing file corruption), please go to the extension options → InterfaceCustom CSS, and add this code:

:root[data-path="panel/panel"] #no-storage-access {
    display: none;
}

Then the error will be hidden.

What do you think?

@aiprog Click here for the comment in Chinese 点此展开中文

It’s fine, Firefox 104 will be released in a couple weeks, so we can focus on #179. Storage access display is not a problem, it works fine anyway. Thank you 👍

t0rzz avatar Aug 12 '22 15:08 t0rzz

Now that Firefox 104 has fixed #179, this display error has been removed. Please update Firefox and the extension and see if it works.

jingyu9575 avatar Aug 25 '22 11:08 jingyu9575

@jingyu9575 Looks like AMO roll back this ext to 3.2b4 , pls check ?

Remonli avatar Sep 29 '22 00:09 Remonli

It seems that this version is rejected by the reviewer. Here is the detailed review. I believe it is a mistake as all the mentioned files do not exist in my extension. I've resubmitted and let's see what happens.

Details:
This add-on didn't pass review because of the following problems:

1)  ‘eval’ is only necessary in rare cases. Please use a different method or explain why eval is required in your add-on.
libs\ToolHelper.js - line 171, 181, 243, 249

In addition, the following is required to complete the review:

1) Your add-on includes a third-party library. Please provide the origin of the exact library version you were using and make sure you are using an exact copy of the original maintainers release version. For more information, refer to https://extensionworkshop.com/documentation/publish/third-party-library-usage/ .
Please add only one version of a library.
libs\layer\mobile\layer.js
libs\layer\layer.js

2) This version contains minified, concatenated or otherwise machine-generated code. Please provide the original sources, together with instructions on how to generate the final XPI. Source code must be provided as an archive and uploaded using the source code upload field, which can be done during submission or on the version page in the developer hub.

Please read through the instructions at https://extensionworkshop.com/documentation/publish/source-code-submission/ .
libs\fetch-filter.min.js
content-script.js

Version(s) affected and disabled:
3.2b5

jingyu9575 avatar Sep 29 '22 01:09 jingyu9575

Finally version 3.2b7 with this fix passes review. Nothing has been changed except the version number.

jingyu9575 avatar Oct 14 '22 11:10 jingyu9575