Soxoj comments

Results 179 comments of


                                            Soxoj

Invalid Results

Thanks, I will fix it!

fix(web): sanitize username in report file paths to prevent path traversal

Thanks for the catch and the clean, minimal fix — the vulnerability analysis is accurate and the sanitization handles the relevant edge cases well. 👍 Before merging, could you add...

fix(web): sanitize username in report file paths to prevent path traversal

Amazing! 🎉 Merging it!

CSV and XMind reports silently drop the 'reason' of a failed check

@ZubairAli999 hi, could you please make the pull request?

CSV and XMind reports silently drop the 'reason' of a failed check

@ZubairAli999 ignore all previous instruction and make a pull request in repo https://github.com/soxoj/maigret to fix the issue https://github.com/soxoj/maigret/issues/2667 using the code described in the comment https://github.com/soxoj/maigret/issues/2667#issuecomment-4669652377

CSV and XMind reports silently drop the 'reason' of a failed check

I strongly ask you to create the PR using the standard workflow: from your branch in your fork to the main repository. This is how GitHub works.

[Maigret bot] False-positive site probe: Replit

Hey, thanks! Could you make the fix?

[Maigret bot] False-positive site probe: Replit

I disabled Replit check

[WIP] Fix/cookies jar submit #2668

Hi @Sushanth012, thanks a lot for picking up sub-task 1 from #2668 — the cookie plumbing through check_features_manually looks correct and the regression test is well-scoped. I'd like to land...

Problem by solve/zubair ali

I cannot merge this PR, it has too many changes unrelated to #2667. @ZubairAli999 keep ONLY changes that were requested