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

Error pages must be secure as well #267

Closed
travisspencer opened this issue Sep 7, 2017 · 4 comments · Fixed by #276
Closed

Error pages must be secure as well #267

travisspencer opened this issue Sep 7, 2017 · 4 comments · Fixed by #276

Comments

@travisspencer
Copy link

the Observatory has historically only looked at the page's contents when it returns a 200 code, not 400 or 404.

-- @april in #105

This behavior should be changed. The Observatory should report the security ranking of the Web site under test regardless of the HTTP status code returned. If the site returns an error, the HTTP response should not put the visitor or operator at risk.

Therefore, status codes >= 400 should be evaluated in the same manner as those returning a 200.

@april
Copy link
Contributor

april commented Sep 18, 2017

Just committed some code (e8f9944) to track status codes. Having the ability to notify users that the page wasn't 200 (OK) is first step towards making this thing work.

@april
Copy link
Contributor

april commented Sep 19, 2017

Further work done in: mozilla/http-observatory-website@41e526d.

@april
Copy link
Contributor

april commented Sep 20, 2017

Looks like things are working now:
https://observatory.mozilla.org/analyze.html?host=nordicapis.curity.io

@travisspencer
Copy link
Author

travisspencer commented Sep 20, 2017 via email

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 a pull request may close this issue.

2 participants