Skip to content
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

fix(external_tools): incorrect "update available" strings #882

Conversation

AnonymousWP
Copy link
Contributor

@AnonymousWP AnonymousWP commented Apr 17, 2023

Fixes

  • Incorrect update string theHarvester: before, it somehow detected version v5, which doesn't even exist. However, the current version ('v3') isn't detected properly and I'm not sure how to solve it. It doesn't even seem to update the tool, but that could also be a detection issue as I said.

Before:

image

image

After:

image

image

Changes

  • Incorrect version number of reNgine at login page

serrapa and others added 30 commits February 3, 2023 09:47
Rebase from yogeshojha/rengine
Fix yogeshojha#802 issue (Assetfinder results not created)
…g-for-one-liners-in-README

847 use inline code formatting for one liners in readme
The incorrect container names caused that the containers, volumes and networks weren't being removed as it requires the containers to be stopped
ops(`uninstall.sh`): fix incorrect container names
…e-string

fix(versioning): incorrect current release at login page
…e-string

fix(external_tools): incorrect theHarvester version
…e-string

fix(external_tools): specify command and path
@psyray
Copy link
Contributor

psyray commented Sep 9, 2023

Why don't use v\d.* as regex for naabu?

Because Naabu doesn't return the v

naabu -version

                  __
  ___  ___  ___ _/ /  __ __
 / _ \/ _ \/ _ \/ _ \/ // /
/_//_/\_,_/\_,_/_.__/\_,_/

		projectdiscovery.io

[INF] Current Version: 2.1.7

Feel free to improve my bad regex 😉

I don't know why but without the space at the beginning the version returned is v34, but with the space it returned 2.1.7
It's weird ...

Without the space before
image

With the space before
image

@AnonymousWP
Copy link
Contributor Author

I pushed some new commits, but I think some of your external_tools.yaml are incorrect: I believe is_github_cloned should only be true if update_command is done by a git pull and the install_command is git clone <repo_url>. Now you've changed that are installed by Go to true as well.

@psyray Can you let me know what you think of it?

@AnonymousWP
Copy link
Contributor Author

I don't think it has the desired result, lol. A lot still can't recognise the version, and if it can, the version number remains the same after updating. It's very annoying.

image
image

@AnonymousWP
Copy link
Contributor Author

Lot of conflicts due to rebase and now being outdated.

@psyray
Copy link
Contributor

psyray commented Nov 21, 2023

We could close this one, I have reworked this part on PR #995 and it has been merged

@psyray psyray closed this Nov 21, 2023
@AnonymousWP AnonymousWP deleted the fix-incorrect-update-available-string branch November 21, 2023 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants