Backported simple support for BIMI indicators

[dodmi]

No description provided.

[dodmi]

Can you help me a little?
It' working, but I don't know how to fix the Type Check errors :(

[lieser]

The problem is that TypeScript does not understand the Mozilla specific module system used here (The jsm files with EXPORTED_SYMBOLS and the Cu.import(...) calls).
The result is that it treats them as normal JavaScript files. And wrongly assumes they share the same scope.

Two possible workarounds would be:

• Don't declare the variables in the global scope of the module, but e.g. inside the BIMI object. Either by making them (getter) functions or using a Immediately Invoked Function Expression when creating the BIMI object (https://developer.mozilla.org/en-US/docs/Glossary/IIFE#avoid_polluting_the_global_namespace, like it is done for the Verifier object in modules/dkimVerifier.jsm.js.
• Add a // @ts-ignore to simply tell TypeScript to ignore this error.

[dodmi]

• Don't declare the variables in the global scope of the module, but e.g. inside the BIMI object. Either by making them (getter) functions or using a Immediately Invoked Function Expression when creating the BIMI object (https://developer.mozilla.org/en-US/docs/Glossary/IIFE#avoid_polluting_the_global_namespace, like it is done for the Verifier object in modules/dkimVerifier.jsm.js.

Thank you, I did this and it seems to be fine now.

[lieser]

Note that this PR has a conflict that needs to be resolved.

[dodmi]

Note that this PR has a conflict that needs to be resolved.

Well yes, in this PR DKIM_InternalError is still used. I‘ll fix it.