-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
** (MatchError) no match of right hand side value: {:ok, :http} #523
Comments
This issue may be resolved on the master branch. |
Oh even better, I didn't see an open issue with this error code so I thought I might be the first person to discover it. There go my hopes at achieving internet fame! |
Got the same thing I think?
|
@mcfiredrill Yes, this is exactly the error I had. Based on the above comment you can either point to master or like I did, put a bit of a hacky solution in. |
Fixed in master for me thanks @Teifion |
using master didn't help at all - only introduced new problems and issues.
[...]
|
Using master is the fix for the match errors above. For the later error
You need to upgrade to OTP 25.1 or above, where |
This bug is a breaking change to Nostrum and should be hotfixed or at least put in a new version 0.8.1. That is more reliable than to change one's dependency version back to |
PR's welcome |
Discord have (possibly just this morning) changed something and the login breaks.
I have managed to work around it by changing line 103 in ratelimiter.ex to
{:ok, _} = :gun.await_up(conn_pid)
but this doesn't feel like a good solution long term. I'm creating the issue in part to help people searching for it.The text was updated successfully, but these errors were encountered: