-
Notifications
You must be signed in to change notification settings - Fork 8
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
Haitham dev #7
Haitham dev #7
Conversation
src/main/api/project.ts
Outdated
export function createJsonFile(name: string, domain: string) { | ||
const obj = { | ||
name, | ||
domain, | ||
subFinder: defaultToolObj(), | ||
liveDomains: defaultToolObj(), | ||
screenWin: defaultToolObj(), | ||
archive: defaultToolObj(), | ||
waybackurls_js: defaultToolObj(), | ||
waybackurls_parameter: defaultToolObj(), | ||
subFinder: defaultJobDetails, | ||
liveDomains: defaultJobDetails, | ||
screenWin: defaultJobDetails, | ||
archive: defaultJobDetails, | ||
waybackurls_js: defaultJobDetails, | ||
waybackurls_parameter: defaultJobDetails, | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is wrong structure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still need to be inside recon objcet
src/main/db/connect.ts
Outdated
@@ -11,6 +11,8 @@ export interface ProjectDetails { | |||
screens?: JobDetails; | |||
params?: JobDetails; | |||
liveDomains?: JobDetails; | |||
archive?: JobDetails; | |||
waybackurls_js?: JobDetails; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't use snake case here
wayBackUrlsJs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its shit
src/main/recon/httpx.ts
Outdated
await db.update({ | ||
recon: { | ||
screens: { | ||
result: parseInt(domainsFound, 10), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make it '' empty string
src/main/recon/waybackurls.ts
Outdated
const res = await execAsync(command); | ||
const domainsFound = resultFromStd(res.stderr, /\bFound (\d+) subdomains?/); | ||
const db = connectJson(path.join(`${outputDir}/details.json`)); | ||
await db.update({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this regex work ?!
src/main/recon/waybackurls.ts
Outdated
const res = await execAsync(command); | ||
const domainsFound = resultFromStd(res.stderr, /\bFound (\d+) subdomains?/); | ||
const db = connectJson(path.join(`${outputDir}/details.json`)); | ||
await db.update({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
src/main/recon/waybackurls.ts
Outdated
const res = await execAsync(command); | ||
const domainsFound = resultFromStd(res.stderr, /\bFound (\d+) subdomains?/); | ||
const db = connectJson(path.join(`${outputDir}/details.json`)); | ||
await db.update({ | ||
recon: { | ||
liveDomains: { | ||
result: parseInt(domainsFound, 10), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
No tool outputs the counter but subfinder , the live domains outputs the domains themself i tryed to count them but already all method doesn't give the output in the run time ,onece i close the app the i find the output files i tryed to use the debug tool the app doesn't go after the awit liveDomian() at all i tryed to put the subfinder command instated of the livedomaind command in same function of the liveDomain method it works normalygit add . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the default values are not initiated, this is a good way, but need to work on the above requests
src/main/main.ts
Outdated
ipcMain.on('httpx-screens', async (event, args) => { | ||
const { projectName } = args[0]; | ||
const res = screenwin(`${PROJECT_DIR}/${projectName}`); | ||
return res; | ||
event.returnValue = res; | ||
}); | ||
ipcMain.handle('waybackurls-archive', async (event, args) => { | ||
ipcMain.on('waybackurls-archive', async (event, args) => { | ||
const { projectName } = args[0]; | ||
const res = wwayback(`${PROJECT_DIR}/${projectName}`); | ||
return res; | ||
event.returnValue = res; | ||
}); | ||
ipcMain.handle('waybackurls-js', async (event, args) => { | ||
ipcMain.on('waybackurls-js', async (event, args) => { | ||
const { projectName } = args[0]; | ||
const res = fetchJs(`${PROJECT_DIR}/${projectName}`); | ||
return res; | ||
event.returnValue = res; | ||
}); | ||
ipcMain.handle('waybackurls-parameter', async (event, args) => { | ||
ipcMain.on('waybackurls-parameter', async (event, args) => { | ||
const { projectName } = args[0]; | ||
const res = parameter(`${PROJECT_DIR}/${projectName}`); | ||
return res; | ||
event.returnValue = res; | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The handler are not correct
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what are the correct ?
it already works the problem in the tools methos ,it is not here .
src/main/recon/httpx.ts
Outdated
const db = connectJson(path.join(`${outputDir}/details.json`)); | ||
await db.update({ | ||
recon: { | ||
liveDomains: { | ||
result: parseInt(domainsFound, 10), | ||
result: domainsFound, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This return an array, so it should be array.length()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually do it look to teh fun imp
You need to pass the stderr not stdout as these tools do print in the error stream |
"You need to pass the stderr not stdout as these tools do print in the error stream" |
can you explain what issue you have !? |
No description provided.