upstash / qstash-js

Message queue for serverless
https://docs.upstash.com/qstash
MIT License
133 stars 11 forks source link

Add Missing Fields in Requests & Add DLQ Filtering and Batch Cancel API #114

Closed CahidArda closed 3 days ago

CahidArda commented 1 week ago

Updating Qstash JS with the recent updates to backend.

This pr includes the following changes as @mdumandag listed:

  1. 1e6e690: message type has missing fields (like endpoint, api, caller ip, bodyBase64 etc.)
  2. bc1c164: dlq message type has missing fields (like response status, response headers etc.)
  3. d4fc931: dlq list method missing filter and count (the python client has a pending pr for it) https://github.com/upstash/qstash-py/pull/21
  4. 60e89d5: event filter missing api field
  5. 5c01b1c: event list response missing some fields (like api, queue name, schedule id, and fields in this
  6. d0f588f: scheduele type missing fields (like bodyBase64 and caller ip)
  7. dd93920: missing batch cancel apis (the python client has a pending pr for it) https://github.com/upstash/qstash-py/pull/22
mdumandag commented 1 week ago

We might do some changes in the cancelAll API in the qstash server, so let's not merge this (or merge, but not release) for now. I will let you know when the changes are finalized&deployed

mdumandag commented 4 days ago

@CahidArda we can merge the PR with the current api for the batch cancel, we decided to proceed with the current state on the qstash server