Digital_Voice_Modem_Project
/
dvmhost
mirror of https://github.com/DVMProject/dvmhost.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

deprecate Peer ACL blacklist mode; (#91)

Browse Source
4.31h_maint
Bryan Biedenkapp 10 months ago committed by GitHub
parent baded95e1f
commit 5010fda595
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
10 changed files with 21 additions and 139 deletions
Show Stats Download Patch File Download Diff File

8
configs/fne-config.example.yml
Unescape View File

@ -253,16 +253,14 @@ system:
time: 2
#
# Peer whitelist and blacklist configuration
# Peer ACL configuration
#
peer_acl:
# Flag indicating whether or not the peer ACLs are enabled.
enable: false
# Peer ACL mode: whitelist or blacklist
mode: whitelist
# Full path to the white/blacklist file.
# Full path to the peer ACL file.
file: peer_list.dat
# Amount of time between updates of white/blacklist file. (minutes)
# Amount of time between updates of peer ACL file. (minutes)
time: 2
#

2
src/common/lookups/LookupTable.h
Unescape View File

@ -48,7 +48,7 @@ namespace lookups
/**
* @brief Initializes a new instance of the LookupTable class.
* @param filename Full-path to the lookup table file.
* @param reloadTime Interval of time to reload the channel identity table.
* @param reloadTime Interval of time to reload the lookup table.
*/
LookupTable(const std::string& filename, uint32_t reloadTime) :
Thread(),

31
src/common/lookups/PeerListLookup.cpp
Unescape View File

@ -50,8 +50,8 @@ bool PeerListLookup::m_locked = false;
/* Initializes a new instance of the PeerListLookup class. */
PeerListLookup::PeerListLookup(const std::string& filename, Mode mode, uint32_t reloadTime, bool peerAcl) : LookupTable(filename, reloadTime),
m_acl(peerAcl), m_mode(mode)
PeerListLookup::PeerListLookup(const std::string& filename, uint32_t reloadTime, bool peerAcl) : LookupTable(filename, reloadTime),
m_acl(peerAcl)
{
/* stub */
}
@ -158,32 +158,7 @@ bool PeerListLookup::isPeerAllowed(uint32_t id) const
return true; // if not enabled, allow all peers
}
bool allowed = false;
if (m_mode == WHITELIST) {
allowed = isPeerInList(id);
} else if (m_mode == BLACKLIST) {
allowed = !isPeerInList(id);
}
return allowed;
}
/* Sets the mode to either WHITELIST or BLACKLIST. */
void PeerListLookup::setMode(Mode mode)
{
__LOCK_TABLE();
m_mode = mode;
__UNLOCK_TABLE();
}
/* Gets the current mode. */
PeerListLookup::Mode PeerListLookup::getMode() const
{
return m_mode;
return isPeerInList(id);
}
/* Gets the entire peer ID table. */

27
src/common/lookups/PeerListLookup.h
Unescape View File

@ -152,21 +152,13 @@ namespace lookups
*/
class HOST_SW_API PeerListLookup : public LookupTable<PeerId> {
public:
/**
* @brief Peer List Mode
*/
enum Mode {
WHITELIST, //! Peers listed are whitelisted
BLACKLIST //! Peers listed are blacklisted
};
/**
* @brief Initializes a new instance of the PeerListLookup class.
* @param filename Full-path to the list file.
* @param mode Mode to operate in (WHITELIST or BLACKLIST).
* @param peerAcl Flag indicating if the lookup is enabled.
* @param reloadTime Interval of time to reload the lookup table.
* @param peerAcl Flag indicating these rules are enabled for enforcing access control.
*/
PeerListLookup(const std::string& filename, Mode mode, uint32_t reloadTime, bool peerAcl);
PeerListLookup(const std::string& filename, uint32_t reloadTime, bool peerAcl);
/**
* @brief Clears all entries from the list.
@ -223,17 +215,6 @@ namespace lookups
*/
bool isPeerListEmpty() const { return m_table.size() == 0U; }
/**
* @brief Sets the mode to either WHITELIST or BLACKLIST.
* @param mode The mode to set.
*/
void setMode(Mode mode);
/**
* @brief Gets the current mode.
* @returns Mode Current peer list operational mode.
*/
Mode getMode() const;
/**
* @brief Gets the entire peer ID table.
* @returns std::unordered_map<uint32_t, PeerId>
@ -261,8 +242,6 @@ namespace lookups
bool save() override;
private:
Mode m_mode;
static std::mutex m_mutex; //! Mutex used for change locking.
static bool m_locked; //! Flag used for read locking (prevents find lookups), should be used when atomic operations (add/erase/etc) are being used.
};

11
src/fne/HostFNE.cpp
Unescape View File

@ -399,16 +399,8 @@ bool HostFNE::readParams()
std::string peerListLookupFile = systemConf["peer_acl"]["file"].as<std::string>();
bool peerListLookupEnable = systemConf["peer_acl"]["enable"].as<bool>(false);
std::string peerListModeStr = systemConf["peer_acl"]["mode"].as<std::string>("whitelist");
uint32_t peerListConfigReload = systemConf["peer_acl"]["time"].as<uint32_t>(30U);
lookups::PeerListLookup::Mode peerListMode;
if (peerListModeStr == "blacklist") {
peerListMode = lookups::PeerListLookup::BLACKLIST;
} else {
peerListMode = lookups::PeerListLookup::WHITELIST;
}
LogInfo("Talkgroup Rule Lookups");
LogInfo(" File: %s", talkgroupConfig.length() > 0U ? talkgroupConfig.c_str() : "None");
if (talkgroupConfigReload > 0U)
@ -421,12 +413,11 @@ bool HostFNE::readParams()
// try to load peer whitelist/blacklist
LogInfo("Peer List Lookups");
LogInfo(" Enabled: %s", peerListLookupEnable ? "yes" : "no");
LogInfo(" Mode: %s", peerListMode == lookups::PeerListLookup::BLACKLIST ? "blacklist" : "whitelist");
LogInfo(" File: %s", peerListLookupFile.length() > 0U ? peerListLookupFile.c_str() : "None");
if (peerListConfigReload > 0U)
LogInfo(" Reload: %u mins", peerListConfigReload);
m_peerListLookup = new PeerListLookup(peerListLookupFile, peerListMode, peerListConfigReload, peerListLookupEnable);
m_peerListLookup = new PeerListLookup(peerListLookupFile, peerListConfigReload, peerListLookupEnable);
m_peerListLookup->read();
// try to load peer whitelist/blacklist

21
src/fne/network/FNENetwork.cpp
Unescape View File

@ -714,11 +714,7 @@ void FNENetwork::taskNetworkRx(NetPacketRequest* req)
}
if (!network->m_peerListLookup->isPeerAllowed(peerId) && !network->m_peerListLookup->isPeerListEmpty()) {
if (network->m_peerListLookup->getMode() == lookups::PeerListLookup::BLACKLIST) {
LogWarning(LOG_NET, "PEER %u RPTL, blacklisted from access", peerId);
} else {
LogWarning(LOG_NET, "PEER %u RPTL, failed whitelist check", peerId);
}
LogWarning(LOG_NET, "PEER %u RPTL, failed peer ACL check", peerId);
network->writePeerNAK(peerId, TAG_REPEATER_LOGIN, NET_CONN_NAK_PEER_ACL, req->address, req->addrLen);
@ -751,11 +747,7 @@ void FNENetwork::taskNetworkRx(NetPacketRequest* req)
}
if (!network->m_peerListLookup->isPeerAllowed(peerId) && !network->m_peerListLookup->isPeerListEmpty()) {
if (network->m_peerListLookup->getMode() == lookups::PeerListLookup::BLACKLIST) {
LogWarning(LOG_NET, "PEER %u RPTL, blacklisted from access", peerId);
} else {
LogWarning(LOG_NET, "PEER %u RPTL, failed whitelist check", peerId);
}
LogWarning(LOG_NET, "PEER %u RPTL, failed peer ACL check", peerId);
network->writePeerNAK(peerId, TAG_REPEATER_LOGIN, NET_CONN_NAK_PEER_ACL, req->address, req->addrLen);
@ -805,12 +797,7 @@ void FNENetwork::taskNetworkRx(NetPacketRequest* req)
bool validAcl = true;
if (network->m_peerListLookup->getACL()) {
if (!network->m_peerListLookup->isPeerAllowed(peerId) && !network->m_peerListLookup->isPeerListEmpty()) {
if (network->m_peerListLookup->getMode() == lookups::PeerListLookup::BLACKLIST) {
LogWarning(LOG_NET, "PEER %u RPTK, blacklisted from access", peerId);
} else {
LogWarning(LOG_NET, "PEER %u RPTK, failed whitelist check", peerId);
}
LogWarning(LOG_NET, "PEER %u RPTK, failed peer ACL check", peerId);
validAcl = false;
} else {
lookups::PeerId peerEntry = network->m_peerListLookup->find(peerId);
@ -1158,7 +1145,6 @@ void FNENetwork::taskNetworkRx(NetPacketRequest* req)
if (connection->connected() && connection->address() == ip) {
// is this peer allowed to request keys?
if (network->m_peerListLookup->getACL()) {
if (network->m_peerListLookup->getMode() == lookups::PeerListLookup::WHITELIST) {
lookups::PeerId peerEntry = network->m_peerListLookup->find(peerId);
if (peerEntry.peerDefault()) {
break;
@ -1169,7 +1155,6 @@ void FNENetwork::taskNetworkRx(NetPacketRequest* req)
}
}
}
}
std::unique_ptr<KMMFrame> frame = KMMFactory::create(req->buffer + 11U);
if (frame == nullptr) {

38
src/fne/network/RESTAPI.cpp
Unescape View File

@ -650,7 +650,6 @@ void RESTAPI::initializeEndpoints()
m_dispatcher.match(FNE_PUT_PEER_ADD).put(REST_API_BIND(RESTAPI::restAPI_PutPeerAdd, this));
m_dispatcher.match(FNE_PUT_PEER_DELETE).put(REST_API_BIND(RESTAPI::restAPI_PutPeerDelete, this));
m_dispatcher.match(FNE_GET_PEER_COMMIT).get(REST_API_BIND(RESTAPI::restAPI_GetPeerCommit, this));
m_dispatcher.match(FNE_GET_PEER_MODE).get(REST_API_BIND(RESTAPI::restAPI_GetPeerMode, this));
m_dispatcher.match(FNE_GET_FORCE_UPDATE).get(REST_API_BIND(RESTAPI::restAPI_GetForceUpdate, this));
@ -1334,43 +1333,6 @@ void RESTAPI::restAPI_GetPeerCommit(const HTTPPayload& request, HTTPPayload& rep
/* */
void RESTAPI::restAPI_GetPeerMode(const HTTPPayload& request, HTTPPayload& reply, const RequestMatch& match)
{
if (!validateAuth(request, reply)) {
return;
}
json::object response = json::object();
setResponseDefaultStatus(response);
lookups::PeerListLookup::Mode mode = m_peerListLookup->getMode();
bool acl = m_peerListLookup->getACL();
std::string modeStr;
if (acl) {
switch (mode) {
case lookups::PeerListLookup::WHITELIST:
modeStr = "WHITELIST";
break;
case lookups::PeerListLookup::BLACKLIST:
modeStr = "BLACKLIST";
break;
default:
modeStr = "UNKNOWN";
break;
}
}
else {
modeStr = "DISABLED";
}
response["mode"].set<std::string>(modeStr);
reply.payload(response);
}
/* */
void RESTAPI::restAPI_GetForceUpdate(const HTTPPayload& request, HTTPPayload& reply, const RequestMatch& match)
{
if (!validateAuth(request, reply)) {

7
src/fne/network/RESTAPI.h
Unescape View File

@ -269,13 +269,6 @@ private:
* @param match HTTP request matcher.
*/
void restAPI_GetPeerCommit(const HTTPPayload& request, HTTPPayload& reply, const network::rest::RequestMatch& match);
/**
* @brief
* @param request HTTP request.
* @param reply HTTP reply.
* @param match HTTP request matcher.
*/
void restAPI_GetPeerMode(const HTTPPayload& request, HTTPPayload& reply, const network::rest::RequestMatch& match);
/**
* @brief

1
src/fne/network/RESTDefines.h
Unescape View File

@ -43,7 +43,6 @@
#define FNE_PUT_PEER_ADD "/peer/add"
#define FNE_PUT_PEER_DELETE "/peer/delete"
#define FNE_GET_PEER_COMMIT "/peer/commit"
#define FNE_GET_PEER_MODE "/peer/mode"
#define FNE_GET_FORCE_UPDATE "/force-update"

2
src/peered/PeerEdMain.cpp
Unescape View File

@ -208,7 +208,7 @@ int main(int argc, char** argv)
g_logDisplayLevel = 0U;
g_pidLookups = new PeerListLookup(g_iniFile, PeerListLookup::WHITELIST, 0U, false);
g_pidLookups = new PeerListLookup(g_iniFile, 0U, false);
g_pidLookups->read();
LogMessage(LOG_HOST, "Loaded peer ID file: %s", g_iniFile.c_str());

Write Preview
Loading…
Cancel
Save

Powered by TurnKey Linux.