amqp-ts icon indicating copy to clipboard operation
amqp-ts copied to clipboard

Update amqplib version from 0.4.x (0.4.2) to 0.5.x (0.5.5)

Open austin-beer opened this issue 4 years ago • 6 comments

amqp-ts currently uses an outdated version of amqplib. This PR updates it to use the latest version.

Changelog for amqplib: https://github.com/squaremo/amqp.node/blob/master/CHANGELOG.md

austin-beer avatar Oct 11 '19 16:10 austin-beer

it seems like this could possibly fix node-red-contrib-amqp when run in docker:

2020-02-23T20:50:28.450Z Install : node-red-contrib-amqp 1.0.2

2020-02-23T20:50:28.552Z npm install --no-audit --no-update-notifier --save --save-prefix="~" --production [email protected]
2020-02-23T20:50:33.846Z [err] npm
2020-02-23T20:50:33.846Z [err]  WARN notsup Unsupported engine for [email protected]: wanted: {"node":">=0.8 <6 || ^6"} (current: {"node":"10.19.0","npm":"6.13.4"})
2020-02-23T20:50:33.846Z [err] npm WARN notsup Not compatible with your version of node/npm: [email protected]
2020-02-23T20:50:33.847Z [err] 
2020-02-23T20:50:33.850Z [out] + [email protected]
2020-02-23T20:50:33.850Z [out] added 22 packages from 61 contributors in 3.782s
2020-02-23T20:50:33.906Z rc=0

uminac avatar Feb 23 '20 20:02 uminac

Can this please be merged ?!

CN91 avatar Jun 30 '20 12:06 CN91

might as well go to amqplib v0.5.6

uminac avatar Jun 30 '20 14:06 uminac

Yes but then it still needs to be merged! ;)

CN91 avatar Jun 30 '20 17:06 CN91

2 years passed and still not merged. :-(

romiras avatar Oct 14 '21 10:10 romiras

Sorry, better consider this dead. I unfortunately have no time to keep this library up to date (it is already very outdated). Our organization also does not use it any more.

Also new technologies (e.g. async await) and versions of libraries it is built upon have made it more or less obsolete for most purposes.

From: Roman Mirr @.> Sent: donderdag 14 oktober 2021 12:10 To: abreits/amqp-ts @.> Cc: Subscribed @.***> Subject: Re: [abreits/amqp-ts] Update amqplib version from 0.4.x (0.4.2) to 0.5.x (0.5.5) (#45)

2 years passed and still not merged. :-(

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/abreits/amqp-ts/pull/45#issuecomment-943212610 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCPLHMVFBDIENYQVYHDEM3UG2UAPANCNFSM4I73THVQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub . https://github.com/notifications/beacon/ABCPLHK64VSYJ572CTYLR7TUG2UAPA5CNFSM4I73THV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHA4EQQQ.gif

abreits avatar Oct 14 '21 10:10 abreits