Skip to content

Commit edc36de

Browse files
committed
Put new error messages on one line for easier greppability
1 parent c309461 commit edc36de

1 file changed

Lines changed: 6 additions & 12 deletions

File tree

src/UserSentry.hh

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -152,22 +152,18 @@ public:
152152
} while (1);
153153
if (result == nullptr) {
154154
if (retval) { // There's an actual error in the lookup.
155-
m_log.Emsg("UserSentry", "Failure when looking up UID for username", username.c_str(), strerror(retval));
156-
m_log.Emsg("UserSentry", "Multiuser denying access");
155+
m_log.Emsg("UserSentry", "Multiuser denying access: Failure when looking up UID for username", username.c_str(), strerror(retval));
157156
} else { // Username doesn't exist.
158-
m_log.Emsg("UserSentry", "XRootD mapped request to username that does not exist:", username.c_str());
159-
m_log.Emsg("UserSentry", "Multiuser denying access");
157+
m_log.Emsg("UserSentry", "Multiuser denying access: XRootD mapped request to username that does not exist:", username.c_str());
160158
}
161159
return;
162160
}
163161
if (pwd.pw_uid < g_minimum_uid) {
164-
m_log.Emsg("UserSentry", "Username", username.c_str(), "maps to a system UID; rejecting lookup");
165-
m_log.Emsg("UserSentry", "Multiuser denying access");
162+
m_log.Emsg("UserSentry", "Multiuser denying access: Username", username.c_str(), "maps to a system UID; rejecting lookup");
166163
return;
167164
}
168165
if (pwd.pw_gid < g_minimum_gid) {
169-
m_log.Emsg("UserSentry", "Username", username.c_str(), "maps to a system GID; rejecting lookup");
170-
m_log.Emsg("UserSentry", "Multiuser denying access");
166+
m_log.Emsg("UserSentry", "Multiuser denying access: Username", username.c_str(), "maps to a system GID; rejecting lookup");
171167
return;
172168
}
173169

@@ -185,8 +181,7 @@ public:
185181
break;
186182
} while (1);
187183
if (-1 == retval) {
188-
m_log.Emsg("UserSentry", "Failure when looking up supplementary groups for username", username.c_str());
189-
m_log.Emsg("UserSentry", "Multiuser denying access");
184+
m_log.Emsg("UserSentry", "Multiuser denying access: Failure when looking up supplementary groups for username", username.c_str());
190185
return;
191186
}
192187

@@ -197,8 +192,7 @@ public:
197192
m_log.Emsg("UserSentry", "Switching FS uid for user", username.c_str());
198193
m_orig_uid = setfsuid(result->pw_uid);
199194
if (m_orig_uid < 0) {
200-
m_log.Emsg("UserSentry", "Failed to switch FS uid for user", username.c_str());
201-
m_log.Emsg("UserSentry", "Multiuser denying access");
195+
m_log.Emsg("UserSentry", "Multiuser denying access: Failed to switch FS uid for user", username.c_str());
202196
return;
203197
}
204198
m_orig_gid = setfsgid(result->pw_gid);

0 commit comments

Comments
 (0)