Fabio/webssh2
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat: validateSshTerm checks if term is undefined or null before validation

Browse source
This commit is contained in:
Bill Church 2024-08-17 14:20:01 +00:00
parent 82c0da0ff7
commit 28f329e315
No known key found for this signature in database
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
app/utils.js
View file

@ -1,6 +1,8 @@
// server // server
// /app/utils.js // /app/utils.js
const validator = require("validator"); const validator = require("validator")
const createDebug = require("debug")
const debug = createDebug("webssh2:utils")
/** /**
* Sanitizes an object by replacing sensitive properties with asterisks. * Sanitizes an object by replacing sensitive properties with asterisks.
@ -34,7 +36,6 @@ function sanitizeObject(
return obj return obj
} }
/** /**
* Validates the SSH terminal name using validator functions. * Validates the SSH terminal name using validator functions.
* Allows alphanumeric characters, hyphens, and periods. * Allows alphanumeric characters, hyphens, and periods.
@ -42,6 +43,11 @@ function sanitizeObject(
* @returns {boolean} True if the terminal name is valid, false otherwise * @returns {boolean} True if the terminal name is valid, false otherwise
*/ */
function validateSshTerm(term) { function validateSshTerm(term) {
debug(`validateSshTerm: %O`, term)
if (term === undefined || term === null) {
return false
}
return ( return (
validator.isLength(term, { min: 1, max: 30 }) && validator.isLength(term, { min: 1, max: 30 }) &&
validator.matches(term, /^[a-zA-Z0-9.-]+$/) validator.matches(term, /^[a-zA-Z0-9.-]+$/)
@ -49,4 +55,4 @@ function validateSshTerm(term) {
} }
exports.sanitizeObject = sanitizeObject exports.sanitizeObject = sanitizeObject
exports.validateSshTerm = validateSshTerm; exports.validateSshTerm = validateSshTerm