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

[Bug]: Http logger references the deprecated winstond package #2544

Closed
Dimava opened this issue Feb 11, 2025 · 2 comments
Closed

[Bug]: Http logger references the deprecated winstond package #2544

Dimava opened this issue Feb 11, 2025 · 2 comments

Comments

@Dimava
Copy link

Dimava commented Feb 11, 2025

🔎 Search Terms

http transport, docs, winstond

The problem

I was trying to get something to test HTTP transport and got the recommended by docs winstond package which immediately crashed because of not being compatible with modern node

TypeError: Class constructor Logger cannot be invoked without 'new'
    at exports.Server (\node_modules\winstond\lib\winstond\server.js:15:18)
    at new exports.Server (\node_modules\winstond\lib\winstond\http\index.js:20:19)
    at exports.createServer (\node_modules\winstond\lib\winstond\http\index.js:13:10)

Please either update winstond to a working version or update docs linking to some other thing that can be used to test HTTP transport

What version of Winston presents the issue?

docs, winstond@latest

What version of Node are you using?

v20+

If this worked in a previous version of Winston, which was it?

No response

Minimum Working Example

require('winstond').http.createServer()

Additional information

No response

@DABH
Copy link
Contributor

DABH commented Feb 11, 2025

This is the wrong repo for this issue - https://github.com/winstonjs/winstond would be the right one - but given that no updates have been made to winstond in 13 years, it's safe to say that particular sub-project is not maintained.

I'll probably go ahead and mark that repo as archived so we don't hit issues with this in the future.

@DABH DABH closed this as completed Feb 11, 2025
@Dimava
Copy link
Author

Dimava commented Feb 11, 2025

🤦‍♂ I've explained it poorly

My main problem is

The `Http` transport is a generic way to log, query, and stream logs from an arbitrary Http endpoint, preferably [winstond][1]. It takes options that are passed to the node.js `http` or `https` request:

this repo's docs linking a non-working package

Now docs mention an archieved package 😂

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

No branches or pull requests

2 participants