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

The Firefox client does nothing #96

Open
forteller opened this issue Dec 4, 2014 · 8 comments
Open

The Firefox client does nothing #96

forteller opened this issue Dec 4, 2014 · 8 comments

Comments

@forteller
Copy link

On my Ubuntu Gnome machine the Mitro Firefox extention does nothing useful. It just opens a box where it shows the Mitro websites, with absolutely no secrets. I've tried in many versions of Firefox, but it's always the same. Mitro is not a viable sollution until this is fixed. Thanks!

Please see attached screenshot
screenshot at 02-16-20

@vijayp
Copy link
Collaborator

vijayp commented Dec 6, 2014

strange; what version of ff?

@forteller
Copy link
Author

Currently I'm using 34, but it's been like this with all versions of Firefox I've tried it on.

@wjam
Copy link

wjam commented Dec 18, 2014

I'm also experiencing this issue on the latest nightly (Linux). Here is a stacktrace that appears in the browser console when opening the extension while logged in:

TypeError: document.getElementById(...) is null
Stack trace:
updateLoginState/</</</<@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/____/.mozilla/firefox/egzrgc5a.Temp/extensions/[email protected]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/popup.js:4:335
Client.prototype.setMethod/this[a]</<@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/____/.mozilla/firefox/egzrgc5a.Temp/extensions/[email protected]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/client.js:13:299
Client/this.processIncoming@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/____/.mozilla/firefox/egzrgc5a.Temp/extensions/[email protected]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/client.js:2:226
ExtensionHelper/this.bindClient/<@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/____/.mozilla/firefox/egzrgc5a.Temp/extensions/[email protected]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/helpers.js:7:411
onEvent@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/____/.mozilla/firefox/egzrgc5a.Temp/extensions/[email protected]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://gre/modules/commonjs/sdk/content/content-worker.js:45:22
onChromeEvent@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/____/.mozilla/firefox/egzrgc5a.Temp/extensions/[email protected]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://gre/modules/commonjs/sdk/content/content-worker.js:91:16

Also note that when not logged in, the box that appears is also different:

mitrologgedout

@wjam
Copy link

wjam commented Dec 18, 2014

I believe this will be fixed by PR #89 and is a duplicate of issue #40

@antitoxic
Copy link

Same here, Latest firefox on Fedora 21.

@denadai2
Copy link
Contributor

@wjam is correct :)

@vijayp
Copy link
Collaborator

vijayp commented Mar 16, 2015

Are there any pending PRs required to make this work or is the latest
published version correct?

On Wednesday, February 25, 2015, Marco De Nadai [email protected]
wrote:

@wjam https://github.com/wjam is correct :)


Reply to this email directly or view it on GitHub
#96 (comment).

@winstonhong
Copy link

Hello Folks.

I have tested Ubuntu14.04/Firefox 37, Windows 7/Firefox 37, Ubuntu14.04/Chrome 42, and Windows 7/Chrome 42.

I confirm that this issue has been fixed by denadai2.

@vijayp, I confirm that the two PRs by denadai2 for this commit is still pending.

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

No branches or pull requests

6 participants