hapijs / hapijs/catbox-memory

Error Handling is not done properly

Open
#89 1 comment 0 reactions 0 assignees View on GitHub
bug
Dominant language
JavaScript
Stars
33
Forks
30
PR merge metrics
No merged PRs in 30d

Description

### 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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.