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

PR: Docs #231

Open
wants to merge 61 commits into
base: main
Choose a base branch
from
Open

PR: Docs #231

wants to merge 61 commits into from

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>>, 
"[email protected]", "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: [email protected]
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
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