You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
dns.resolveMx returns invalid result in Node <22 #53708
I am able to reproduce this on freebsd (13.3) with blek.cz and visa.com but not gmail.com and other domains however unlike the above test it also is an issue in v22.3.0 installed from ports.
I verified that it was fixed by #53155 in Node.js 22.3.0.
Probably due to this (from c-ares changelog):
Due to a logic flaw dns name compression writing was not properly implemented
which would result in the name prefix not being written for a partial match.
This could cause issues in various record types such as MX records when using
the deprecated API. Regression introduced in 1.28.0. Issue #757
Activity
JQuags commentedon Jul 4, 2024
I am able to reproduce this on freebsd (13.3) with blek.cz and visa.com but not gmail.com and other domains however unlike the above test it also is an issue in v22.3.0 installed from ports.
targos commentedon Jul 5, 2024
I verified that it was fixed by #53155 in Node.js 22.3.0.
Probably due to this (from c-ares changelog):
avivkeller commentedon Jul 6, 2024
Does that make this is a known limitation?
targos commentedon Jul 6, 2024
No
avivkeller commentedon Jul 6, 2024
So, IIUC the solution would be to backport c-ares/c-ares@d7c14a1?
targos commentedon Jul 6, 2024
No, the solution is to cherry-pick or backport #53155. That's why I added lts-watch- labels to it.
avivkeller commentedon Jul 6, 2024
Oh, thanks, I don't know that much about backporting, thanks for the info!