Bug 122936 - Add a new flakiness dashboard clone
Summary: Add a new flakiness dashboard clone
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Website (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-16 19:34 PDT by Ryosuke Niwa
Modified: 2013-10-17 12:22 PDT (History)
6 users (show)

See Also:


Attachments
Adds the app (47.75 KB, patch)
2013-10-16 19:44 PDT, Ryosuke Niwa
no flags Details | Formatted Diff | Diff
Removed defer attributes (47.73 KB, patch)
2013-10-16 19:49 PDT, Ryosuke Niwa
andersca: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2013-10-16 19:34:32 PDT
Add a new flakiness dashboard clone
Comment 1 Ryosuke Niwa 2013-10-16 19:44:12 PDT
Created attachment 214416 [details]
Adds the app
Comment 2 Ryosuke Niwa 2013-10-16 19:49:47 PDT
Created attachment 214418 [details]
Removed defer attributes
Comment 3 Ryosuke Niwa 2013-10-16 19:52:00 PDT
My plan is to check this initial version in and then iterate over time instead of trying to address all use cases, etc...

We can always delete it if we later find that the existing flamines dashboard is better.
Comment 4 Anders Carlsson 2013-10-16 19:53:57 PDT
Comment on attachment 214418 [details]
Removed defer attributes

View in context: https://bugs.webkit.org/attachment.cgi?id=214418&action=review

> Websites/test-results/index.html:23
> +    var urls = removeJunkFromResultsPage(removeJunksFromNRWTStdout(rawInput)).split('\n');

Junks should probably be Junk here.
Comment 5 Ryosuke Niwa 2013-10-16 20:09:07 PDT
Committed r157555: <http://trac.webkit.org/changeset/157555>
Comment 6 Alexey Proskuryakov 2013-10-16 22:48:46 PDT
Is this already deployed on one of webkit.org servers? I have some security concerns that may limit where this can be deployed.
Comment 7 Ryosuke Niwa 2013-10-17 00:02:31 PDT
(In reply to comment #6)
> Is this already deployed on one of webkit.org servers? I have some security concerns that may limit where this can be deployed.

No. What are your security concerns?
Comment 8 Alexey Proskuryakov 2013-10-17 08:49:19 PDT
Flakiness dashboard is particularly vulnerable because it's a persistent database that any attacker can write arbitrary data to. While there is some validation in report.php, I think that it's best to have multiple layers or protection, ideally by isolating it to its own domain.
Comment 9 Ryosuke Niwa 2013-10-17 12:22:38 PDT
(In reply to comment #8)
> Flakiness dashboard is particularly vulnerable because it's a persistent database that any attacker can write arbitrary data to.

With that token, the current flakiness dashboard also can also store arbitrary data any attacker sends.

> While there is some validation in report.php, I think that it's best to have multiple layers or protection, ideally by isolating it to its own domain.

Indeed. We'll probably host it at test-results.webkit.org.