-
Notifications
You must be signed in to change notification settings - Fork 37
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
Compatibility with Thunderbird 78 #199
Comments
Thanks a lot for reporting the problem. I will try to look into it this weekend, but can't promise anything. |
Looks like TB removed the legacy XUL overlay loader this add-on still used. This mean the add-on needs to be converted into the new add-on type (MailExtensions). Which will probably not be something small. I defiantly do plan to do this conversion into the new type. But you should not expect there to be a compatible version anytime soon. |
Don't let this great add-on die... Is there a big work involved due to TB change to version 75? |
Like I said, I don't plan to let this add-on die, in fact I already started with the work a while ago. So unfortunately you will have to still wait some time. I will post here if there is a version ready there at least the basic works. |
I have a very early version ready that you could start to try out: [email protected] Note that only the basic DKIM verification works for now. Nothing else. |
Super, I can confirm it's working on TB75.0b3 (64-bit) Win10. |
With TB 76.0b1 (64-Bit) macOs Version 4.0.0pre0 is not working. I get the following error console message:
|
The same on Win10 TB 76.0b1 (64-Bit).. |
Thanks for letting me know, already noticed it will I worked on the porting today. Here is a new version that should work with TB 76 [email protected]. The version has the same limitations as the previous. Although the options page is now available again, the page doesn't work. |
Yes, it's working now on Win10 TB 76.0b1 (64-bit). |
To lessen the amount of work needed for the migration of the add-on to a webextension, I'm thinking about dropping some features. If someone depends on any of the following, please let me know:
|
I think most important is simply to see above the message DKIM test result (now it's working fine). Another good option would be to have an ability to create a filter - for example - From: ends with: ebay.com + DKIM is invalid (or missing) -> then: Move to Trash". I think these two options are the most important. Perhaps on mouse over warning icon (or OK icon - which does not exists yet) - would be nice to see more information, for example "Insecure signature algorithm: 2048 bits". |
Also waiting to see the new version, that is compatible with Thunderbird 78 |
@DavidOsipov did you already try the last version I posted above? I'm hoping to finish a new pre-release in the next few days, there at least most of the basic stuff and options work. |
Thank you for your work! I installed it but my configuration of the Add-on is not being saved after I close the add-on configuration tab. |
I just published a new pre-release 4.0.0pre1 at https://github.com/lieser/dkim_verifier/releases. It still has some limitations (see release page), but it is a lot closer to the final state than the last version I uploaded here in this issue. Please also start reporting and issues you see with with this verion (excluding the ones mentioned on the release page). Thanks.
Using the libunbound resolver needs a locally available libunbound library, regardless of the used DNS server. Please create a separate issue if you have more questions about this. |
I get this error with the prerelease version 4.0.0pre1 and TB 78.1.1:
|
Thanks for the report, but I can not reproduce it. Can you please give more information?
Note that this could also be a potential bug in TB. This error seems to olny happen if |
I had the account set up to only download message headers. I've noticed the described error only happens with messages I downloaded during the current run of TB. If I download a message body and restart TB, switching to it doesn't generate any error. I've tried with all other addons off, and the behavior is the same. |
Thanks for investigating the problem, with your info I could now reproduce it. Looks like a bug in TB to me, so I filled a bug for it: https://bugzilla.mozilla.org/show_bug.cgi?id=1659555. I also don't see an easy workaround, so lets hope it gets fixed quickly in TB itself. |
Now this plugin is not compatible with the new 78.2.2 version of Thunderbird. |
Thunderbird 81.0b4 64-bit Win OS version is the newest now.. and it's working fine. |
4.0.0pre1 works for me with TB 78.2.2 |
Question to all who used the key storing feature: |
Issue encountered:
|
@leagris Thanks for reporting the error. Did you have debug enabled, and also saved the messages before the error? |
i know i'm very late to this, but it's sad to see that the status bar icon was scrapped. i'd love to see the icon come back even if next to the "From" header, similar to the well-known signer Favicon icon, which speaking of also hasn't worked for me since the TB update that broke the add-on. |
@Railander I created the separated issue #237 for re-adding the status bar icon. But don't expect this to some back unless there will be more requesting this, as it currently it is very low on my to long priority list. Regarding the Favicon icon, are you sure you have enabled the option, and are viewing a mail of one of the supported domains for this (https://github.com/lieser/dkim_verifier/blob/master/data/favicon.json)? This should work in TB 78 too. If not, please disable all other add-ons, restart TB and try again. Maybe there is an incompatibility with another add-on. |
Attached is a test version for people affected by recently fixed bugs. The storing of DKIM keys is also migrated in this. |
i understand the status bar might be more involving, i'd be fine even if the ball icon was readded in the "From:" header instead of color filtering the text or background. regarding the favicon, it's always been enabled (just double-checked) and at some point it stopped showing. for example, our corporate domain uses g-suite and it did show the favicon back then but no longer does. |
Did you look if the signing domain appears in the list I linked to? Maybe the domain changed, at least i can't remember that I removed a google related domain from it. To debug the problem, please:
|
@peci1 Note that your issue with only downloading the message headers (https://bugzilla.mozilla.org/show_bug.cgi?id=1659555) seems to be fixed in the latest daily. Don't know then it will be fixed in the normal release of Thunderbird. |
Thanks for letting me know! |
Good news, the conversion to an MailExtension is finally completed. I will still do some testing, and probably release the final 4.0.0 version next weekend. If you want to already try it out, attached is the current state. |
Great thank you Philippe - working fine so far in 78.9.1 |
One comment - the DKIM button in the header row on right. My theme is dark them - the button is in dark them but clicking it brings up white background with 4 buttons. I'd say change the white bg which holds the 4 buttons to something from dark palette - like the rest of TB when in dark theme. Best gene |
Thanks for reporting the issue with the dark theme. Just tried the one built into TB (on Windows), and although it looks bad and definitely needs improvements, it does not look like what you described. Will look into it next week. Note that the difference in the buttons you descibe is probably from most of the buttons being disabled. They only get enabled if the action actually makes sense for the currently viewed mail and the configured options. |
Bigger issue to me is the white background. |
And these are really minor issues - the important stuff is the functionality and thats working great - thank you! |
@gene-git The dark theme issue should be fixed. Can you please try out the attached version? Thanks. |
Yes - confirmed - it looks fine now - thank you! |
Took a lot longer than I initially hoped for, but with the release of version 4.0.0 I did just now the migration is finally over. On ATN it is still under review, and should show up in the next couple of days. Thanks a lot to all of your for your patience, and reporting any bugs you found during the process. |
thank you ... :) |
There exist a 4.0.0 pre-release there the main verification already works in TB 78. It is available for download at https://github.com/lieser/dkim_verifier/releases
The xpi can be downloaded under Assets at the bottom of each release.
Also note that the outdated Enigmail add-on can cause problems. Disable or remove the Enigmail add-on and restart Thunderbird to resolve this.
Installed today 3.1.0 version on TB 73.0b1. I see that add-on is installed - but on plugin manage page is just blank page. Also no action on TB mail panel - no text or menu available..
Error console messages:
The text was updated successfully, but these errors were encountered: