Top 10 ports #238
Top 10 ports #238
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #238 +/- ##
==========================================
- Coverage 40.79% 40.70% -0.09%
==========================================
Files 104 105 +1
Lines 10126 10168 +42
Branches 1494 1493 -1
==========================================
+ Hits 4131 4139 +8
- Misses 5130 5164 +34
Partials 865 865
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Zadamsa
force-pushed
the
top-10-ports
branch
2 times, most recently
from
c723319 to
abd2f62
Compare
Zadamsa
force-pushed
the
top-10-ports
branch
2 times, most recently
from
7e649d8 to
8eaa5fa
Compare
input/parser.cpp
Outdated
| pkt->src_port = ntohs(udp->source); | ||
| pkt->dst_port = ntohs(udp->dest); | ||
|
|
||
| stats.top_ports.insert(pkt->src_port); |
There was a problem hiding this comment.
V procesních vláknech, které provádí zpracování paketů by neměla být žádná složitější logika zbytečně navíc.
Tato aktualizace top portů má v sobě docela složitou logiku, která by tu být neměla.
Jediné, co by procesní vlákna zde měla dělat je zvýšení čítače pro daný port.
Veškerá složitější logika, která zobrazí top 10 portů atd. by se měla řešit při vyčítání telemetrii.
Zadamsa
force-pushed
the
top-10-ports
branch
2 times, most recently
from
6e0bb5d to
9285a37
Compare
input/topPorts.hpp
Outdated
| * \brief Get the top ports. | ||
| * \return Pair of the top ports array and their count. | ||
| */ | ||
| std::pair<std::array<std::pair<uint16_t, size_t>, TopPortsCount>, size_t> |
There was a problem hiding this comment.
Toto je opravdu hodne neprehledne, kde na prvni pohled vubec nevis o co jde.
Slo by to urcite lepe strukturovat, napr. takto:
struct PortStats {
uint16_t portNumber;
uint64_t portFrequency;
};
// asi bych spise pouzil vector, protoze muzes mit i mene zachycenych portu nez tech TopPortsCount
std::vector<PortStats> get_top_ports() const
Co myslis?
std::pair<std::array<std::pair<uint16_t, size_t>, TopPortsCount>, size_t>
vs
std::vector<PortStats>
Zaroven by mi prislo i zajime, sbirat statistiky pro TCP a UDP zvlast.
Pak dat vysledek typu:
53 / UDP - 238243
79a4bc1 to
944d19d
Compare
Zadamsa
force-pushed
the
top-10-ports
branch
from
944d19d to
60a204f
Compare
3 participants
No description provided.