Open wingedfox opened 6 years ago
Hello,
I've found that RenewOnSettle override does no work because provided settings are not used by client. https://github.com/noodlefrenzy/node-amqp10/blob/master/lib/policies/policy_utilities.js#L107 policy provides receiverLink field, but the only place it mentioned and it is not from the overrides https://github.com/noodlefrenzy/node-amqp10/blob/master/lib/amqp_client.js#L289
RenewOnSettle
receiverLink
Either policy or link constructor should be updated to use overrides properly.
Hello,
I've found that
RenewOnSettleoverride does no work because provided settings are not used by client. https://github.com/noodlefrenzy/node-amqp10/blob/master/lib/policies/policy_utilities.js#L107 policy providesreceiverLinkfield, but the only place it mentioned and it is not from the overrides https://github.com/noodlefrenzy/node-amqp10/blob/master/lib/amqp_client.js#L289Either policy or link constructor should be updated to use overrides properly.