[Bug]: TypeError: Cannot set property message of [object Object] which has only a getter

[elmeister]

[Bug]: TypeError: Cannot set property message of [object Object] which has only a getter

[nicholasgriffintn]

I don't have a clue what you're talking about honestly, please provide more detail with some sort of reproducible example, without that, we can't help you.

[elmeister]

Ok, so here's the zod library, which is pretty popular

Here is their ZodError implementation

For the minimal reproducible example, here's the error class with the sane idea as in ZodError:

class CustomError extends Error {
  issues: string[]

  constructor(issues: string[]) {
    super()

    this.issues = issues
  }

  get message() {
    return this.issues.join(', ')
  }
}

Now if within the handleMessage() the error above will be thrown

throw new CustomError(['issue1', 'issue2'])

It will fail here
In these consumer.ts lines of code

err.message =
  err instanceof TimeoutError
    ? `Message handler timed out after ${this.handleMessageTimeout}ms: Operation timed out.`
    : `Unexpected message handler failure: ${err.message}`;

So instead of "Unexpected message handler" we will receive only the error from consumer.ts

TypeError: Cannot set property message of [object Object] which has only a getter

[nicholasgriffintn]

Ahh I see so you're throwing a zod error and that is causing sqs-consumer to error.

Not an issue with sqs consumer itself but with the reformatting of the error from zod that was thrown, as sqs consumer is changing the property 'message' which you have pre defined as a get.

[elmeister]

Zod package is commonly used to validate and add types to unknown data, which perfectly fits for SQS message body, and Zod throws its own error with getter for message, when data was not valid.
The issue is manageable by adding try/catch wrapper for the handleMessage()callback content.

[nicholasgriffintn]

Sorry not sure I explained well enough, but basically, I know what zod is, the issue here is that the error sent has a property that is only a get, which means that sqs-consumer is unable to edit the message property itself.

This removes its ability to adjust the message property and thus offer its own error functionality, you're basically overwriting that ability with this implementation.

The error being outputted is correct and is actually on your app's side, and isn't a bug with sqs-consumer itself.

[elmeister]

I do not overwrite that ability in my app, it's how zod's parse() method works.
But anyways, closing this.

[github-actions]

This issue has been closed for more than 30 days. If this issue is still occuring, please open a new issue with more recent context.