Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(ArgumentError) argument error: Fields.Helpers.get_salt/1 #229

Closed
nelsonic opened this issue Oct 17, 2022 · 1 comment
Closed

(ArgumentError) argument error: Fields.Helpers.get_salt/1 #229

nelsonic opened this issue Oct 17, 2022 · 1 comment
Assignees
Labels
BLOCKED :fire: Core team's HIGHEST priority, blocking critical work chore a tedious but necessary task often paying technical debt help wanted If you can help make progress with this issue, please comment! priority-1 Highest priority issue. This is costing us money every minute that passes. tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written technical A technical issue that requires understanding of the code, infrastructure or dependencies

Comments

@nelsonic
Copy link
Member

While attempting to run the tests for auth (main branch) on my localhost,
after running source .env I attempted both mix test and the alias mix c
and got the following error:

 mix c
Generated auth app
The database for Auth.Repo has been dropped
** (ArgumentError) argument error
    (fields 2.8.2) lib/helpers.ex:19: Fields.Helpers.get_salt/1
    (fields 2.8.2) lib/helpers.ex:15: Fields.Helpers.hash/2
    (fields 2.8.2) lib/hash.ex:22: Fields.Hash.dump/1
    (ecto 3.7.2) lib/ecto/type.ex:915: Ecto.Type.process_dumpers/3
    (elixir 1.13.4) lib/enum.ex:2396: Enum."-reduce/3-lists^foldl/2-0-"/3

I think I know why this is happening ... but not 100%.
Hence opening this issue proactively before diving into solving it
so that:
a. I capture the steps I took to resolve it. ✍️
b. I can refer back to those steps if this happens again in the future 🔗
c. Google indexes this error message and corresponding solution in case anyone else in the community sees this error and needs the fix! 🔍

@nelsonic nelsonic added help wanted If you can help make progress with this issue, please comment! priority-1 Highest priority issue. This is costing us money every minute that passes. chore a tedious but necessary task often paying technical debt technical A technical issue that requires understanding of the code, infrastructure or dependencies BLOCKED :fire: Core team's HIGHEST priority, blocking critical work tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written labels Oct 17, 2022
@nelsonic nelsonic self-assigned this Oct 17, 2022
@nelsonic nelsonic added this to More ToDo ThanCanEver Be Done in Nelson's List via automation Oct 17, 2022
@nelsonic
Copy link
Member Author

Re-installing the dependencies in the auth project "fixed" the issue ... 🙄
But obviously I'd prefer a more robust/repeatable solution. 💭

New/next issue to fix coverage dip: #230

Nelson's List automation moved this from More ToDo ThanCanEver Be Done to Done Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLOCKED :fire: Core team's HIGHEST priority, blocking critical work chore a tedious but necessary task often paying technical debt help wanted If you can help make progress with this issue, please comment! priority-1 Highest priority issue. This is costing us money every minute that passes. tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written technical A technical issue that requires understanding of the code, infrastructure or dependencies
Projects
Nelson's List
  
Done
Development

No branches or pull requests

1 participant