`Cloak.Ecto.SHA256.hash_string/1` break when the value is `nil` about cloak_ecto HOT 3 OPEN

@SimonLab
It did resolve the issue for me, together with your suggestion: "Cloak.Ecto.SHA256, default: "")"
I hope the PR will solve the issue without "workarounds" :)

from cloak_ecto.

This is my solution for the issue:

    changeset
    |> get_field(:contact_email)
    |> case do
      nil ->
        changeset

      email ->
        put_change(changeset, :contact_email_hash, email)
    end

from cloak_ecto.

Thanks @vitalis this resolves the issue when the contact_email is nil but I don't think it does when the current contact_email_hash is nil.

If I'm not mistaken put_change(changeset, :contact_email_hash, email) will call the equal?/2 which then calls the private function has_string/1 which contains the call to String.valid?, breaking with the current nil value

from cloak_ecto.