-
Notifications
You must be signed in to change notification settings - Fork 91
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
Fix wrong error message on Removal page #142
base: main
Are you sure you want to change the base?
Conversation
The "Status" message on https://hstspreload.org/removal/ page is confusing, e.g. "Status: wikipedia.org is currently preloaded, but no longer meets the requirements. It may be at risk of removal." This message is clearly for the preload page rather than the removal page.
@ericlaw1979 Could you please review? |
It seems like the error message should only potentially omit "It may be at risk of removal." in this scenario, but showing the other status seems fine? |
@ericlaw1979 I think just showing "Status: xxx is currently preloaded" is fine. The code mistakenly treats the error of removal as error of preloading. "But has the following issues" and "but no longer meets the requirements. It may be at risk of removal" make it look like there's an error that may cause the domain to be removed from the preload list, but it's actually fine: the error ("Contains preload directive") prevents the removal, so it's not at risk of removal. |
The domain is at risk of removal (ignoring #106 in the case of wikipedia.org) – the front page of hstspreload.org clearly states this:
However, we could certainly do a better job of making clear that it can't be removed in its current state through the web form. |
For what it's worth, I would prefer a fix that clarifies that the domain may automatically be removed in the future, but the current state of the PR looks good to me, too. |
Actually it's not at risk of removal. Wikipedia.org sends |
The "Status" message on https://hstspreload.org/removal/ page is confusing, e.g. "Status: wikipedia.org is currently preloaded, but no longer meets the requirements. It may be at risk of removal."
This message is clearly for the preload page rather than the removal page.