I'm not an elixir expert, so apologies if I'm understanding this issue incorrectly. When trying to upgrade jose to 1.11.9, there's an added spec to verify_strict/3:
|
@spec verify_strict(t(), [String.t()], binary()) :: {valid? :: boolean(), jwt :: t(), jws :: JOSE.JWS.t()} |
Our dialyzer check for our application is failing because we were anticipating such results like {:error, reason} in addition to the currently spec'd {valid? :: boolean(), jwt :: t(), jws :: JOSE.JWS.t()}. I believe that the catch result should also be added to the type spec, here is the catch code block:
|
catch |
|
class, reason -> |
|
{class, reason} |
|
end |
.
I'm not an elixir expert, so apologies if I'm understanding this issue incorrectly. When trying to upgrade jose to 1.11.9, there's an added spec to
verify_strict/3:erlang-jose/lib/jose/jwt.ex
Line 312 in 66e6eec
Our dialyzer check for our application is failing because we were anticipating such results like
{:error, reason}in addition to the currently spec'd{valid? :: boolean(), jwt :: t(), jws :: JOSE.JWS.t()}. I believe that the catch result should also be added to the type spec, here is the catch code block:erlang-jose/lib/jose/jwt.ex
Lines 352 to 355 in 09de6d2