Skip to content

Commit eba055b

Browse files
authored
Merge pull request #2447 from mavlink/pr-v2.12-ftp-warning
[BACKPORT v2.12] core: remove way too verbose warnings
2 parents e31106b + fc9d8e1 commit eba055b

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

src/mavsdk/core/mavlink_ftp_client.cpp

+6-2
Original file line numberDiff line numberDiff line change
@@ -107,13 +107,17 @@ void MavlinkFtpClient::process_mavlink_ftp_message(const mavlink_message_t& msg)
107107
mavlink_msg_file_transfer_protocol_decode(&msg, &ftp_req);
108108

109109
if (ftp_req.target_system != 0 && ftp_req.target_system != _system_impl.get_own_system_id()) {
110-
LogWarn() << "Received FTP with wrong target system ID!";
110+
if (_debugging) {
111+
LogDebug() << "Received FTP message with wrong target system ID";
112+
}
111113
return;
112114
}
113115

114116
if (ftp_req.target_component != 0 &&
115117
ftp_req.target_component != _system_impl.get_own_component_id()) {
116-
LogWarn() << "Received FTP with wrong target component ID!";
118+
if (_debugging) {
119+
LogDebug() << "Received FTP message with wrong target component ID";
120+
}
117121
return;
118122
}
119123

src/mavsdk/core/mavlink_ftp_server.cpp

+6-2
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,17 @@ void MavlinkFtpServer::process_mavlink_ftp_message(const mavlink_message_t& msg)
4545

4646
if (ftp_req.target_system != 0 &&
4747
ftp_req.target_system != _server_component_impl.get_own_system_id()) {
48-
LogWarn() << "wrong sysid!";
48+
if (_debugging) {
49+
LogDebug() << "Received FTP message with wrong target system ID";
50+
}
4951
return;
5052
}
5153

5254
if (ftp_req.target_component != 0 &&
5355
ftp_req.target_component != _server_component_impl.get_own_component_id()) {
54-
LogWarn() << "wrong compid!";
56+
if (_debugging) {
57+
LogDebug() << "Received FTP message with wrong target component ID";
58+
}
5559
return;
5660
}
5761

0 commit comments

Comments
 (0)