Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: extend logger and implement open telemetry #37

Closed
2 tasks done
0-vortex opened this issue Oct 2, 2022 · 1 comment · Fixed by #53
Closed
2 tasks done

Feature: extend logger and implement open telemetry #37

0-vortex opened this issue Oct 2, 2022 · 1 comment · Fixed by #53
Assignees
Labels
💡 feature A label to note if work is a feature released

Comments

@0-vortex
Copy link
Contributor

0-vortex commented Oct 2, 2022

Type of feature

🍕 Feature

Current behavior

Our logs are kept on the digital ocean runner history, but our application doesn't save to a file those logs nor does it pass them along to a 3rd party.

While our current pino implementation is very close to hit the milestone, the additional telemetry instrumentation makes this ticket worth a couple of hours extra read.

Suggested solution

While this is a critical issue for our production builds, I will most likely tackle this in the next couple of days, however, for reference, I will leave here a bunch of links that are worth investigating around this particular issue.

Articles I found noteworthy:

Code I discovered in the process:

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@0-vortex 0-vortex added the 💡 feature A label to note if work is a feature label Oct 2, 2022
@0-vortex 0-vortex self-assigned this Oct 2, 2022
@0-vortex 0-vortex added this to the hacktoberfest insights functionality milestone Oct 2, 2022
github-actions bot pushed a commit that referenced this issue Oct 13, 2022
## [1.15.0-beta.1](v1.14.0...v1.15.0-beta.1) (2022-10-13)

### 🤖 Build System

* implement minimal pino logger ([b991dad](b991dad))

### 🍕 Features

* implement http and database logging capabilities ([3a662d0](3a662d0)), closes [#37](#37)

### 🎨 Styles

* add fastify express middleware adapter linting ([fc74284](fc74284))

### 🧑‍💻 Code Refactoring

* correct http timestamp logging ([3782b21](3782b21))
* correct type dependencies ([9895327](9895327))
* improve sql logging and slow query timeout ([5385049](5385049))
* make logging beautiful everywhere ([99ac2c6](99ac2c6))
github-actions bot pushed a commit that referenced this issue Oct 17, 2022
## [1.15.0](v1.14.0...v1.15.0) (2022-10-17)

### 🤖 Build System

* implement minimal pino logger ([b991dad](b991dad))

### 🍕 Features

* implement http and database logging capabilities ([3a662d0](3a662d0)), closes [#37](#37)

### 🧑‍💻 Code Refactoring

* correct http timestamp logging ([3782b21](3782b21))
* correct type dependencies ([9895327](9895327))
* improve sql logging and slow query timeout ([5385049](5385049))
* make logging beautiful everywhere ([99ac2c6](99ac2c6))

### 🎨 Styles

* add fastify express middleware adapter linting ([fc74284](fc74284))
* remove extraneous console.logs ([e3d7bd7](e3d7bd7))
@github-actions
Copy link

🎉 This issue has been resolved in version 1.15.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 feature A label to note if work is a feature released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant