Azure / azure-functions-durable-js

JavaScript library for using the Durable Functions bindings
https://www.npmjs.com/package/durable-functions
MIT License
128 stars 46 forks source link

Fix: change maxBodyLength axios config to allow large inputs #480

Closed nials-relevant closed 1 year ago

nials-relevant commented 1 year ago

Closes #479

davidmrdavid commented 1 year ago

Thanks @nials-relevant for your contribution. I'm a bit torn on accepting this change mostly because inputs and outputs to DF APIs are supposed to be relatively small, and therefore the default limit placed by axios makes for a convenient way of indirectly enforcing this on a DF client.

However, I also recognize that users who know what they are doing would benefit from this. Perhaps there's a middle ground where this setting can be configured, but we log a warning whenever large inputs over a given size are detected. Would something like that fit your usecase, @nials-relevant ?