Skip to content

Accounts management, authentication and authorization#8

Merged
armanddidierjean merged 44 commits intomainfrom
account-creation
Apr 25, 2022
Merged

Accounts management, authentication and authorization#8
armanddidierjean merged 44 commits intomainfrom
account-creation

Conversation

@armanddidierjean
Copy link
Member

@armanddidierjean armanddidierjean commented Apr 10, 2022 •
edited
Loading

Account management

  • Create, recover, update users
  • Email confirmation

Group management

  • Create, modify
  • Default groups and account types

Security

  • Bcrypt hash and verification

Utils

  • SMTP email sender

Authorization

  • OAuth password flow using JWT
  • Groups dependencies using JWT

Tests

  • Async and database support

Tests can now be used and created

Workflow

  • Python basic typechecking
  • Add a .env dotfile

TODO

Use an email and an UUID type
Change cruds parameters and return types
Add loggin

@armanddidierjean armanddidierjean force-pushed the account-creation branch 2 times, most recently from 4b87583 to 09d288a Compare April 11, 2022 18:05
@armanddidierjean armanddidierjean changed the base branch from Database-migration to main April 11, 2022 18:06
@Jake-Ransom Jake-Ransom force-pushed the account-creation branch 2 times, most recently from d0e3d3e to 83610f5 Compare April 14, 2022 09:48
@armanddidierjean armanddidierjean force-pushed the account-creation branch 2 times, most recently from 1db6e96 to 2c1c08e Compare April 15, 2022 19:55
@armanddidierjean armanddidierjean marked this pull request as ready for review April 15, 2022 20:04
Victor-Angot
Victor-Angot previously approved these changes Apr 15, 2022
View reviewed changes
@armanddidierjean armanddidierjean force-pushed the account-creation branch 2 times, most recently from a1d7119 to f59d939 Compare April 19, 2022 13:13
@armanddidierjean armanddidierjean changed the title Account creation and password reset process Accounts management, authentication and authorization Apr 22, 2022
Victor-Angot
Victor-Angot previously approved these changes Apr 22, 2022
View reviewed changes
Jake-Ransom and others added 22 commits April 25, 2022 16:41
@Jake-Ransom @armanddidierjean
Account type in English
fa37f6c 
fix some exceptions and types
@Jake-Ransom @armanddidierjean
Add user to the group corresponding to his account_type when he is co…
339fef8 
…nfirmed

Add needed "account_type's groups" to database on first run
crud groups: get_group_by_name
@Jake-Ransom @armanddidierjean
Add Patch groups to update a group
7c1faf0
@Jake-Ransom @armanddidierjean
Define account_type by their group_id so that the name can be change …
e983aac 
…without consequences

