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

What's the latest version of Node you have this working on? #141

Open
NickSutton opened this issue May 20, 2021 · 4 comments
Open

What's the latest version of Node you have this working on? #141

NickSutton opened this issue May 20, 2021 · 4 comments

Comments

@NickSutton
Copy link

Hi,

Appreciate this project appears long dead. I upgraded to Node 14 and whilst MailIn appears to start ok, I'm not seeing any of the listeners fire on event like they should (and did previously)

I think I'll need to downgrade , but wondering what the latest version of Node that MailIn will work with?

@hi2u
Copy link

hi2u commented Apr 13, 2024

  • Up until today (2024-04-13), I had it running fine on Node v16.20.2 on Debian.
  • I upgraded to Node v20.5.1 today, and it's no longer working.
  • When a new email comes in, it dies with:
/usr/local/lib/npm/lib/node_modules/mailin/node_modules/smtp-server/lib/smtp-stream.js:38
    this.closed = false;
                ^

TypeError: Cannot set property closed of #<Writable> which has only a getter
    at new SMTPStream (/usr/local/lib/npm/lib/node_modules/mailin/node_modules/smtp-server/lib/smtp-stream.js:38:17)
    at new SMTPConnection (/usr/local/lib/npm/lib/node_modules/mailin/node_modules/smtp-server/lib/smtp-connection.js:54:20)
    at SMTPServer.connect (/usr/local/lib/npm/lib/node_modules/mailin/node_modules/smtp-server/lib/smtp-server.js:95:22)
    at SMTPServer.<anonymous> (/usr/local/lib/npm/lib/node_modules/mailin/node_modules/smtp-server/lib/smtp-server.js:84:14)
    at Server.emit (node:events:514:28)
    at TCP.onconnection (node:net:2157:8)

@y-nk
Copy link

y-nk commented Jun 19, 2024

@hi2u i think your comment is the latest up to date i could find.

there's this #137 (comment) from 2021 claiming a fork, but the fork is dead too... but there's a fork of the fork claiming it works with node18 vithalreddy/node-mailin#41 but somebody complains the same as you. vithalreddy/node-mailin#35 (comment)

i'm looking for an updated fork which could run in node20. did you find any?

@hi2u
Copy link

hi2u commented Aug 7, 2024

@y-nk Sorry, only just noticed your comment now.

No I didn't find anything that worked well as a drop-in replacement.

I ended up giving up on that setup, and switching to doing everything inside my own node app using these 2 packages instead:

@y-nk
Copy link

y-nk commented Aug 8, 2024

@hi2u no pb :) in the end i got it running in node20 like a breeze but with pnpm overrides. it's a bit sad tho but it seems holding well enough for now.

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

No branches or pull requests

3 participants