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

[import] Proxy host cookie can lead to hit the wrong host #169

Open
kptdobe opened this issue May 3, 2023 · 1 comment
Open

[import] Proxy host cookie can lead to hit the wrong host #169

kptdobe opened this issue May 3, 2023 · 1 comment

Comments

@kptdobe
Copy link
Contributor

kptdobe commented May 3, 2023

Let's say we import https://www.a.com/page.html page. The requested page via the proxy is: http://localhost:3001/page.html?host=https://www.a.com.
This will set the hlx-proxyhost cookie value to https://www.a.com so that resources referenced on the https://www.a.com/page.html page (like images, js, css...) will not need the host query parameter (if their url do not contain the host name). For example, an image with src="/image.png" will be served by the proxy from https://www.a.com without requiring the host query param because the cookie contains the host.

Now if during the import process we need some resources from another host, something like a json file https://www.b.com/sheet.json. We can leverage the proxy and request http://localhost:3001/sheet.json?host=https://www.b.com which will prevent the CORS issues.
But... this request will re-set the hlx-proxyhost cookie value to https://www.b.com and then corrupt the import process: the subsequent requests without the host query param will try to fetch from b.com and not a.com like described above.

To solve that problem, we need to change how the cookie is set or consumed:

  • we want the cookie to be set to a.com so that we do not have to re-write all resources url / href / src to append the host query parameter
  • if we need to hit a different host, we can specify the host query param to get around the cookie value but NOT override the cookie value

As soon as we start dealing with different hosts, I think we need adobe/helix-cli#2072.

cc @mhaack

@mhaack
Copy link
Contributor

mhaack commented May 4, 2023

Can we set the cookie based on the URL of the imported page itself at the time you click import? With that it should be safe to not run into any race conditions.

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

No branches or pull requests

2 participants