-
Notifications
You must be signed in to change notification settings - Fork 43
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
Login error: Null value for $.viewer.message_threads.sync_sequence_id #138
Comments
Yeah, I'm currently getting this issue too. |
same here. cant login because of it. seems like something has changed. |
I suspect this has something to do with Facebook dropping support for ancient versions of Messenger: https://messengerblog.com/general-messenger/update-on-support-for-older-versions-of-messenger-and-facebook-apps/ |
See also: dequis/purple-facebook#306 |
dup #130 |
NOT a dup of #130. This message is NOT transient - it is a permanent error. something has changed server side, and this plugin won't work unless a workaround is enacted. Linking a closed bug and calling it a dup because the symptom is superficially the same is not helpful as it creates the impression the bug reports are invalid - there is a genuine issue here and it has been triggered by a serverside change which is why it's appearing in both implementations as per dequis/purple-facebook#306 |
@exitdown: I accept that the original issue refers to a transient bug. However, I mildly disagree that bringing up a closed bug is not useful: similar prior issues may still be of relevance and providing some link to with associated information ensures there's more information available now, and there's continuity in the treatment. (I also note that dequis/purple-facebook#179 was brought up similarly.) |
@jashank yes - if you had added some context to that effect your point is valid - however by simply posting a "dup" with no context most people would read it as dismissive of the issue - which is why i made it clear in my subsequent post that it was not a dupe of that previously closed bug :) either way,hopefully now its obvious there's an issue here's hoping the protocol modifications are relatively easy so we can all get back to using this excellent software :) |
well, fuck. |
Fix 9390579 I'll prepare a release shortly. edit nevermind see below |
|
fuck |
At least this issue is something I predicted that could happen, but was hoping it wouldn't. The /send_message2 mqtt topic (that we use to send json messages instead of thrift) is very old. Bad news is that I'll have to implement the newer, more complex one right now. Good news is that we'll get stickers. |
thank you for your efforts dequis! |
Yes, thanks dequis! Been using the plugin for a while now and really appreciate it :) |
I tried the new plugin and now I get "Arguments are invalid" |
Same here, but I think that's what dequis was saying about having to implement complex stuff |
I can confirm this issue as well Edit: Maybe this should be opened as new issue? I installed the latest commit: 20170329
|
@malinkb read up |
Please try latest master. 1629e2f fixes message sending - turned out to be just a boring "set user agent version" thing. The old-style 93a62d3 fixes invalid arguments errors when fetching channel lists. edit: nope nevermind not yet |
And af132a8 fixes a last minute bug. You can test it now. |
Thanks dequis, currently being an idiot and failing to work out how to install it, but that's just me |
Builds should appear in the package repos shortly: https://build.opensuse.org/package/show/home:jgeboski/bitlbee-facebook edit: done |
Thanks for such a fast fix & package! |
Works now. Thanks for fast fix :) |
Ping @spwhitton, this may be worth uploading to debian, too. |
@dequis Are you planning tagging a new release soon? |
Yeah I'll prepare the long overdue 1.1.0 release now. Here's a long-form explanation of what happened here: As others suggested, https://messengerblog.com/general-messenger/update-on-support-for-older-versions-of-messenger-and-facebook-apps/ is indeed relevant. They deploy to production on tuesdays which is when it broke. I was aware of this, but, as far as I knew, nothing needed updating, because it says that versions older than 5 will be discontinued but we're compatible up to version 38, mostly. I suspected the Versioning in facebook is complicated, which is both good and bad. Good because it allows them to have excellent backwards compatibility (as far as they want to keep it). Bad because it's hard to understand where to draw the line on what a version means. There were two fixes:
Not too bad overall. Next time we won't have the surprise version check since we're sending a version exactly where they expect it. Also, I'm happy with the approach they've taken to deprecate very old versions. This announcement sounds like they are committing to 3 years or so of support for old versions, which sounds great. EDIT: I should mention, the graphql hash for the first (and most important) fix was found by @EionRobb, go thank him too. |
(keeping this open for visibility) |
You're awesome, @dequis! |
Thanks for the quick fix @dequis. I tipped you some btc. |
This fixes especially bitlbee/bitlbee-facebook#138. Reported by: cognet git-svn-id: svn+ssh://svn.freebsd.org/ports/head@437203 35697150-7ecd-e111-bb59-0022644237b5
This fixes especially bitlbee/bitlbee-facebook#138. Reported by: cognet
Hi guys! It would be awesome to get a new debian-package as well... Do you think it's possible? Awesome response time on the patch! :) 👍 |
The builds in https://build.opensuse.org/package/show/home:jgeboski/bitlbee-facebook happen automatically a few minutes after each commit. |
Aha. Sorry - Did not know that one. :) Thanks mate! Great! |
I just realized I had a problem and it's already fixed 👍 |
Do I have to recreate the facebook account in bitlbee for this fix? I tried updating to |
Recreating accounts is not needed. You do need to restart the bitlbee daemon and reconnect your irc client. |
I am still having this issue. Can you somebody give me clear suggestion? |
@lew1s did you
If so, please open an issue with a bit more debug info, because this specific issue seems to be fixed for most people on this thread. |
@lew1s The issue was persisting for me too after following the above steps, and even after a complete reboot. I was using the latest ubuntu 16.04 apt package of bitlbee ( |
@thallada "was"? does it still not work? Check that you're using the right bitlbee version with |
@dequis sorry for not being clear. Upgrading to 3.5.1 fixed the issue for me. |
I am on windows and dont know where to update bitlbee-facebook. Is there some guide for windoes? |
@lew1s Are you sure you're not using purple-facebook instead? https://github.com/dequis/purple-facebook/wiki/Installing-on-Windows I mean, bitlbee doesn't really support windows, except through cygwin. If you use cygwin the same installation methods in the README apply. |
I installed puple-facebook from the repo but I am still getting the error. |
@JR1994 This is bitlbee-facebook, not purple-facebook. |
I'm locking this thread since this is resolved. If anyone has any issues please open a new issue instead. |
ADMIN EDIT
This is already fixed in the latest release. This ticket is only open for visibility
Release notes: https://github.com/bitlbee/bitlbee-facebook/releases
Original ticket description
Got an
Error: Failed to send message
this morning and got disconnected from Facebook, and now I'm getting aLogin error: Null value for $.viewer.message_threads.sync_sequence_id
every time BitlBee reconnects to Facebook.The text was updated successfully, but these errors were encountered: