Skip to content

Advance Authentication #43

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 36 commits into from
Sep 13, 2024
Merged

Advance Authentication #43

merged 36 commits into from
Sep 13, 2024

Conversation

@gierens
Copy link
Member

@gierens gierens commented Sep 13, 2024

Implement additional middleware for user and project extraction from database. Also improve test suite to setup user and project.

@gierens
feat(api/auth): rename project to os_project
7adc1a0 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(api/hello): rename project to os_project
9f52ccd 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(api/auth): start extract_user_and_project middleware function
f1dd76d 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(api/startup): apply extract_user_and_project to secured routes
45dfc09 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(api/migrations): revise project table and add todo comments
a964079 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(scripts): add name to db container in init_db.sh
b0a2b18 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(scripts): add remove_db.sh
75ed277 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(scripts): rename mariadb.sh to enter_db.sh
bc14adb 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
fix(scripts): correct container naming in init_db.sh
3f41df4 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(api/migrations): add user_project and user_user table via prod q…
dee25ba 
…ueries

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/hello): revise table name in database_insert_works query
a3890b3 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(scripts): add recreate_db.sh
1206ace 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(api/auth): implement extract_user_and_project
71c7df6 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
chore(vim): create .vim/coc-settings.json with cspell dictionary
d5a5779 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(husky): add cargo sqlx check to pre-push hook
af2ab7c 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
chore(sqlx): update offline query data
c9ab318 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/health_check): add user/project to db in secured_health_chec…
b239bf8 
…k_works_with_valid_token

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
fix(api/auth): correct where in extract_user_and_project query
bd9ed19 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/helpers): rename project variables in TestApp.mock_keystone_…
948d1c1 
…auth

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/helpers): add insert_user_into_db and insert_project_into_db
53c45a7 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/health_check): revise secured_health_check_works_with_valid_…
9729551 
…token with insert helpers

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/helpers): add random_uuid and random_alphanumeric_string
d411747 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/hello): replace Uuid::new by random_uuid
b65b2ae 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/health_check): replace Uuid::new with random_uuid
06609cd 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/helpers): make random_uuid public
708b437 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/helpers): add TestApp.setup_test_user_and_project
a25bc2e 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
feat(api/cargo): add dev-dependency rand
071f17a 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/health_check): revise secured_health_check_works_with_valid_…
c2874a7 
…token with TestApp.setup_test_user_and_project

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/helpers): add random_number function
4a0c6da 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
test(api/helpers): remove hardcoded ids in insert_user/project_into_db
da4e45c 
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@github-actions github-actions bot added the ci label Sep 13, 2024
@gierens gierens merged commit 6d9e162 into main Sep 13, 2024
11 checks passed
@gierens gierens deleted the advance-authentication branch September 13, 2024 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

api ci db dep sh src test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@gierens