Re: Review Request 42143: Removed HTTPTest.Auth test.

2016-01-20 Thread Till Toenshoff

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

Ship it!


Ship It!

- Till Toenshoff


On Jan. 13, 2016, 6:13 p.m., Alexander Rojas wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42143/
> ---
> 
> (Updated Jan. 13, 2016, 6:13 p.m.)
> 
> 
> Review request for mesos, Benjamin Bannier, Ben Mahler, Jan Schlicht, and 
> Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Manual HTTP authentication is being phased out in favor of the HTTP 
> authentication support built-in into libprocess, hence this test is no longer 
> necessary.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/src/tests/http_tests.cpp 
> e5999d8b49937a17033482c21536edb5c10420e6 
> 
> Diff: https://reviews.apache.org/r/42143/diff/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Alexander Rojas
> 
>



Re: Review Request 42143: Removed HTTPTest.Auth test.

2016-01-13 Thread Alexander Rojas

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

(Updated Jan. 13, 2016, 6:13 p.m.)


Review request for mesos, Benjamin Bannier, Ben Mahler, Jan Schlicht, and Till 
Toenshoff.


Changes
---

Fixed description.


Repository: mesos


Description (updated)
---

Manual HTTP authentication is being phased out in favor of the HTTP 
authentication support built-in into libprocess, hence this test is no longer 
necessary.


Diffs
-

  3rdparty/libprocess/src/tests/http_tests.cpp 
e5999d8b49937a17033482c21536edb5c10420e6 

Diff: https://reviews.apache.org/r/42143/diff/


Testing
---

make check


Thanks,

Alexander Rojas



Review Request 42143: Removed HTTPTest.Auth test.

2016-01-11 Thread Alexander Rojas

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

Review request for mesos, Benjamin Bannier, Ben Mahler, and Till Toenshoff.


Repository: mesos


Description
---

Manual HTTP authentication is being phased out in favor of authentication 
support built in libprocess, as such, this test is no longer necesary.


Diffs
-

  3rdparty/libprocess/src/tests/http_tests.cpp 
e5999d8b49937a17033482c21536edb5c10420e6 

Diff: https://reviews.apache.org/r/42143/diff/


Testing
---

make check


Thanks,

Alexander Rojas



Re: Review Request 42143: Removed HTTPTest.Auth test.

2016-01-11 Thread Jan Schlicht

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

Ship it!


Ship It!

- Jan Schlicht


On Jan. 11, 2016, 4:06 p.m., Alexander Rojas wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42143/
> ---
> 
> (Updated Jan. 11, 2016, 4:06 p.m.)
> 
> 
> Review request for mesos, Benjamin Bannier, Ben Mahler, Jan Schlicht, and 
> Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Manual HTTP authentication is being phased out in favor of authentication 
> support built in libprocess, as such, this test is no longer necesary.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/src/tests/http_tests.cpp 
> e5999d8b49937a17033482c21536edb5c10420e6 
> 
> Diff: https://reviews.apache.org/r/42143/diff/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Alexander Rojas
> 
>



Re: Review Request 42143: Removed HTTPTest.Auth test.

2016-01-11 Thread Benjamin Bannier

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

Ship it!


Ship It!

- Benjamin Bannier


On Jan. 11, 2016, 4:06 p.m., Alexander Rojas wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42143/
> ---
> 
> (Updated Jan. 11, 2016, 4:06 p.m.)
> 
> 
> Review request for mesos, Benjamin Bannier, Ben Mahler, Jan Schlicht, and 
> Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Manual HTTP authentication is being phased out in favor of authentication 
> support built in libprocess, as such, this test is no longer necesary.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/src/tests/http_tests.cpp 
> e5999d8b49937a17033482c21536edb5c10420e6 
> 
> Diff: https://reviews.apache.org/r/42143/diff/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Alexander Rojas
> 
>