Save fault when session terminates unsuccessfully
This commit is contained in:
1 parent
bded263adf
commit
63cd8dd2de
1 file changed
+15
-3
+15
-3
@@ -198,6 +198,7 @@ async function writeResponse(
|
|||||||
});
|
});
|
||||||
await endSession(sessionContext);
|
await endSession(sessionContext);
|
||||||
} else if (close) {
|
} else if (close) {
|
||||||
|
session.clearProvisions(sessionContext);
|
||||||
await endSession(sessionContext);
|
await endSession(sessionContext);
|
||||||
} else {
|
} else {
|
||||||
const now = Date.now();
|
const now = Date.now();
|
||||||
@@ -734,6 +735,17 @@ async function nextRpc(sessionContext: SessionContext): Promise<void> {
|
|||||||
async function endSession(sessionContext: SessionContext): Promise<void> {
|
async function endSession(sessionContext: SessionContext): Promise<void> {
|
||||||
let saveCache = sessionContext.cacheUntil != null;
|
let saveCache = sessionContext.cacheUntil != null;
|
||||||
|
|
||||||
|
if (sessionContext.provisions.length) {
|
||||||
|
const fault = {
|
||||||
|
code: "session_terminated",
|
||||||
|
message: "The TR-069 session was unsuccessfully terminated",
|
||||||
|
timestamp: sessionContext.timestamp,
|
||||||
|
};
|
||||||
|
recordFault(sessionContext, fault);
|
||||||
|
// No need to save retryNow
|
||||||
|
for (const f of Object.values(sessionContext.faults)) delete f.retryNow;
|
||||||
|
}
|
||||||
|
|
||||||
const promises = [];
|
const promises = [];
|
||||||
|
|
||||||
promises.push(
|
promises.push(
|
||||||
@@ -892,11 +904,11 @@ export function onConnection(socket: Socket): void {
|
|||||||
const now = Date.now();
|
const now = Date.now();
|
||||||
|
|
||||||
const lastActivity = sessionContext.lastActivity;
|
const lastActivity = sessionContext.lastActivity;
|
||||||
const timeoutMsg = logger.flatten({
|
const timeoutMsg = {
|
||||||
sessionContext: sessionContext,
|
sessionContext: sessionContext,
|
||||||
message: "Session timeout",
|
message: "Session timeout",
|
||||||
sessionTimestamp: sessionContext.timestamp,
|
sessionTimestamp: sessionContext.timestamp,
|
||||||
});
|
};
|
||||||
|
|
||||||
const timeout =
|
const timeout =
|
||||||
sessionContext.lastActivity + sessionContext.timeout * 1000 - now;
|
sessionContext.lastActivity + sessionContext.timeout * 1000 - now;
|
||||||
@@ -916,7 +928,7 @@ export function onConnection(socket: Socket): void {
|
|||||||
const _sessionContext = await session.deserialize(sessionContextString);
|
const _sessionContext = await session.deserialize(sessionContextString);
|
||||||
if (_sessionContext.lastActivity === lastActivity) {
|
if (_sessionContext.lastActivity === lastActivity) {
|
||||||
logger.accessError(timeoutMsg);
|
logger.accessError(timeoutMsg);
|
||||||
await endSession(_sessionContext);
|
await endSession(sessionContext);
|
||||||
}
|
}
|
||||||
}, timeout + 1000).unref();
|
}, timeout + 1000).unref();
|
||||||
|
|
||||||
|
|||||||
Reference in new issue
Block a user