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

issue/910: Fix CORS warning for localhost #939

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

Conversation

oobayly
Copy link

@oobayly oobayly commented Sep 23, 2024

See #910

As per the issue, test the hostname of the for a localhost URL, by

  1. Domain - localhost
  2. IPv4 localhost subnet - 127.0.0.1/8
  3. IPv6 localhost - [::1]

Launch Checklist

  • Briefly describe the changes in this PR.
  • Link to related issues.
  • Include before/after visuals or gifs if this PR includes visual changes.
  • Write tests for all new functionality.
  • Add an entry to CHANGELOG.md under the ## main section.

@HarelM
Copy link
Collaborator

HarelM commented Sep 23, 2024

Can you add a test to make sure this will be kept solved in the future?

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