Skip to content

Commit

Permalink
Refactor JSONCookie function for clarity and compliance
Browse files Browse the repository at this point in the history
- Update JSDoc to clarify behavior and return types.
- Replace str.substr(0, 2) with str.startsWith to check for the prefix 'j:'.
- Simplify error handling by removing unused catch parameter.
- Maintain functionality for parsing JSON cookies, returning undefined for invalid inputs.
  • Loading branch information
Ayoub-Mabrouk committed Nov 1, 2024
1 parent 5d61e1e commit 856ccfe
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,18 +76,18 @@ function cookieParser (secret, options) {
* Parse JSON cookie string.
*
* @param {String} str
* @return {Object} Parsed object or undefined if not json cookie
* @return {Object|undefined} Parsed object or undefined if not json cookie
* @public
*/

function JSONCookie (str) {
if (typeof str !== 'string' || str.substr(0, 2) !== 'j:') {
if (typeof str !== 'string' || !str.startsWith('j:')) {
return undefined
}

try {
return JSON.parse(str.slice(2))
} catch (err) {
} catch {
return undefined
}
}
Expand Down

0 comments on commit 856ccfe

Please sign in to comment.