3 ms·
Love Socket! A lot of folks (think most) were loading the compromised package through another package, @ledgerhq/connect-kit-loader [1], via a CDN call [2]. Wou
by meagher 3y ago
Love Socket! A lot of folks (think most) were loading the compromised package through another package, @ledgerhq/connect-kit-loader [1], via a CDN call [2]. Would be great if Socket could pick up on this because Socket's @ledgerhq/connect-kit-loader page [3] doesn't include any warning.
[1]: https://socket.dev/npm/package/@ledgerhq/connect-kit-loader https://socket.dev/npm/package/@ledgerhq/connect-kit-loader
[2]: https://github.com/LedgerHQ/connect-kit/blob/main/packages/connect-kit-loader/src/index.ts#L83 https://github.com/LedgerHQ/connect-kit/blob/main/packages/c...
[3]: https://socket.dev/npm/package/@ledgerhq/connect-kit-loader/overview/1.1.2 https://socket.dev/npm/package/@ledgerhq/connect-kit-loader/...
- feross 3y agoThanks for reporting this. Fixed: https://socket.dev/npm/package/@ledgerhq/connect-kit-loader/overview/1.1.2 https://socket.dev/npm/package/@ledgerhq/connect-kit-loader/... We don't currently detect 'implicit dependencies' loaded via CDN URLs, though we'll look into what it would take to support this.
- meagher 3y agoSounds good. Excited for this and all future Socket improvements!