Skip to content

Commit

Permalink
fix: update tuple types for sasl option (#78)
Browse files Browse the repository at this point in the history
The sasl option for producers and consumers should be `{:atom, :string,
:string}` instead of `{:string, :string, :string}` to match Brod and all
documentation.
  • Loading branch information
btkostner authored Dec 26, 2023
1 parent 460e678 commit 2860d8b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/kafee/consumer.ex
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ defmodule Kafee.Consumer do
and password. For example, to use plain username and password
authentication you'd set this to `{:plain, "username", "password"}`.
""",
type: {:tuple, [:string, :string, :string]}
type: {:tuple, [:atom, :string, :string]}
],
ssl: [
default: false,
Expand Down
2 changes: 1 addition & 1 deletion lib/kafee/producer.ex
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ defmodule Kafee.Producer do
and password. For example, to use plain username and password
authentication you'd set this to `{:plain, "username", "password"}`.
""",
type: {:tuple, [:string, :string, :string]}
type: {:tuple, [:atom, :string, :string]}
],
ssl: [
default: false,
Expand Down

0 comments on commit 2860d8b

Please sign in to comment.