Skip to content

Commit cce218f

Browse files
cmanallenDav1dde
andauthored
log(replays): Add organization-id and project-id to log messages (#3949)
So we can look up log messages by project and org. #skip-changelog --------- Co-authored-by: David Herberth <[email protected]>
1 parent 1543031 commit cce218f

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

relay-server/src/services/processor/replay.rs

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ use std::error::Error;
33
use std::net::IpAddr;
44

55
use bytes::Bytes;
6+
use relay_base_schema::project::ProjectId;
67
use relay_dynamic_config::{Feature, GlobalConfig, ProjectConfig};
78
use relay_event_normalization::replay::{self, ReplayError};
89
use relay_event_normalization::RawUserAgentInfo;
@@ -28,6 +29,8 @@ pub fn process(
2829
let replays_disabled = state.should_filter(Feature::SessionReplay);
2930
let scrubbing_enabled = project_state.has_feature(Feature::SessionReplayRecordingScrubbing);
3031
let replay_video_disabled = project_state.has_feature(Feature::SessionReplayVideoDisabled);
32+
let project_id = project_state.project_id;
33+
let organization_id = project_state.organization_id;
3134

3235
let meta = state.envelope().meta().clone();
3336
let client_addr = meta.client_addr();
@@ -79,6 +82,8 @@ pub fn process(
7982
global_config,
8083
client_addr,
8184
user_agent,
85+
project_id,
86+
organization_id,
8287
)?;
8388
item.set_payload(ContentType::Json, replay_event);
8489
}
@@ -101,6 +106,8 @@ pub fn process(
101106
user_agent,
102107
scrubbing_enabled,
103108
&mut scrubber,
109+
project_id,
110+
organization_id,
104111
)?;
105112
item.set_payload(ContentType::OctetStream, replay_video);
106113
}
@@ -113,13 +120,16 @@ pub fn process(
113120

114121
// Replay Event Processing.
115122

123+
#[allow(clippy::too_many_arguments)]
116124
fn handle_replay_event_item(
117125
payload: Bytes,
118126
event_id: &Option<EventId>,
119127
config: &ProjectConfig,
120128
global_config: &GlobalConfig,
121129
client_ip: Option<IpAddr>,
122130
user_agent: &RawUserAgentInfo<&str>,
131+
project_id: Option<ProjectId>,
132+
organization_id: Option<u64>,
123133
) -> Result<Bytes, ProcessingError> {
124134
let filter_settings = &config.filter_settings;
125135

@@ -151,6 +161,8 @@ fn handle_replay_event_item(
151161
relay_log::warn!(
152162
error = &error as &dyn Error,
153163
?event_id,
164+
project_id = project_id.map(|v| v.value()),
165+
organization_id = organization_id,
154166
"invalid replay event"
155167
);
156168
Err(match error {
@@ -259,6 +271,8 @@ fn handle_replay_video_item(
259271
user_agent: &RawUserAgentInfo<&str>,
260272
scrubbing_enabled: bool,
261273
scrubber: &mut RecordingScrubber,
274+
project_id: Option<ProjectId>,
275+
organization_id: Option<u64>,
262276
) -> Result<Bytes, ProcessingError> {
263277
let ReplayVideoEvent {
264278
replay_event,
@@ -282,6 +296,8 @@ fn handle_replay_video_item(
282296
global_config,
283297
client_ip,
284298
user_agent,
299+
project_id,
300+
organization_id,
285301
)?;
286302

287303
// Process as a replay-recording envelope item.

0 commit comments

Comments
 (0)