diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2017-01-07 23:14:37 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-01-07 23:14:37 -0800 |
commit | f958dafcaeed643f1232a23f5c2d4f6ba141bfea (patch) | |
tree | 04272dd512cf96ca0425ffee7c88b6f2936b4844 /tests/unit/net/lib.rs | |
parent | cdf14730ff4c17afdfdcc0cefbd0683a4324ba8e (diff) | |
parent | a3026499f43860c0d02170534bee20f8f5cc7faa (diff) | |
download | servo-f958dafcaeed643f1232a23f5c2d4f6ba141bfea.tar.gz servo-f958dafcaeed643f1232a23f5c2d4f6ba141bfea.zip |
Auto merge of #14865 - mrnayak:sri-fetch, r=jdm
Implement Subresource Integrity
Implemented response validation part of https://w3c.github.io/webappsec-subresource-integrity/.
Implemented step eighteen of the main fetch. If a request has integrity metadata, then following steps are performed
1) Wait for response body
2) If the response does not have a termination reason and response does not match request’s integrity metadata, set response and internalResponse to a network error.
Dependency updated: html5ever-atoms from 0.1.2 to 0.1.3. This will not completely fix #14523, It will implement changes related to response validation. Request validation algorithm implementation needs CSP.
I did not update any WPT-Test. In my local system, I found some assertion issue dependent on the order of execution of test-case. It would be helpful if someone could do "try" build on these changes to get wpt results.
r? @jdm
<!-- Please describe your changes on the following line: -->
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
<!-- Either: -->
- [X] There are tests for these changes
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/14865)
<!-- Reviewable:end -->
Diffstat (limited to 'tests/unit/net/lib.rs')
-rw-r--r-- | tests/unit/net/lib.rs | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/unit/net/lib.rs b/tests/unit/net/lib.rs index 990247e9ab2..47f732dc7b8 100644 --- a/tests/unit/net/lib.rs +++ b/tests/unit/net/lib.rs @@ -33,6 +33,7 @@ extern crate url; #[cfg(test)] mod hsts; #[cfg(test)] mod http_loader; #[cfg(test)] mod filemanager_thread; +#[cfg(test)] mod subresource_integrity; use devtools_traits::DevtoolsControlMsg; use hyper::server::{Handler, Listening, Server}; |