Properly support the 2020 api.
ruseinov opened this issue · 6 comments
Currently there is a problem with a webhook:
[2020-05-21T08:43:02Z DEBUG actix_web::middleware::logger] Error in response: Error(Webhook(BadParse(Error("invalid `InvoiceLineItemId`, expected unknown id prefix", line: 161, column: 3))), State { next_error: None, backtrace: InternalBacktrace { backtrace: Some( 0: backtrace::backtrace::trace_unsynchronized
It has been temporary fixed here:
ruseinov#1
I'm not sure exactly how good that fix is but it's definitely better than a failing webhook.
One other thing that I have noticed is that the current implementation fails when receiving test-webhooks. The reason for that is pretty simple, unlike stripe-cli, test webhooks send bogus Ids that look like "_000000" and that breaks id validation. I'm not exactly sure that needs to be fixed, but figured I'd report it anyway.
Is there a reason you haven't submitted this as a pull request?
@bryanburgers I am not 100% sure this is a proper fix as I didn’t dig too deep in the code and the lack of documentation makes it hard to understand some of the implementation details and the reasoning behind them.
I’m guessing these was a reason for certain ID types to be restricted in the way they were, so I would love to hear author’s opinion on the matter.
I am having the same issue.
Is this only on tests webhooks ?
what is blocking from the PR to be merged ? I would like to use it with the current master which use tokio v1