-
Notifications
You must be signed in to change notification settings - Fork 158
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
Investigate report about failed installs #125
Comments
So, here's the status on this bug. The root cause is a 3rd party library stickytape. It's a known issue and we actually don't use this library at all in our current master & dev branches. The new code was sent to PortSwigger (https://github.com/PortSwigger/inql/commits/master), but the BApp Store release is being delayed due to some issues that we can't influence. The code in PortSwigger's repo matches InQL release 5.0.2, so hopefully that's the version that should appear in the BApp Store any time soon. We're working towards the next release 5.1 in our dev branch, so that might occur sooner than the BApp Store release. I'm leaving this issue opened as a reminder to check Windows compatibility before the 5.1 release, but I'm afraid the original issue with the BApp Store version will remain unsolved until PortSwigger releases public update. |
There is a report on Twitter about InQL failing during installation: https://twitter.com/irsdl/status/1698658521166815250
Post mentions Windows, but it could be a red herring. #28 is almost definitely unrelated to this as it was fixed a while ago. I've seen somewhat similar bug where updates were broken due to changes in how the settings were persisted in different versions. However this should have been addressed by removing old style settings: https://github.com/doyensec/inql/blob/master/python/inql/extender.py#L73
This should be investigated further before the 5.1 release.
The text was updated successfully, but these errors were encountered: