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

fix(node): TLSSocket._start #19996

Closed
wants to merge 3 commits into from

Conversation

bartlomieju
Copy link
Member

@bartlomieju bartlomieju commented Jul 31, 2023

Closes #19983
Closes #18303
Closes #16681
Closes #19978

@bartlomieju bartlomieju marked this pull request as draft July 31, 2023 13:29
@@ -19,6 +19,7 @@ import {
constants as TCPConstants,
TCP,
} from "ext:deno_node/internal_binding/tcp_wrap.ts";
import { kHandle } from "ext:deno_node/internal/stream_base_commons.ts";
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bartlomieju can you please also fix it as 0.177.2 for esm.sh? thanks!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bartlomieju can you please also fix it as 0.177.2 for esm.sh? thanks!

I don't think so, it will require a lot of changes and most likely be using APIs not available in 0.177.2.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense

@bartlomieju
Copy link
Member Author

Reproduction

import nodemailer  from "nodemailer";

nodemailer.createTestAccount((err, account) => {
    if (err) {
        console.error('Failed to create a testing account. ' + err.message);
        return process.exit(1);
    }

    console.log('Credentials obtained, sending message...');

    // Create a SMTP transporter object
    let transporter = nodemailer.createTransport({
        host: account.smtp.host,
        port: account.smtp.port,
        secure: account.smtp.secure,
        auth: {
            user: account.user,
            pass: account.pass
        }
    });

    console.log("transporter", transporter);

    // Message object
    let message = {
        from: 'Sender Name <sender@example.com>',
        to: 'Recipient <recipient@example.com>',
        subject: 'Nodemailer is unicode friendly ✔',
        text: 'Hello to myself!',
        html: '<p><b>Hello</b> to myself!</p>'
    };
    console.log("before send");
    transporter.sendMail(message, (err, info) => {
        console.log("after send");
        if (err) {
            console.log('Error occurred. ' + err.message);
            return process.exit(1);
        }

        console.log('Message sent: %s', info.messageId);
        // Preview only available when sending through an Ethereal account
        console.log('Preview URL: %s', nodemailer.getTestMessageUrl(info));
    });
});

@bartlomieju
Copy link
Member Author

Closing in favor of #20120

@bartlomieju bartlomieju deleted the tlssocket_start branch August 10, 2023 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants