----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7272/#review12030 -----------------------------------------------------------
third_party/libprocess/include/process/http.hpp <https://reviews.apache.org/r/7272/#comment25649> s/encodeURIComponent/encodeURI ? third_party/libprocess/include/process/http.hpp <https://reviews.apache.org/r/7272/#comment25647> Rephrase? Returns error on the occurence of a malformed % in s. third_party/libprocess/include/process/http.hpp <https://reviews.apache.org/r/7272/#comment25650> s/decodeURIComponent/decodeURI/ ? third_party/libprocess/include/process/http.hpp <https://reviews.apache.org/r/7272/#comment25648> probably useful to also include the position 'i' in the error string. - Vinod Kone On Sept. 25, 2012, 11:01 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7272/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2012, 11:01 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > We currently don't decode percent-encoded queries! > > > Diffs > ----- > > src/tests/files_tests.cpp 17d284b82de4d9e1dedf1c26f4cedcf8269c4072 > third_party/libprocess/include/process/http.hpp > d680261f9a460627e0ea8c492b38c6dac83f3b56 > third_party/libprocess/src/decoder.hpp > f582faf985a17b1b333331da3582caa20b806ec0 > third_party/libprocess/src/tests.cpp > 1df7ae61f24643a0ceab146646e77da6909dfd5f > > Diff: https://reviews.apache.org/r/7272/diff/ > > > Testing > ------- > > Tests for encoding / decoding. > > > Thanks, > > Ben Mahler > >
