Skip to content

DBC22-5426: token based auth for RabbitMQ connection#1313

Open
bcgov-brwang wants to merge 1 commit into
mainfrom
feature/DBC22-5426
Open

DBC22-5426: token based auth for RabbitMQ connection#1313
bcgov-brwang wants to merge 1 commit into
mainfrom
feature/DBC22-5426

Conversation

@bcgov-brwang
Copy link
Copy Markdown
Collaborator

@bcgov-brwang bcgov-brwang commented Apr 29, 2026

📝 Submitter

🔗 JIRA Ticket


✅ Quality Assurance & Requirements

  • Requirements Met: I have confirmed that all acceptance criteria from the JIRA ticket are fulfilled.
  • Tested desktop in local or dev envs
  • Tested mobile in local or dev envs
  • Ran unit tests locally
  • SonarCloud: I have verified that the SonarCloud analysis is clean/passing for this branch.

⚙️ Configuration & Environment

  • New Env Variables: Does this PR require new environment variables? (Yes/No)

    If yes, please list them here and ensure they are added to secret manager, the .env.example. and the Vault by an STA.

🧪 How to Test (if required)

  1. Add following env variables:
    RABBITMQ_HOST=142.34.229.61
    RABBITMQ_PORT=5064
    RABBITMQ_VHOST=vh-drivebc-dev-test
    OAUTH2_SCOPE=rabbitmq:read rabbitmq:write
    OAUTH2_TOKEN_URL=https://dev.loginproxy.gov.bc.ca/auth/realms/moti-custom/protocol/openid-connect/token
    OAUTH2_CLIENT_ID=rabbitmq-auth
    OAUTH2_DRIVEBC_RABBITMQ_USERNAME=dev-drivebc-consumer
    OAUTH2_DRIVEBC_RABBITMQ_PASSWORD=xxxxxx

  2. Verify consumer can connect to RabbitMQ correctly


🔍 Reviewer Checklist

  • Reviewed code for logic and cleanliness
  • Re-tested desktop/mobile in local or dev envs
  • Verified no new console warnings/errors
  • Confirmed that any new env variables are understood/documented

Comment thread src/backend/apps/consumer/token.py Fixed
@bcgov-brwang bcgov-brwang force-pushed the feature/DBC22-5426 branch 2 times, most recently from fde0a75 to d0ab482 Compare April 30, 2026 21:07
@bcgov-brwang bcgov-brwang changed the title DBC22-5426: token based auth DBC22-5426: token based auth for RabbitMQ connection Apr 30, 2026
@bcgov-brwang bcgov-brwang requested a review from wmuldergov April 30, 2026 21:20
@bcgov-brwang bcgov-brwang marked this pull request as ready for review April 30, 2026 21:20
@sonarqubecloud
Copy link
Copy Markdown

@bcgov-brwang bcgov-brwang force-pushed the feature/DBC22-5426 branch from 0c2d248 to 6122e6c Compare May 7, 2026 22:14
DBC22-5426: got token based auth working

DBC22-5426: added test cases to cover new code

DBC22-5426: fixed sonarcloud issues

DBC22-5426: fixed left sonarcloud issue

DBC22-5426: fixed left sonarcloud issue 2

DBC22-5426: removed auth client secret when connect

DBC22-5426: rebased from main

DBC22-5426: token based auth

DBC22-5426: got token based auth working

DBC22-5426: added test cases to cover new code

DBC22-5426: fixed sonarcloud issues

DBC22-5426: fixed left sonarcloud issue

DBC22-5426: fixed left sonarcloud issue 2

DBC22-5426: removed auth client secret when connect

DBC22-5426: added missing functions
@bcgov-brwang bcgov-brwang force-pushed the feature/DBC22-5426 branch from f686b7f to a0a5f09 Compare May 7, 2026 22:17
@wmuldergov wmuldergov added the ready for review The PR is ready for review label May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review The PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants