-
-
Notifications
You must be signed in to change notification settings - Fork 252
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
update version in the code #36
Comments
The same issue, but, after check I see also outdated versions (at least for Drupal)
v1.33.7 Functionality available for 'drupal':
Versions should be 8.6.13, 7.65 Trying to update:
Still the same version + outdated Drupals 7&8´s |
Hi @blshkv , It always report 1.3.37 because it is hard-coded here. This is because I had an issue where the tool became completely broken because the code that attempted to read the version wasn't able to read the changelog. I would consider a PR for this if you provide one, otherwise it is just an aesthetic issue so it would not be a high priority for me atm. Thanks! |
hi @masterwebsk This occurs because a couple of new drupal versions have been released and I haven't been able to implement a new hash for them. I will do my best to get this updated asap however at the moment I am super swamped so a PR would be better if you're in a hurry. droopescan has the ability to update itself, you'd need to clone the master branch of the repo and then run Keep in mind this is a development tool not meant for the end user. It clones Thanks! |
@droope thank you for your replies. How about go KISS, hardcode the correct version and update it every time if you can't find a better solution? BTW, We (at Pentoo) install all tools as system-wide packages and expect users to run them under regular account (if possible), that means self-update feature will not work. In fact, we do not allow to update tools itself (such as binaries, configuration etc) because it is more like a malicious behaviour. All system-wide files must be installed and be under control of a package manager. Please keep it in mind while further implementing this feature. |
hi @blshkv Thanks for your feedback! I'll definitely keep in mind the idea of hard-coding the version. At the moment I use release scripts that don't take the version into account, but I will consider the option in the future. Thanks! |
I've given this more thought and after thinking about it hard-coding the version would cause more issues that it would solve, because even though I am as thorough as I can when releasing new versions I can be liable to forget to update the code as it is. I think the proper solution is to either a) update the release script so that it modifies the version, or I am open for a PR for this if anybody is willing to do it. I am busy with other projects at the moment and I am limiting myself to base maintenance for this project. Thanks! |
I have installed the latest version, however the scanner displays
v1.33.7
during runtimehttps://github.com/droope/droopescan/blob/master/dscan/common/functions.py#L217
The text was updated successfully, but these errors were encountered: