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

Firefox 56 compatibility #796

Open
2 of 8 tasks
SammysHP opened this issue Oct 6, 2017 · 21 comments
Open
2 of 8 tasks

Firefox 56 compatibility #796

SammysHP opened this issue Oct 6, 2017 · 21 comments

Comments

@SammysHP
Copy link
Member

SammysHP commented Oct 6, 2017

There are some deprecation warnings (not necessary to fix because FF56 will be the last version that is compatible with Vimperator) as well as some major bugs:

Bugs:

Warnings:

  • DEPRECATION WARNING: Search service falling back to synchronous initialization. This is generally the consequence of an add-on using a deprecated search service API.
    You may find more details about this deprecation at: https://developer.mozilla.org/en-US/docs/XPCOM_Interface_Reference/nsIBrowserSearchService#async_warning
  • JavaScript 1.6's for-each-in loops are deprecated; consider using ES6 for-of instead
  • String.localeCompare is deprecated; use String.prototype.localeCompare instead
  • String.replace is deprecated; use String.prototype.replace instead
  • String.toLowerCase is deprecated; use String.prototype.toLowerCase instead
@SammysHP
Copy link
Member Author

SammysHP commented Oct 6, 2017

I tried to fix the protocol handler bug for several hours without success. The problem is in common/components/protocol.js:

let channel = Components.classesByID["{61ba33c0-3031-11d3-8cd0-0060b0fc14a3}"]
                        .getService(Ci.nsIProtocolHandler)
                        .newChannel(ioService.newURI("chrome://liberator/content/data", null, null))
                        .QueryInterface(Ci.nsIRequest);

and fails with NS_ERROR_ILLEGAL_VALUE: Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIProtocolHandler.newChannel].

We need help!

@SammysHP
Copy link
Member Author

SammysHP commented Oct 6, 2017

Actually I'm rather happy that this broke! IMHO the chrome-data protocol handler is a big security issue because it might allow websites to gain chrome privileges. But I might be wrong.

@kyao-777
Copy link

kyao-777 commented Oct 7, 2017

@SammysHP what's the meaning of the FF56 will be the last version that is compatible with Vimperator. You cannot update the vimperator for firefox in the future. Sorry, my english is bad.

@SammysHP
Copy link
Member Author

SammysHP commented Oct 7, 2017

@kyao-777
Please read #705.

@kyao-777
Copy link

kyao-777 commented Oct 7, 2017

@SammysHP thanks

@JinhaoPro
Copy link

My English is not quite good, but is there anyone who can tell me how to solve #786 problem?

@SammysHP
Copy link
Member Author

You have to build an XPI from the ff56-fixes branch yourself and use it with a custom built version of Firefox that does not enforce signature verification. Official update might take a few weeks.

@maoanran
Copy link

maoanran commented Oct 18, 2017

@SammysHP , from Firefox 53, new add-ons can only be WebExtensions, so we can't sign it ourselves anymore. Sad.

By the way, I know you guys are planing to rewrite web extension version. I would like to help, If there is any plan system like jira? Thanks.

@P4Cu
Copy link

P4Cu commented Oct 18, 2017

I'm also highly interested in making that working again so I'm willing to spend some of my time on that but if we're going to split the work I would need an architectural description on what should I change to avoid doing the same stuff by several ppl.

For a moment whomever is interested in keeping using it just install older version like:
https://launchpadlibrarian.net/333533798/firefox_55.0.2+build1-0ubuntu0.16.04.1_amd64.deb

@SammysHP
Copy link
Member Author

@maoanran
That's why I recommended to install a version of Firefox that does not require the signature verification.

@P4Cu
Please read #705.

@SammysHP
Copy link
Member Author

Just some answers to recent comments:

  • Vimperator is semi-maintained currently. There won't be any new features unless someone contributes to it. I try my best to keep Vimperator running, but…

  • Firefox 56 is the last version that is compatible with Vimperator. Old extension won't work with FF57 and everything must be programmed again from scratch.

  • It seems that nobody with access to addons.mozilla.org is active at the moment. And an updated version may require several weeks for review and might be rejected after all.

  • Official versions of Firefox require addons to be signed. Otherwise we would provide a version for direct download.

  • All Bugs under Firefox 58 are fixed, sadly it is not easy to distribute the fixes, see above.

@maoanran
Copy link

maoanran commented Oct 24, 2017

@SammysHP A signed version for download would be nice. :)

@SammysHP
Copy link
Member Author

@maoanran
Only Mozilla can sign an extension and that might not happen anymore.

@Luftzig
Copy link

Luftzig commented Oct 25, 2017

@SammysHP I'm trying to understand what can I do to help. If there are tasks that can be tackled by people not familiar with the code base I can suggest this project for the next Goodness Squad meeting in Israel and get several people to work on them for ~3 hours.

@SammysHP
Copy link
Member Author

SSL popup is broken again…

@agush22
Copy link

agush22 commented Oct 30, 2017

Any updated xpi?

@SammysHP
Copy link
Member Author

SammysHP commented Nov 5, 2017

SSL popup is broken again…

Currently it works again… Probably some concurrency issues between Firefox and Vimperator.

@JulioJu
Copy link

JulioJu commented Nov 24, 2017

@SammysHP thanks a lot to your work ! It will be very useful for Waterfox 56 (coming soon).

Some peoples actually think Vimperator is not implemented for Firefox 56 and Waterfox 56 (see https://www.reddit.com/r/waterfox/comments/7eoyfg/dont_take_away_my_vim/). Maybe, you could publish a new release on https://github.com/vimperator/vimperator-labs/releases and add a link to this Waterfox Release on the top of the README ?

@JulioJu
Copy link

JulioJu commented Feb 4, 2018

up

@tecfu
Copy link

tecfu commented Mar 30, 2018

USE FIREFOX 55 AS A WORKAROUND:

For those who wish to upgrade Firefox and have Vimperator already installed (installing a new .xpi doesn't apply to you) note that Vimperator works fine with Firefox 55.

If you are a Ubuntu user like me, you can find Firefox 55 .deb packages here at the Ubuntuzilla repository on Sourceforge:
https://sourceforge.net/projects/ubuntuzilla/files/mozilla/apt/pool/main/f/firefox-mozilla-build/.

@AnomalRoil
Copy link

No, this is not a solution. Firefox 55 is not a long term support release and should never be used anymore.

On the other hand, Firefox 52 is an ESR, which will be supported until August 28. You should use the ESR if you are willing to downgrade to still use Vimperator.

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

No branches or pull requests

10 participants