Skip to content
This repository has been archived by the owner on Nov 5, 2021. It is now read-only.

RequestIdentifier middleware #17

Open
3 tasks
ebebbington opened this issue Jul 15, 2020 · 0 comments
Open
3 tasks

RequestIdentifier middleware #17

ebebbington opened this issue Jul 15, 2020 · 0 comments
Labels
Good First Issue This item is good for newcomers Priority: Low Type: Enhancement This item introduces a new feature

Comments

@ebebbington
Copy link
Member

ebebbington commented Jul 15, 2020 •
edited by crookse
Loading

Summary

What:

Add requestUID (or similar) middleware. Take inspiration from: http://expressjs.com/en/resources/middleware/connect-rid.html

Why:

It'd be nice to track requests by an ID for debugging purposes. Also, a request ID would be useful in logs -- filtering logs by request ID.

Acceptance Criteria

  • Develop middleware -- taking inspiration from connect-rid
  • A VERY DETAILED README (see other middleware READMEs as examples)
  • Write unit tests

Taking on this issue? Read this before you get started.

If you take on this issue and the acceptance criteria does not make sense, post comments on this issue with your questions. Also, you can always hop in the Discord channel at https://discord.gg/c5R3sYuR to get help quickly.

Please read the Contributing Guidelines before starting this issue.
@ebebbington ebebbington added the Type: Enhancement This item introduces a new feature label Jul 15, 2020
@player-4 player-4 added the Good First Issue This item is good for newcomers label Jul 16, 2020
@crookse crookse changed the title Add Request ID Middleware feat: RequestIdentifier middleware Oct 31, 2020
@crookse crookse changed the title feat: RequestIdentifier middleware RequestIdentifier middleware Nov 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Good First Issue This item is good for newcomers Priority: Low Type: Enhancement This item introduces a new feature
Milestone
No milestone
Development

No branches or pull requests
3 participants
@crookse @ebebbington @player-4