Skip to content

Fix/sandbox env and data parameter#814

Merged
coodos merged 2 commits into
mainfrom
fix/sandbox-env-and-data-parameter
Feb 18, 2026
Merged

Fix/sandbox env and data parameter#814
coodos merged 2 commits into
mainfrom
fix/sandbox-env-and-data-parameter

Conversation

@Bekiboo

@Bekiboo Bekiboo commented Feb 18, 2026

Copy link
Copy Markdown
Collaborator

Description of change

  • Fixed dev-sandbox to decode and sign the actual vote data from the data parameter instead of just the sessionId, allowing evoting votes to be properly verified and submitted.
  • Fixed registry .env path from .env to .env (correct relative path)
  • Updated evoting API port to use EVOTING_API_PORT env variable with fallback to 4001

Issue Number

Type of change

  • Fix (a change which fixes an issue)

How the change has been tested

Change checklist

  • I have ensured that the CI Checks pass locally
  • I have removed any unnecessary logic
  • My code is well documented
  • I have signed my commits
  • My code follows the pattern of the application
  • I have self reviewed my code

Summary by CodeRabbit

  • New Features

    • Added support for decoding optional data parameters from authentication and signing redirects, enabling flexible message handling in the authentication flow.
  • Infrastructure & Configuration

    • Added evoting database capability to the system infrastructure.
    • Updated port configuration priority for the e-voting service.
    • Updated environment configuration file path for the registry service.

@Bekiboo Bekiboo self-assigned this Feb 18, 2026
@Bekiboo Bekiboo requested a review from coodos as a code owner February 18, 2026 07:25
@coderabbitai

coderabbitai Bot commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

These changes enable the evoting platform across multiple system layers: adding evoting as a new database in Docker Compose, configuring the evoting API to listen on the appropriate port, fixing an environment variable path in the registry API, and extending the dev-sandbox authentication/signing interface to optionally decode and use custom data from redirects instead of session IDs.

Changes

Cohort / File(s) Summary
Database & Platform Setup
docker-compose.databases.yml, platforms/evoting/api/src/index.ts
Added evoting database to Postgres configuration; updated evoting API port selection to prioritize EVOTING_API_PORT environment variable with default fallback to 4001.
Dev Sandbox Authentication Flow
infrastructure/dev-sandbox/src/routes/+page.svelte
Extended auth and sign redirect handling to optionally decode a base64-encoded data parameter; if present, it replaces sessionId in payload and message fields sent to the server.
Registry API Configuration
platforms/registry/api/src/index.ts
Corrected .env file path resolution from relative path with spacing issue to correct relative path.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • fix: sandbox scripts #803: Modifies POSTGRES_MULTIPLE_DATABASES in docker-compose.databases.yml to include additional databases, directly related to the database setup changes here.

Suggested reviewers

  • coodos

Poem

🐰 A new evoting garden grows,
With databases and APIs in rows,
Data decoded, redirects flow,
The sandbox learns what auth should know! 🌱

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/sandbox-env-and-data-parameter

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coodos coodos merged commit e8c7503 into main Feb 18, 2026
4 of 5 checks passed
@coodos coodos deleted the fix/sandbox-env-and-data-parameter branch February 18, 2026 07:28
@coderabbitai coderabbitai Bot mentioned this pull request Feb 26, 2026
6 tasks
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.

2 participants