nodertc/stun

`validateMessageIntegrity` fails

vibornoff opened this issue · 1 comments

vibornoff commented

Take a look to

stun/src/node_modules/lib/validate.js

Line 53 in 3db7f5d

fingerprintLength = 4 + fingerprintAttr.value.length

fingerprintLength value becomes NaN; should be change to

fingerprintLength = 8 // 4 bytes of attribute header + 4 bytes of CRC32
reklatsmasters commented

Thanks, missed test fingerprint + message integrity