Skip to content

Conversation

@iakkov
Copy link
Owner

@iakkov iakkov commented Nov 8, 2025

Note

Adds a simple local CRUD frontend for users, enables CORS in the user-service, and updates docker-compose by adding Redis and tweaking Kafka deps.

  • Frontend:
    • CRUD UI: Add frontend/index.html and frontend/app.js with forms and table to list, get, create, update, and delete users via http://localhost:8080/users.
  • Backend:
    • CORS: Introduce CorsConfig to allow localhost origins, common methods, and headers.
  • DevOps:
    • Docker Compose: Add redis service with persisted volume; remove kafka depends_on condition; clean up duplicate Redis definition.

Written by Cursor Bugbot for commit f73e133. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Bug: Kafka startup fails: Dependency removed.

The depends_on configuration for kafka's dependency on zookeeper was removed. Kafka requires Zookeeper to be running before it starts, so removing this dependency can cause Kafka to fail during startup if it attempts to connect to Zookeeper before Zookeeper is ready.

docker-compose.yml#L37-L41

kafka:
image: confluentinc/cp-kafka:7.5.0
container_name: kafka
restart: unless-stopped

Fix in Cursor Fix in Web


@@ -0,0 +1,201 @@
const API_BASE_URL = 'http://localhost:8080/users';
Copy link

Choose a reason for hiding this comment

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

Bug: Backend Unreachable: Port Mismatch in Docker

The API_BASE_URL points to http://localhost:8080/users, but the user-service runs on port 8082 when using Docker (as configured in docker-compose.yml and application-docker.yml). The frontend won't be able to connect to the backend service, causing all API requests to fail.

Fix in Cursor Fix in Web

<td>${user.email}</td>
<td>${user.age}</td>
</tr>
`).join('');
Copy link

Choose a reason for hiding this comment

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

Bug: XSS: Unsanitized User Data Injection

User data (user.id, user.name, user.email, user.age) is directly interpolated into HTML via innerHTML without sanitization. If any user field contains malicious HTML or JavaScript, it will execute in the browser, creating an XSS vulnerability.

Fix in Cursor Fix in Web

@iakkov iakkov merged commit 3cc7aef into develop Nov 10, 2025
3 checks passed
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