abreits / amqp-ts

AmqpSimple, a RabbitMQ tested Amqp library written in and for Typescript
MIT License
133 stars 37 forks source link

bug fix about 'ApplicationName' #62

Closed lilis7575 closed 1 month ago

lilis7575 commented 3 years ago

https://github.com/abreits/amqp-ts/issues/20 It is also the content of the above issue. This is a problem that occurred when using amqp-ts in electron.

import * as Amqp from "amqp-ts";

If the content is declared in app.ts and used, the application will refer to amqp-ts immediately at runtime, This application will be assigned an ApplicationName. At this time, if the process.env.AMQPTS_APPLICATIONNAME value of the process environment is not assigned, the length of process.argv is not present, or is less than 1, an error inevitably occurs.

Since ApplicationName is not a global variable, we know that it is only used in the consumerQueueName method.

ApplicationName has a valid value only when the application is started and the startConsumer is executed as needed.

Therefore, modify to allocate AppliacationName when consumerQueueName method is called.

lilis7575 commented 3 years ago

Please check my PR