Skip to content

PR: Docs #231

Open
nelsonic wants to merge 61 commits intomainfrom
gen-erd-#207
Open

PR: Docs #231
nelsonic wants to merge 61 commits intomainfrom
gen-erd-#207

Conversation

@nelsonic
Copy link
Member

@nelsonic nelsonic commented Oct 17, 2022

Busy adding some docs ... 👨‍💻 ⏳

@nelsonic nelsonic added enhancement New feature or enhancement of existing functionality chore a tedious but necessary task often paying technical debt in-progress An issue or pull request that is being worked on by the assigned person labels Oct 17, 2022
@nelsonic nelsonic self-assigned this Oct 17, 2022
@nelsonic nelsonic temporarily deployed to dwylauth October 17, 2022 06:05 Inactive
@codecov
Copy link

codecov bot commented Oct 17, 2022

Codecov Report

Merging #231 (2bbba99) into main (6415642) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main      #231    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           27        14    -13     
  Lines          586       190   -396     
==========================================
- Hits           586       190   -396     
Impacted Files Coverage Δ
lib/auth/accounts.ex 100.00% <100.00%> (ø)
lib/auth/accounts/person.ex 100.00% <100.00%> (ø)
lib/auth/accounts/person_notifier.ex 100.00% <100.00%> (ø)
lib/auth/accounts/person_token.ex 100.00% <100.00%> (ø)
lib/auth_web/controllers/error_html.ex 100.00% <100.00%> (ø)
lib/auth_web/controllers/error_json.ex 100.00% <100.00%> (ø)
lib/auth_web/controllers/page_controller.ex 100.00% <100.00%> (ø)
..._web/controllers/person_confirmation_controller.ex 100.00% <100.00%> (ø)
..._web/controllers/person_registration_controller.ex 100.00% <100.00%> (ø)
...eb/controllers/person_reset_password_controller.ex 100.00% <100.00%> (ø)
... and 29 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nelsonic nelsonic temporarily deployed to dwylauth October 18, 2022 10:10 Inactive
@nelsonic
Copy link
Member Author

Build failing due to:

** (ErlangError) Erlang error: {:badarg, {'aead.c', 90}, 'Unknown cipher'}
    (crypto 5.0.2) :crypto.aead_cipher_nif(:aes_256_gcm, "", 
<<164, 15, 193, 21, 34, 216, 157, 59, 173, 102, 0, 45, 69, 112, 115, 87>>, 
"admin@dwylauth.com", "AES256GCM", 16, true)
    (fields 2.8.2) lib/aes.ex:36: Fields.AES.encrypt/1

Looks like this is an issue with Fields not shipping the latest version of the AES cypher ...

https://github.com/dwyl/auth/actions/runs/3272492583/jobs/5383565807
image

BRB!

@nelsonic nelsonic temporarily deployed to dwylauth October 18, 2022 14:29 Inactive
@nelsonic
Copy link
Member Author

Oh oh ... looks like the auth Repository secrets were removed ... 🤷‍♂️

https://github.com/dwyl/auth/settings/secrets/actions

auth-repo-secrets-fail

Lame! 🤦‍♂️

Re-adding them now. ⏳

@nelsonic
Copy link
Member Author

Ref:

env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
MIX_ENV: test
AUTH_API_KEY: ${{ secrets.AUTH_API_KEY }}
AUTH_URL: auth.dwyl.com
ADMIN_EMAIL: admin@dwylauth.com
EMAIL_APP_URL: ${{ secrets.EMAIL_APP_URL }}
ENCRYPTION_KEYS: ${{ secrets.ENCRYPTION_KEYS }}
GOOGLE_CLIENT_ID: ${{ secrets.GOOGLE_CLIENT_ID }}
GOOGLE_CLIENT_SECRET: ${{ secrets.GOOGLE_CLIENT_SECRET }}
SECRET_KEY_BASE: ${{ secrets.SECRET_KEY_BASE }}

@nelsonic nelsonic temporarily deployed to dwylauth October 18, 2022 15:14 Inactive
@nelsonic
Copy link
Member Author

Re-added all the repository secrets ...

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore a tedious but necessary task often paying technical debt enhancement New feature or enhancement of existing functionality in-progress An issue or pull request that is being worked on by the assigned person 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

Status: 🏗 In progress

Development

Successfully merging this pull request may close these issues.

2 participants