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.

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

✨ Implement OIDC based auth with Keycloak #147

Merged
merged 17 commits into from
May 16, 2023
Merged

✨ Implement OIDC based auth with Keycloak #147

merged 17 commits into from
May 16, 2023

Conversation

znatty22
Copy link
Member

Closes #137

@znatty22 znatty22 added the feature New functionality label May 11, 2023
@znatty22 znatty22 self-assigned this May 11, 2023
@znatty22 znatty22 changed the base branch from master to major-upgrade-2023 May 11, 2023 18:01
@znatty22
👷 Add OIDC config to dockerfile
154f75b
@znatty22
🔇 Turn off audit logging until bug is fixed
70bbbf4 
If setup for OIDC and the issuer url is > 100 chars the db errors and
cannot create the entry in the audit log upon authentication

Submitted a ticket for it here: https://support.smilecdr.com/portal/11/SMILE-6737
@znatty22
📌 Pin all requirements to a version
eb7d443
@znatty22
⬆️ Add pytest to dev req
b944921
@znatty22
✅ Convert test.sh to pytest suite
198e766
@znatty22
🔥 Rm old test.sh
45859be
@znatty22
✨ Create real keycloak proxy endpoint for testing
a3d4af2
@znatty22
✅ Test OIDC auth
348cbcd
@znatty22
⚡ Force inline storage for resource JSON < 10K chars
92645cd 
See https://smilecdr.com/docs/fhir_storage_relational/resource_storage_mode.html#resource-body-storage
@znatty22
🐳 Set ulimit nofiles to 5000 per smilecdr prd checklist
ed445cc 
See https://smilecdr.com/docs/installation/production_checklist.html#host-server-setup
@znatty22
🔧 Increase timeout for intg test gh workflow
cb2a980
@znatty22
🐛 Change 10K to 4K to match db schema
d2577b5 
Docs say set dao_config.inline_resource_storage_below_size to 10K but
the db schema only allows for 4K
@znatty22 znatty22 marked this pull request as ready for review May 16, 2023 20:05
@znatty22 znatty22 merged commit 982aec0 into major-upgrade-2023 May 16, 2023
1 check passed
@znatty22 znatty22 deleted the oidc branch May 16, 2023 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@znatty22 znatty22

Labels
feature New functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement OIDC based authentication
1 participant
@znatty22