keithjgrant / omnibear

A Micropub browser extension
MIT License
36 stars 8 forks source link

Sign In does not work anymore #23

Closed cweiske closed 6 years ago

cweiske commented 7 years ago

When signing into omnibear with https://commentpara.de/, I only see the website with

OmnibearAuthentication successful Retrieving access token...

And then nothing.

Happens with Chromium 60.

keithjgrant commented 7 years ago

I noticed similar behavior when trying to get the plugin working in Firefox. I’ll have to see if something in the permissions has changed

keithjgrant commented 7 years ago

Authentication is working for me using indieauth.com as my authentication endpoint. On the login screen, can you right-click -> "Inspect" and see if there are any console errors and/or network errors? On the Retrieving access token screen, what is in the url bar?

keithjgrant commented 7 years ago

Version 0.5.2 is publishing in the Chrome store now. This includes some additional permissions for the plugin that were required to make the plugin work in Firefox, and I’m hopeful these changes will fix Chromium as well.

cweiske commented 7 years ago

Omnibear 0.5.2 does neither work with Chromium 60.

There is no error (apart from the missing favicon.ico) on the omnibear.com site console. There is no error in the background page console.

It stops as before at

OmnibearAuthentication successful Retrieving access token...

The full URL in the address bar is:

http://omnibear.com/auth/success/?code=ZW1vamk9JTVDMzYwJTVDMjM3JTVDMjIyJTVDMjUxJm1lPWh0dHBzJTNBJTJGJTJGY29tbWVudHBhcmEuZGUlMkZ1c2VyJTJGMy5odG0mc2NvcGU9cG9zdCtjcmVhdGUrZGVsZXRlK3VwZGF0ZSZzaWduYXR1cmU9RklYTUU%3D&me=https%3A%2F%2Fcommentpara.de%2Fuser%2F3.htm&state=very-secret-omnibear-state

Btw, you can try with https://commentpara.de/ yourself - anyone can login there.

grantcodes commented 6 years ago

I'm having the same issue in the vivaldi browser.

Same thing with stopping at "Retrieving access token..." and no errors to be seen anywhere. Works fine for me in chrome 60 though.

grantcodes commented 6 years ago

OK I know what is causing the issue, and it's a super easy fix in vivaldi but I don't know if it has any other side effects.

https://github.com/keithjgrant/omnibear/blob/347f24f6e83e0ce9a52cdcbc8fef98c0b38f90db/src/background.js#L86

If you remove changeInfo.status === 'loading' from this line it works fine, in vivaldi at least the status property is sent in a separate event. I'm not sure if removing that breaks anything but seems ok to me.

keithjgrant commented 6 years ago

Interesting. It looks like Vivaldi fires that event multiple times, each time passing only part of the changeInfo object: screen shot 2017-09-27 at 3 43 47 pm

So both the status and the url are not ever present at the same time to verify together. I’ll have to do a little experimenting to see if I really need that check for loading

cweiske commented 6 years ago

Omnibear 0.5.3 with Chromium continues to have the problem.

keithjgrant commented 6 years ago

0.6.0 has been published to both Chrome and Firefox extension stores. I’ve removed the check for "loading" status, so this should be working now. Let me know if this resolves the issue.

cweiske commented 6 years ago

With omnibear 0.6.0 and chromium 60 I get a JavaScript error on the background page:

background.js:1 POST chrome-extension://cjieakdeocmiimmphkfhdfbihhncoocn/null net::ERR_FILE_NOT_FOUND
(anonymous) @   background.js:1
value   @   background.js:1
u   @   background.js:1
EventImpl.dispatchToListener    @   extensions::event_bindings:380
publicClassPrototype.(anonymous function)   @   extensions::utils:149
EventImpl.dispatch_ @   extensions::event_bindings:364
dispatchArgs    @   extensions::event_bindings:237
dispatchEvent   @   extensions::event_bindings:246

The omnibear website still says "Retrieving access token..."

keithjgrant commented 6 years ago

Strange. I’m able to login in Version 60.0.3098.0 OS X 64bit using commentpara.de. I can do both the anonymous login and the named login. Are you on Windows or Mac?

I am having trouble posting via commentpara.de, but login is working.

Maybe try removing the extension entirely & reinstalling?

cweiske commented 6 years ago

I tried removing and re-installing the extension, but that did not help. I'm on Version 61.0.3163.79 (Official Build) Built on Ubuntu , running on Ubuntu 14.04 (64-bit)

cweiske commented 6 years ago

I can sign in on firefox 55 though. This is enough for me now, so you can close this issue if you don't want to work on this anyore.

keithjgrant commented 6 years ago

This might happen if you somehow had localStorage disabled. I use localStorage to save some of your auth settings, including the token endpoint. I wonder, do you have the "block sites from setting any data" option enabled?

grxbu

cweiske commented 6 years ago

That was already enabled. I did now create a new clean profile, installed omnibear as only extension and it still gave the same error on the background page. Let's call it a day and see my machine as heisenbuggy.

cweiske commented 6 years ago

I am having trouble posting via commentpara.de, but login is working.

This was a bug in commentpara.de - posting likes works now.