FormidableLabs / FormidableLabs/pino-lambda

Doesn't work with buffered logging when multiple JSONs could be put into on chunk separated by newline

Ouverte
#54 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
bug
Langage dominant
TypeScript
Étoiles
135
Forks
14
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

### Is there an existing issue for this?

- [X] I have searched the existing issues

### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

### Code Sandbox link

_No response_

### Bug report

```markdown
I would suggest to convert the library to pino transport using https://github.com/pinojs/pino-abstract-transport. Now it is very easy to crash a lambda after `JSON.parse` in the destination.ts is trying to parse a chunk containing smth like

{...}\n{...}
```
```

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par lire destination.ts et la documentation de pino-abstract-transport afin de comprendre comment les chunks mis en mémoire tampon sont traités. Reproduisez l’échec avec un chunk contenant deux objets JSON séparés par un saut de ligne, puis vérifiez que le logging mis en mémoire tampon ne plante plus lors de son traitement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
aws, typescript
Domaine
backend, cloud
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.