Skip to content

Conversation

@jocelyneholdbrook
Copy link
Contributor

No description provided.

@tomolopolis
Copy link
Member

Copy link
Member

@tomolopolis tomolopolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pretty much there I think but a couple of small things

… Clear-text logging of sensitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Copy link
Collaborator

@alhendrickson alhendrickson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! lgtm

@jocelyneholdbrook jocelyneholdbrook merged commit 046754b into main Jan 20, 2026
10 checks passed
@jocelyneholdbrook jocelyneholdbrook deleted the CU-869bt7y6a_Add-passthrough-proxy-to-allow-routing-via-Traefik-and-enable-OIDC-auth branch January 20, 2026 15:14
tomolopolis pushed a commit that referenced this pull request Jan 22, 2026
…el roles and add separate client for backend auth (#298)

* refactor(medcat-trainer): Externalise VITE_ env vars, introduce separate client for backend auth for clarity, update API migrations, ensure roles can be read from top-level role group.

* doc(medcat-trainer): Provide more comprehensive OIDC auth guide

* doc(medcat-trainer): Update OIDC Auth guide and installation file

* refactor(medcat-trainer): Address code review feedback

* chore(medcat-trainer): Potential fix for code scanning alert no. 151: Clear-text logging of sensitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

* refactor(medcat-trainer): Address more code review feedback

---------

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants