LasCC / HackTools

The all-in-one browser extension for offensive security professionals 🛠
https://hacktools.sh
5.84k stars 651 forks source link

npm run build Can't resolve bugs #93

Closed supernaruto16 closed 3 years ago

supernaruto16 commented 3 years ago

ERROR in ./src/components/FeedRSS.js Module not found: Error: Can't resolve './rss/Cve' in '/home/madscientist/Documents/CTFTools/chrome-extensions/Hack-Tools/src/components' @ ./src/components/FeedRSS.js 8:0-28 103:24-27 116:26-29 @ ./src/components/LayoutApp.js @ ./src/App.js

+ Then i found it's caused by 2 typing errors

./src/components/LayoutApp.js:19: import FileTransfer from "./file_transfer/File_transfer"; => should be "./file_transfer/file_transfer" ./src/components/FeedRSS.js:8: import cve from './rss/Cve'; => should be "./rss/cve"

+ Btw, the command to build locally with npm in README.md should be

npm install && npm run build


+ Hope u will fix those on git version
jayluxferro commented 3 years ago

Hi @supernaruto16 , I did npm run build and I got no errors. What OS and npm version are you using? My environment is macOS and the npm version I'm using is 6.14.10

supernaruto16 commented 3 years ago

Hi, my environment is Ubuntu 20.04.1 and npm 6.14.11, but i don't think environment is related to the bugs. I think the reason is syntax error in those js code above.

supernaruto16 commented 3 years ago
jayluxferro commented 3 years ago

I just tested the build on linux and figured out the issue. npm imports on macOS seems to be case insensitive, which makes it work. You can submit a pull request for the bug and also the README. Just rename the files to Cve and File_transfer respectively.

Thanks.