Make sure that account_type's groups can't be deleted because it would break everything
@Jake-Ransom @armanddidierjean
Choose user_id over id_user so that it the same in all the code
35c4e5d
@armanddidierjean
[VS Code] Set python.analysis.typeCheckingMode to strict
4b40f00
@armanddidierjean
Fix types and names
7609c11
@armanddidierjean
Add password validation to change-password endpoint
9eae98f
@armanddidierjean
Add password validation to reset-password endpoint
2b0db7b
@armanddidierjean
Set python typeCheckingMode to basic
b6d4b37
@armanddidierjean
Cleaning
26062e3
@armanddidierjean
Add first examples to schemas_core
dc2609d
@armanddidierjean
Format
278f2b1
@armanddidierjean
Remove tests
67273e6 
And create a commons file
@Jake-Ransom @armanddidierjean
Basic configuration for tests
0bd3f49
@Jake-Ransom @armanddidierjean
Examples for the doc and the tests
501930c
@armanddidierjean
[Security] Add password hashing
a89140f
@armanddidierjean
[Auth] Add oauth and JWT support
91131b4
@armanddidierjean
Add groupTypes to complet accountTypes
5af3a90
@armanddidierjean
[Authorization] Add a dependency to check if a user is in a group
f5fd4c0
@armanddidierjean
Fix recover endpoint: embed email parameter
67d3900
@armanddidierjean
Add type annotation
247a563
@armanddidierjean armanddidierjean merged commit 5c84e5e into main Apr 25, 2022
@armanddidierjean armanddidierjean mentioned this pull request May 8, 2022
Closed
3 tasks
Rotheem pushed a commit that referenced this pull request Aug 26, 2025
@armanddidierjean
Fix(state management): store state in global variable to allow the sc…
de8bbdb 
…heduler to access db (#8)

* Fix user batch invitation response model

* Fix: add missing param in send_emails_from_queue_task

* Get db directly using SessionLocal

* Store state in global Python variable

* Use arq 0.26.3

* Don't keep arq job results after completion

to be able to queue new jobs with the same id

* Cancel planned notification with the same job_id

before queuing a new one

* fixup state

* Access GLOBAL_STATE in tests init

* Remove unexpected state param while disconnecting

* Lint

* Parametrize test_factory fixture

* Lint

* Refactor test settings

* Lint
armanddidierjean added a commit that referenced this pull request Aug 30, 2025
@armanddidierjean
Fix(state management): store state in global variable to allow the sc…
c725395 
…heduler to access db (#8)

* Fix user batch invitation response model

* Fix: add missing param in send_emails_from_queue_task

* Get db directly using SessionLocal

* Store state in global Python variable

* Use arq 0.26.3

* Don't keep arq job results after completion

to be able to queue new jobs with the same id

* Cancel planned notification with the same job_id

before queuing a new one

* fixup state

* Access GLOBAL_STATE in tests init

* Remove unexpected state param while disconnecting

* Lint

* Parametrize test_factory fixture

* Lint

* Refactor test settings

* Lint
# Conflicts:
#	app/core/users/endpoints_users.py
#	app/types/scheduler.py
#	tests/test_myeclpay.py
#	tests/test_payment.py
armanddidierjean added a commit that referenced this pull request Sep 2, 2025
@armanddidierjean
Fix(state management): store state in global variable to allow the sc…
67169df 
…heduler to access db (#8)

* Fix user batch invitation response model

* Fix: add missing param in send_emails_from_queue_task

* Get db directly using SessionLocal

* Store state in global Python variable

* Use arq 0.26.3

* Don't keep arq job results after completion

to be able to queue new jobs with the same id

* Cancel planned notification with the same job_id

before queuing a new one

* fixup state

* Access GLOBAL_STATE in tests init

* Remove unexpected state param while disconnecting

* Lint

* Parametrize test_factory fixture

* Lint

* Refactor test settings

* Lint
# Conflicts:
#	app/core/users/endpoints_users.py
#	app/types/scheduler.py
#	tests/test_myeclpay.py
#	tests/test_payment.py
armanddidierjean added a commit that referenced this pull request Sep 2, 2025
@armanddidierjean
Fix(state management): store state in global variable to allow the sc…
9ebbab1 
…heduler to access db (#8)

* Fix user batch invitation response model

* Fix: add missing param in send_emails_from_queue_task

* Get db directly using SessionLocal

* Store state in global Python variable

* Use arq 0.26.3

* Don't keep arq job results after completion

to be able to queue new jobs with the same id

* Cancel planned notification with the same job_id

before queuing a new one

* fixup state

* Access GLOBAL_STATE in tests init

* Remove unexpected state param while disconnecting

* Lint

* Parametrize test_factory fixture

* Lint

* Refactor test settings

* Lint
# Conflicts:
#	app/core/users/endpoints_users.py
#	app/types/scheduler.py
#	tests/test_myeclpay.py
#	tests/test_payment.py
armanddidierjean added a commit that referenced this pull request Sep 3, 2025
@armanddidierjean
Fix(state management): store state in global variable to allow the sc…
007ea34 
…heduler to access db (#8)

* Fix user batch invitation response model

* Fix: add missing param in send_emails_from_queue_task

* Get db directly using SessionLocal

* Store state in global Python variable

* Use arq 0.26.3

* Don't keep arq job results after completion

to be able to queue new jobs with the same id

* Cancel planned notification with the same job_id

before queuing a new one

* fixup state

* Access GLOBAL_STATE in tests init

* Remove unexpected state param while disconnecting

* Lint

* Parametrize test_factory fixture

* Lint

* Refactor test settings

* Lint
# Conflicts:
#	app/core/users/endpoints_users.py
#	app/types/scheduler.py
#	tests/test_myeclpay.py
#	tests/test_payment.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AtomHare AtomHare Awaiting requested review from AtomHare

1 more reviewer

@Victor-Angot Victor-Angot Victor-Angot left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

core

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@armanddidierjean @AtomHare @Victor-Angot @Nathanael-lscx @Jake-Ransom