Skip to content

Support CORS policy for requests#8

Open
arrake0101 wants to merge 1 commit into
hacash:mainfrom
arrake0101:main
Open

Support CORS policy for requests#8
arrake0101 wants to merge 1 commit into
hacash:mainfrom
arrake0101:main

Conversation

@arrake0101

@arrake0101 arrake0101 commented Nov 2, 2024

Copy link
Copy Markdown

Because websites need to be separated from the front end, cross-domain access requests are required.
After the merge, pelease redistribute a version and redeploy it.
Thank you.
image

@arrake0101

arrake0101 commented Nov 3, 2024

Copy link
Copy Markdown
Author

I have developed hacd.fun, welcome to visit.

@jojoin

jojoin commented Nov 3, 2024

Copy link
Copy Markdown
Member

There are multiple considerations here:

  1. Adding a generic Header will increase the load on network requests, especially if Hacash is used as a large-scale data service in the future.
  2. some third-party services may not want their back-end data to be called by other front-ends (e.g. to save on traffic charges).
  3. If you need your own node service to be cross-domain, you can add any desired Header via the Nginx proxy.
    The preference here is to keep the CORS policy unadded, is there more to consider?

@arrake0101

Copy link
Copy Markdown
Author

I think proxy requests consume extra traffic, which is unnecessary.
Later, I consider loading this item through the configuration file.

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