You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2018/08/28 17:40:41 UTC

Re: Review Request 68420: Added /files API test for reserved query characters.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68420/
-----------------------------------------------------------

(Updated Aug. 28, 2018, 10:40 a.m.)


Review request for mesos and Benjamin Mahler.


Changes
-------

Updated with better explanation and a new test name and commit message.


Summary (updated)
-----------------

Added /files API test for reserved query characters.


Bugs: MESOS-9168
    https://issues.apache.org/jira/browse/MESOS-9168


Repository: mesos


Description (updated)
-------

Due to a bug in the HTTP client in libprocess (MESOS-9168), the
use of reserved query characters in paths did not work when using
the HTTP client in libprocess. This adds a test to ensure that
the /files API correctly handles reserved query characters in
file paths.


Diffs (updated)
-----

  src/tests/files_tests.cpp d09279077ec704167991933349ee943ab44d0aa9 


Diff: https://reviews.apache.org/r/68420/diff/2/

Changes: https://reviews.apache.org/r/68420/diff/1-2/


Testing
-------


Thanks,

Andrew Schwartzmeyer