catbox-memory icon indicating copy to clipboard operation
catbox-memory copied to clipboard

Error Handling is not done properly

Open Ashik155 opened this issue 1 year ago • 1 comments

Runtime

nodejs

Runtime version

16.15

Module version

4.1.1

Last module version without issue

No response

Used with

hapi server cache.

Any other relevant information

image Recentyl we have noticed that this error is not getting handled properly.

What are you trying to achieve or the steps to reproduce?

a proper error handling and following that next steps for smooth processing

What was the result you got?

Cache size limit reached, stack: Error: Cache size limit reached\n at module.exports.internals.Connection.set (/opt/bba/server/node_modules/@hapi/catbox-memory/lib/index.js:184:19)\n at module.exports.set (/opt/bba/server/node_modules/@hapi/catbox/lib/client.js:91:31)\n at module.exports.internals.Policy.set (/opt/bba/server/node_modules/@hapi/catbox/lib/policy.js:298:31)\n at internals.onPreResponse (/opt/bba/server/node_modules/@hapi/yar/lib/index.js:307:17)\n at async module.exports.internals.Manager.execute (/opt/bba/server/node_modules/@hapi/hapi/lib/toolkit.js:45:28)\n at async Request._invoke (/opt/bba/server/node_modules/@hapi/hapi/lib/request.js:339:30)\n at async Request._postCycle (/opt/bba/server/node_modules/@hapi/hapi/lib/request.js:402:32)\n at async Request._reply (/opt/bba/server/node_modules/@hapi/hapi/lib/request.js:381:9)\n"}

What result did you expect?

It should provide better error handling

Ashik155 avatar Mar 11 '24 18:03 Ashik155

Please provide a full reproduction for your issue. Note that duplicating your issue is neither necessary nor helpful.

Marsup avatar Mar 11 '24 19:03 Marsup