Skip to content

Conversation

@jrobertboos
Copy link
Contributor

Description

This update introduces the CORSMiddleware to the FastAPI application, allowing cross-origin requests from any origin. The middleware is configured to accept all methods and headers, enhancing the application's flexibility in handling requests from different clients.

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

Related Tickets & Documents

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

This update introduces the CORSMiddleware to the FastAPI application, allowing cross-origin requests from any origin. The middleware is configured to accept all methods and headers, enhancing the application's flexibility in handling requests from different clients.
@jrobertboos jrobertboos marked this pull request as ready for review July 8, 2025 16:13
@jrobertboos
Copy link
Contributor Author

If you could review this @tisnik that would be great!

Copy link
Contributor

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

lgtm, thank you

@tisnik tisnik merged commit b2f72b2 into lightspeed-core:main Jul 8, 2025
16 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