Re: Review Request 49401: Updated certificate validation to check 'IP Address' SAN.

2016-06-30 Thread Lukas Loesche

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




3rdparty/libprocess/src/libevent_ssl_socket.cpp (line 515)
<https://reviews.apache.org/r/49401/#comment205424>

Why does turning on support for IP Address verification turn off support 
for Hostname verification? Shouldn't it be an added functionality not a 
replacement? Or am I reading the code wrong?


- Lukas Loesche


On June 30, 2016, 12:19 a.m., Till Toenshoff wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49401/
> ---
> 
> (Updated June 30, 2016, 12:19 a.m.)
> 
> 
> Review request for mesos, Adam B, Albert Strasheim, Artem Harutyunyan, Joris 
> Van Remoortere, and Lukas Loesche.
> 
> 
> Bugs: MESOS-5724
> https://issues.apache.org/jira/browse/MESOS-5724
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Allows the verification of X509 certificates based on an IP address
> instead of a hostname. Introduces a new environment variable;
> `SSL_VERIFY_IPADD` which, when set to `true` will disable any
> attempts to reverse-/lookup the hostname for certificate validation.
> Instead the peer certificate verification then relies on the IP
> address of a connection.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/src/libevent_ssl_socket.hpp 1dbdaa8 
>   3rdparty/libprocess/src/libevent_ssl_socket.cpp 19d9ae5 
>   3rdparty/libprocess/src/openssl.hpp 7d55025 
>   3rdparty/libprocess/src/openssl.cpp 0f62aa6 
> 
> Diff: https://reviews.apache.org/r/49401/diff/
> 
> 
> Testing
> ---
> 
> make check on OSX and various linux distros.
> 
> 
> Thanks,
> 
> Till Toenshoff
> 
>



Re: Review Request 40748: Updated documentation to point out the need of a resolvable hostname.

2015-11-26 Thread Lukas Loesche

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

Ship it!


Ship It!

- Lukas Loesche


On Nov. 26, 2015, 1:08 p.m., Till Toenshoff wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40748/
> ---
> 
> (Updated Nov. 26, 2015, 1:08 p.m.)
> 
> 
> Review request for mesos, Bernd Mathiske, Lukas Loesche, and Timothy Chen.
> 
> 
> Bugs: MESOS-3937
> https://issues.apache.org/jira/browse/MESOS-3937
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> see summary.
> 
> 
> Diffs
> -
> 
>   docs/getting-started.md d120ad42d6e783048e6261750a9e7ad650d03669 
> 
> Diff: https://reviews.apache.org/r/40748/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Till Toenshoff
> 
>