Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

_ #355

Closed
niftylettuce opened this issue Aug 2, 2017 · 4 comments
Closed

_ #355

niftylettuce opened this issue Aug 2, 2017 · 4 comments

Comments

@niftylettuce
Copy link

niftylettuce commented Aug 2, 2017 •
edited

No description provided.

@benvinegar
Copy link
Contributor

Are you seeing this with a vanilla express app?

@benvinegar
Copy link
Contributor

In general though, this package is really really spaghetti code... 🍝

I'm assuming you're referring raven-node? Do you have any specific things you think could be improved, and how?

@benvinegar
Copy link
Contributor

There's just so much bloat I feel

If you can highlight something a little more actionable I'd be happy to have a conversation around it.

@benvinegar
Copy link
Contributor

It seems we need to create a new ticket around pm2. Keeping this open in the meantime.

@benvinegar benvinegar reopened this Aug 6, 2017
@niftylettuce niftylettuce changed the title req is not automatically getting set for non-500 status error codes _ Aug 7, 2017
@dcramer dcramer closed this as completed Aug 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@benvinegar @dcramer @niftylettuce