jens/flipper
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Track insecure connection attempts

Browse Source 
Summary: Same as previous diff but for plain socket.

Reviewed By: antonk52

Differential Revision: D46220933

fbshipit-source-id: 235ca860891b8514dd0d743d32fe85f15a2cdd7e
This commit is contained in:
Lorenzo Blasa
2023-05-26 08:02:12 -07:00
committed by Facebook GitHub Bot
parent 135d3e3aa8
commit f25591580c
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
desktop/flipper-server-core/src/comms/ServerController.tsx
View File

@@ -277,13 +277,15 @@ export class ServerController
}
onConnectionAttempt(clientQuery: ClientQuery): void {
// TODO: track plain connection attempt.
const strippedClientQuery = (({device_id, ...o}) => o)(clientQuery);
const id = buildClientId({device_id: 'unknown', ...strippedClientQuery});
this.timestamps.set(id, {
insecureStart: Date.now(),
});
this.logger.track('usage', 'untrusted-request-handler-called', clientQuery);
tracker.track('app-connection-insecure-attempt', clientQuery);
this.connectionTracker.logConnectionAttempt(clientQuery);
const client: UninitializedClient = {

3
desktop/flipper-server-core/src/utils/tracker.tsx
View File

@@ -14,7 +14,7 @@ type AppConnectionPayload = {
os: string;
device: string;
device_id: string;
medium: number | undefined;
medium?: number | undefined;
};
type TrackerEvents = {
@@ -28,6 +28,7 @@ type TrackerEvents = {
'server-proxy-error': {error: string};
'app-connection-created': AppConnectionPayload;
'app-connection-secure-attempt': AppConnectionPayload;
'app-connection-insecure-attempt': AppConnectionPayload;
};
class ServerCoreTracker {