Quasar pages: ticket link and ws fast-path still assume ticket id == payment_hash #40
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Since
7b761a1every ticket row gets a freshurlsafe_short_hash()id and only thepayment_hashcolumn carries the invoice hash. Two spots in the extension's own UI/API still use the old invariant:static/js/display.jspaymentSuccess(paymentHash):link: '/events/ticket/' + paymentHashandwindow.open('/events/ticket/' + paymentHash)→ 404 "Ticket does not exist" for every purchase made through the extension's own event page. Fix: readticket_idsfromPOST /api/v1/tickets/{event_id}/{payment_hash}(or the ws payload) and link the first id (all of them for multi-ticket).views_api.pywebsocket_endpoint:ticket = await get_ticket(payment_hash)looks up by row id, so the already-paid fast path never fires; useget_tickets_by_payment_hash.display.jsalso watches core's/api/v1/ws/{hash}rather than the extension's/events/api/v1/tickets/ws/{hash}.The webapp is unaffected (it uses the poll endpoint's
ticket_ids). Found during the guest-checkout review (#36).