payments

Background

It's unclear to many people what sphinxreplay.db is for.

Context

Replying to the question: "What is sphinxreplay.db for", on Slack:
@Roasbeef : "Security: it prevents replays of onion packets, w/o this ppl can collect onion packetrs then re-inject them and see where they travel to attempt to de-annon"
@Roasbeef : "I think we might be the only impl that handles them pr

Describe the bug
When attempting to delete a user, BTCPay Server throws a 500 error and does not delete it.

When I attempt to disable that same user, the UI says that the user is disabled via the toast message but the user is not disabled and the logs complain as well.

I also tried disabling/deleting after removing admin privs from the user, also without success.

**To Reproduce the b

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