Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CORS handler #144

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add CORS handler #144

wants to merge 3 commits into from

Conversation

a-luz
Copy link
Contributor

@a-luz a-luz commented Nov 4, 2024

Purpose

Cors handler added for routes if enabled

Related Issues

Closes TRI-###

Submission Checklist

  • Commits include the JIRA issue and the #review hashtag (e.g. TRI-### #review <comment>)
  • Code passes linter rules (clj-kondo --lint src)

Testing

  1. On an example project, create a routing table
  2. Add the key called :cors-headers to config.edn
  3. Add a route that has a key :cors set to true in the routing table
  4. Run the example app and make a request to that API route. check that the headers contain the CORS headers

Screenshots

@a-luz a-luz closed this Nov 4, 2024
@a-luz a-luz reopened this Dec 2, 2024
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.

1 participant