payments

The htlcswitch test function initSwitchWithDB may create its own database if nil is passed in. Several tests that pass in nil never remove the temporary directory. I don't think any of the tests call db.Close so the test process will still have the file handle. This leads to too many open files... on my machine. The fix here is to remove the temp directory and call Close on the databas

in WebhookEventTrigger.from_request we log the webhook request IP address, but we extract it in a simplistic way:

https://github.com/dj-stripe/dj-stripe/blob/0dd7925602e9b2ac693391fc9c02c4d87b97f5a9/djstripe/models/webhooks.py#L86-L92

It might be useful to add a settings-overridable function here so that users could use django-ipware (https://github.com/un33k/django-ipware) etc to do a mor

When we return a bitcoin amount, our JSON responses currently specify the unit (btc, sat, msat) in the parameter name, but it's a bit ugly and is missing in some places.

Maybe it would be better to include the unit explicitly in the response. I see two different options:

• Instead of returning a numeric value, return a string including the unit (e.g. "2500 msat", "0.45 btc", "1.5 mbtc"). The

payment_request is returned so you can parse that to display the memo