-
Notifications
You must be signed in to change notification settings - Fork 25
Options to Spell.connect/2 are ignored #52
Comments
True! Happy to look at a PR for this, otherwise I'll get to it in the next 2 weeks. Thanks for the report, @clayrichardson |
No problem, I was trying to figure out the best way to fix this, but I am unsure :) |
I also noticed that when a timeout error happens, there is no retry, thus this seems to be ignored as well: |
😊 🐛 -- thanks! |
Also, I'm not sure if I'm using this lib properly, as I put Spell.connect into my own connect/1 function inside a GenServer, and case on Spell.connect. On {:error, timeout} I call my connect/1 again, but it seems to spawn multiple PIDs which send messages to my GenServer's handle_info callback. So it might be retrying somewhere after the lib returns the timeout error. Maybe this will help, I'm new to Elixir, so I'm probably doing something wrong here: https://github.com/clayrichardson/spell-issue-example/blob/master/lib/ingestory/worker.ex |
This value seems to be ignored: https://github.com/MyMedsAndMe/spell/blob/master/lib/spell.ex#L125
The text was updated successfully, but these errors were encountered: