-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44678/
-----------------------------------------------------------
(Updated March 14, 2016, 6:01 p.m.)
Review request for mesos, Adam B, Alexander Rojas, Joerg Schad, and Till
Toenshoff.
Changes
-------
Addressed comments.
Bugs: MESOS-4850
https://issues.apache.org/jira/browse/MESOS-4850
Repository: mesos
Description
-------
Modified basic HTTP authenticator creator to accept realm.
To accommodate different authentication realms for the master and agent, the
default basic HTTP authenticator needs to accept its authentication realm as a
parameter. This patch adds this parameter and modifies the HTTP authentication
tests to validate it appropriately.
Diffs (updated)
-----
include/mesos/authentication/http/basic_authenticator_factory.hpp
c11bb47c8e02f2e8645cf387d18eb64d1c8cb604
src/authentication/http/basic_authenticator_factory.cpp
62f851685db3b42c52bbcb7cff3e4f4703004ed7
src/master/master.cpp d0380db3b90a9166607445f8dd50cc63d547228e
src/tests/http_authentication_tests.cpp
cf2bb762272fa38e04e5c26aef2858300bbd0459
Diff: https://reviews.apache.org/r/44678/diff/
Testing
-------
`make check` was used to test on both OSX and CentOS 7.
Thanks,
Greg Mann