Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@KVermoteEFC
Copy link
Contributor

No description provided.

@github-actions
Copy link

Looks like you've passed all of the checks!

Copy link
Collaborator

@cfredric cfredric left a comment

Choose a reason for hiding this comment

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

LGTM

@cfredric cfredric merged commit 219d88d into GoogleChrome:main Mar 19, 2024
"https://carcostadvisor.be",
"https://carcostadvisor.fr"
]
}
Copy link

Choose a reason for hiding this comment

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

These seem to be connected to non authorized individuals can you please trace and forward to proper authorities.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi,
I'm not quite sure what you mean as I'm the administrator of these websites (which of course enabled me to add the .well-known/related-website-set.json files in the first place).

Maybe I should point out though that we don't actually use the apex domains carcostadvisor.be and carcostadvisor.fr - these simply redirect the user to carcostadvisor.com, which contains our general welcome page.
Our tenant users always use their designated subdomain, like bmw.carcostadvisor.be or renault.carcostadvisor.fr, etc.

I used to do the redirecting on the DNS-level, but as the RWS workflow expects the https://carcostadvisor.be/.well-known/related-website-set.json URL to work, I had to create 2 static websites for the apex domains, which contain only the "related-website-set.json" and an "index.html" that redirects the user to the carcostadvisor.com website as before.

Does this satisfy your comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI, the static websites are hosted in Azure Storage.

Copy link

@Labestia Labestia left a comment

Choose a reason for hiding this comment

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants