Skip to content

Instantly share code, notes, and snippets.

@cnunciato
Last active December 15, 2017 18:30
Show Gist options
  • Save cnunciato/e9e9db16e50803326c68014e809c66f5 to your computer and use it in GitHub Desktop.
Save cnunciato/e9e9db16e50803326c68014e809c66f5 to your computer and use it in GitHub Desktop.

Notifications APIs

Based on current designs, we need new APIs that support retrieving lists of notifications, filtered by the requesting user, that allow for:

  • Listing all recent (e.g., most recent 50) notifications, grouped by origin.
  • Listing all notifications within an origin, with optional paging.
  • Deleting all notifications, irrespective of origin.
  • Deleting all notifications within an origin.
  • Deleting a single notification.

With that need, I'm thinking of the following new endpoints, respectively:

  • GET /notifications
  • GET /notifications/:origin (with optional ?range=50)
  • DELETE /notifications
  • DELETE /notifications/:origin
  • DELETE /notifications/:origin/:id

Technically we shouldn't need the origin for that last one, but I've included it to more easily differentiate it from DELETE /notifications/:origin.

Example Responses

Note that ...body below represents the body of a notification message, which could take different forms, each of which is detailed in Example Notification Bodies below.

GET /notifications

[
  {
    origin: "core",
    notifications: [
      {
        ...body
      },
      {
        ...body
      }
    ]
  },
  {
    origin: "cnunciato",
    notifications: [
      {
        ...body
      }
    ]
  }
]
GET /notifications/:origin

{
  range_start: 0,
  range_end: 49,
  total_count: 123,
  data: [
    {
      ...body
    },
    {
      ...body
    }
  ]
}
DELETE /notifications
-> 204
DELETE /notifications/:origin
-> 204
DELETE /notifications/:origin/:id
-> 204

Example Notification Bodies

Initally we're interested in the following set of notifications:

  1. Builder started building a package in one of my origins, based on a GitHub webhook.
  2. Builder built (or failed to build) a package in one of my origins.
  3. Builder promoted a package in one of my origins (e.g., to the unstable channel, after a build).
  4. A user promoted a package in one of my origins.
  5. A user demoted a package in one of my origins.
  6. A user submitted a build request for a package in one of my origins.
  7. A user uploaded a package to one of my origins.
  8. A package that is a dependency of one of the packages in any of my origins was promoted to stable.

Suggested responses for these are enumerated below.

1. Job Started

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "job-started",
  category: "info",
  origin: "core",
  data: {
    package: "node"
    group_id: "12345",
    job_id: "67890",
    status: "Processing",
    source: "vcs:github",
    message: "This is the first line of the commit message"
  }
}

2. Job Finished (or Failed)

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "job-finished",
  category: "error",
  origin: "core",
  data: {
    package: "node",
    version: "1.2.3",
    release: "2017010100000000",
    group_id: "12345",
    job_id: "67890",
    status: "Failed",
    source: "vcs:github",
    message: "This is the first line of the commit message",
  }
}

3. Package Promoted (by Builder)

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "package-promoted",
  category: "info",
  origin: "core",
  data: {
    package: "node",
    version: "1.2.3",
    release: "2017010100000000",
    source: "bldr",
    channel: "unstable"
  }
}

4. Package Promoted (by User)

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "package-promoted",
  category: "info",
  origin: "core",
  data: {
    package: "node",
    version: "1.2.3",
    release: "2017010100000000",
    channel: "stable",
    source: "user:cnunciato",
    message: "Releasing! Yay!"
  }
}

5. Package Demoted

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "package-demoted",
  category: "warning",
  origin: "core",
  data: {
    package: "node",
    version: "1.2.3",
    release: "2017010100000000",
    channel: "stable",
    source: "user:cnunciato",
    message: "Some stuff happened, so we had to demote."
  }
}

6. Job Scheduled

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "job-scheduled",
  category: "info",
  origin: "core",
  data: {
    package: "node"
    group_id: "12345",
    job_id: "67890",
    source: "user:cnunciato",
    message: "Do we feel like a message might be useful here, too?"
  }
}

7. Package Uploaded

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "package-uploaded",
  category: "info",
  origin: "core",
  data: {
    package: "node",
    version: "1.2.3",
    release: "2017010100000000",
    channel: "unstable",
    source: "user:cnunciato"
  }
}

8. Dependency Promoted

{
  id: 1234567890,
  timestamp: 1512687331658,
  type: "dep-promoted",
  category: "info",
  origin: "core",
  data: {
    package: "node",
    version: "1.2.3",
    release: "2017010100000000",
    channel: "stable",
    dependent: {
      origin: "cnunciato",
      name: "ghost,
      version: "1.2.3",
      release: "2017010100000000"
    }
  }
}
@cnunciato
Copy link
Author

cnunciato commented Dec 14, 2017

@chefsalim I totally agree and share that concern, yeah. In some form, we're going to need to be able to get these on demand, or close to that, but it'd be ideal not to have to poll for changes and just be notified on an open connection somehow, via socket or SSE. When we looked at this a while back, the Rust options for both were pretty limited, but that might've changed over the past few months. Definitely something we should investigate though.

@cnunciato
Copy link
Author

That said, we can poll somewhat intelligently, or not even poll at all at first, in order to keep the traffic to a minimum.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment