search

mailman-elixir / mailman

Mailman provides a clean way of defining mailers in your Elixir applications
https://github.com/mailman-elixir/mailman
Other
203 stars 73 forks source link

Argument error with elixir 1.11.4 #92

Open lanarchyste opened 3 years ago

lanarchyste commented 3 years ago

I updated elixir to version 1.11.4, and since then I have the following error:

** (ArgumentError) argument error :erlang.split_binary('250-PR0P264CA0221.outlook.office365.com Hello [37.157.224.248]\r\n', 3) (gen_smtp 0.14.0) /app/deps/gen_smtp/src/binstr.erl:114: :binstr.substr/3 (gen_smtp 0.14.0) /app/deps/gen_smtp/src/gen_smtp_client.erl:623: :gen_smtp_client.read_possible_multiline_reply/1 (gen_smtp 0.14.0) /app/deps/gen_smtp/src/gen_smtp_client.erl:488: :gen_smtp_client.try_EHLO/2 (gen_smtp 0.14.0) /app/deps/gen_smtp/src/gen_smtp_client.erl:549: :gen_smtp_client.do_STARTTLS/2 (gen_smtp 0.14.0) /app/deps/gen_smtp/src/gen_smtp_client.erl:520: :gen_smtp_client.try_STARTTLS/3 (gen_smtp 0.14.0) /app/deps/gen_smtp/src/gen_smtp_client.erl:259: :gen_smtp_client.open_smtp_session/2 (gen_smtp 0.14.0) /app/deps/gen_smtp/src/gen_smtp_client.erl:200: :gen_smtp_client.try_smtp_sessions/3 (gen_smtp 0.14.0) /app/deps/gen_smtp/src/gen_smtp_client.erl:189: :gen_smtp_client.send_it/2 (mailman 0.4.2) lib/mailman/external_smtp_adapter.ex:17: Mailman.ExternalSmtpAdapter.deliver/3

skosch commented 3 years ago

Hey, thanks for reporting this. Looks like you're running into this issue, which in turn is due to a breaking change in OTP that's being discussed here.

I've subscribed to those threads; once they agree on a solution they'll hopefully release new versions of both OTP and gen_smtp, which should solve your problem as well. Until then, the easiest thing to do may be either to use gen_smtp's temporary patch or to downgrade your Elixir installation – sorry!

lanarchyste commented 3 years ago

Hi,

Thank you for your quick reply !

For the moment, I have downgraded the version of elixir because I don't have the need to use the latest version.