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

FMN doesn't deliver messages through matrix-irc bridge #217

Closed
ignatenkobrain opened this issue Aug 6, 2017 · 5 comments · Fixed by #228
Closed

FMN doesn't deliver messages through matrix-irc bridge #217

ignatenkobrain opened this issue Aug 6, 2017 · 5 comments · Fixed by #228

Comments

@ignatenkobrain
Copy link

After I switched for IRC to use Matrix, I stopped getting any fedmsg notifications which is weird..

@nirik suggested to open ticket here one day.u

@jeremycline
Copy link
Member

This is odd. I'm able to send PMs to my Matrix user via IRC, but notifications definitely don't arrive. I'm not sure what the cause is, exactly.

@ignatenkobrain
Copy link
Author

I think https://github.com/matrix-org/matrix-appservice-irc/wiki/End-user-FAQ#why-i-dont-get-messages-from-unregistered-to-irc-irc-users might explain why I don't receive those notifications...

TL;DR; matrix doesn't allow unregistered users to send PMs

@ignatenkobrain
Copy link
Author

@jeremycline cool! has this been deployed in prod already? If not, when?

Missing notifications for very long time already!

@jeremycline
Copy link
Member

I was thinking of kicking it out to stage tomorrow. I can ping you when that happens. Infra is in freeze at the moment so it may be a little while until it's in prod, but I don't think it'll be hard to get a freeze exception since FMN doesn't really work right now anyway.

As a work-around you could just use stage since until the fix is in prod since it gets all the production fedmsgs.

@jeremycline
Copy link
Member

This is now in staging and I can get notifications with +R set again so it should also work with Matrix assuming you've also authenticated with nickserv.

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

Successfully merging a pull request may close this issue.

2 participants