Skip to content

Conversation

@technologic-technologic
Copy link
Owner

Change cross origins to allow communication with client..
Description belo

- Allows cross-origin requests from the new frontend port.
@github-actions
Copy link

PR Summary

  • WHAT Changed:

    • Updated the @CrossOrigin annotation in InventoryManagerController.java to change the allowed origin from http://localhost:3000 to http://localhost:8080.
  • WHY It Matters:

    • This change allows the frontend application running on port 8080 to access the backend API, which is essential for proper integration and functionality during development.
  • RISKS:

    • If the frontend is not configured to run on port 8080, it may lead to CORS issues, preventing the application from functioning correctly.
    • Existing clients running on port 3000 will no longer be able to access the API unless they are updated.
  • TESTS to Add:

    • Test the CORS configuration to ensure that requests from http://localhost:8080 are accepted.
    • Verify that requests from http://localhost:3000 are blocked as expected.
  • BREAKING CHANGES:

    • The change in the allowed origin may break existing integrations that rely on the previous configuration (port 3000).
  • Changed File Paths:

    • src/main/java/com/encorazone/inventory_manager/controller/InventoryManagerController.java

@technologic-technologic technologic-technologic added the bug Something isn't working label Oct 13, 2025
@technologic-technologic technologic-technologic linked an issue Oct 13, 2025 that may be closed by this pull request
@technologic-technologic technologic-technologic added this to the Major fixes milestone Oct 13, 2025
@technologic-technologic technologic-technologic merged commit 948c965 into gen-ai Oct 13, 2025
1 check passed
@technologic-technologic technologic-technologic deleted the 21-allow-client-on-port-8080 branch October 13, 2025 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update CORS to allow client on port 8080 Change client default port to 8080

2 participants