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
Updating without administrative privileges fails with some antivirus apps running in background #2352
Comments
Attachment nvda(error_downloading_update).log added by ondrosik on 2012-05-17 09:25 |
Comment 1 by jhomme (in reply to comment description) on 2012-05-17 10:41 Hi, Jim
|
Comment 2 by jteh on 2012-05-17 21:27 Installing the update does and always will require administrator privileges, but you should be prompted for administrator access when installation begins. |
Comment 3 by pvagner on 2012-05-23 06:30
I'm unable to give further suggestion, here is the relevant part from the log he has attached IO - tones.beep (11:09:43): |
Comment 4 by jteh on 2012-05-23 07:37 |
Comment 5 by ondrosik (in reply to comment 4) on 2012-05-23 07:46 |
Comment 6 by pvagner on 2012-05-23 08:16 |
Comment 7 by ondrosik (in reply to comment 6) on 2012-05-23 08:23 |
Comment 8 by ondrosik (in reply to comment 7) on 2012-05-25 06:01 |
Comment 9 by pvagner on 2012-05-25 11:48 |
Comment 10 by pvagner on 2012-05-25 12:01 |
Comment 11 by pvagner on 2012-05-25 13:20 |
Comment 12 by pvagner on 2012-05-25 15:06 |
Comment 13 by pvagner on 2012-05-25 15:07 |
Comment 14 by pvagner on 2012-05-25 18:29 |
Comment 15 by pvagner on 2012-06-04 10:02 |
Comment 16 by ateu on 2012-06-04 14:53 I use windows 32 bit and avast 7 and it's not possible update NVDA. Strangely, when I run ANOTHER computer with windows 64 bit and microsoft secure essential, update feature works fine. |
Comment 17 by jteh (in reply to comment 16) on 2012-06-04 22:54
Is it the download or installation that fails?
It's possibly logged as debug warning. Please set your log level to debug warning and report. |
Comment 18 by ateu on 2012-06-05 01:35 The error appears when the update is being downloaded. |
Attachment nvda_debug_warning.log added by ateu on 2012-06-05 10:07 |
Comment 19 by ateu on 2012-06-05 10:11 |
Comment 20 by jteh (in reply to comment 15) on 2012-06-05 11:31
It looks like Avast is blocking the download. The Avast window is probably requesting confirmation. I don't think there's anything we can do about this. I'm guessing this is working as intended as far as Avast is concerned. Do you have any reason to suspect differently? Replying to pvagner:
Is the error the same; i.e. timeout? |
Comment 21 by pvagner (in reply to comment 20) on 2012-06-05 12:45
Yes it still appears the same. |
Comment 22 by ondrosik (in reply to comment 21) on 2012-06-05 15:38 |
Comment 23 by jteh (in reply to comment 22) on 2012-06-05 22:37
Also, is the rest the same; i.e. still gets to 99% then fails? Replying to ondrosik:
It's standard http, so port 80. |
Comment 24 by ondrosik (in reply to comment 23) on 2012-06-06 12:06 |
Comment 25 by jteh on 2012-06-06 12:26 Peter, can you please try this in the Python console before downloading an update:
Does this help? You can also try increasing this to higher values. |
Comment 26 by ondrosik (in reply to comment 25) on 2012-06-06 17:38 |
Comment 27 by surveyor (in reply to comment 26) on 2012-06-06 17:47
The same problem for me. tried to update from main 5208 to 5213 with http enabled in eset and failed again. |
Comment 28 by ondrosik (in reply to comment 27) on 2012-06-07 09:00
|
Comment 29 by ondrosik (in reply to comment 28) on 2012-06-07 10:05 |
Comment 30 by jteh (in reply to comment 28) on 2012-06-07 11:53
50 seconds is a pretty long timeout.
I wonder whether it is scanning the file just as it finishes downloading but before it gives NVDA the last chunk. That's pretty stupid/weird behaviour. :)
Any idea how long between the last progress beep and the success dialog? Was it really 50 seconds? Development note: We don't want to set the default socket timeout too high, as otherwise, update checks will take forever to fail. However, we can change the timeout for the download itself using remote.fp._sock.settimeout. This is ugly as all hell, but it may be the only way. |
Comment 31 by ondrosik (in reply to comment 30) on 2012-06-07 11:59 |
Comment 32 by jteh (in reply to comment 31) on 2012-06-07 12:30
Wow. So we really do have to set it pretty high. |
Comment 33 by ondrosik (in reply to comment 32) on 2012-06-07 16:48 |
Comment 34 by surveyor on 2012-06-07 18:06 |
Comment 35 by pvagner (in reply to comment 30) on 2012-06-07 22:14
I am afraid this exactly as you have described it is happening. I can notice it even with firefox wget and all other apps downloading some files over http. It is stuck on 99 or 100 percent for a while. Most likelly eset is scanning the file. I've tried another idea, saving the file into a different file where there is no .exe extension because many downloaders are doing this but that doesn't help either. The only working solution so far is higher timeout.
on my slow laptop it really takes a while. I think it's less than 50 secs but still perhaps just in case we might use 60 seconds if we'll add the hack mentioned above. |
Comment 36 by jteh on 2012-06-07 23:44 |
Comment 37 by ondrosik (in reply to comment 35) on 2012-06-08 05:01 Replying to pvagner:
|
Comment 38 by jteh on 2012-06-08 05:15 |
Comment 39 by ondrosik on 2012-07-23 08:43 |
Attachment eco-neti-pot.jpg added by Smartmil8 on 2013-02-17 19:43 |
…. Re nvaccess#9818. In Python 3, timeout is specified as part of urllib.request.urlopen function. Therefore move the code around and the accompanying comment for nvaccess#2352.
* Urllib/Python 3: import urllib.request and urllib.parse, not just urllib. Re #9818. If one imports urllib in Python 3, only the bare bone package will be imported - no parsing, no errors, no opener, etc. Thus import both urllib.request and urllib.parse 9although just importing urllib.request will then load other parts of this package, imported both in update check module for sake of completeness). * Urllib/Python 3: urllib.urlencode -> urllib.parse.urlencode. Re #9818. * Urllib/Python 3: urllib.urlopen -> urllib.request.urlopen for ones without timeout. Re #9818. * Urllib/Python 3: urllib.urlopen -> urllib.request.urlopen for certificates. Re #9818. * Urllib/Python 3: urllib.urlopen -> urlib.request.urlopen with timeout. Re #9818. In Python 3, timeout is specified as part of urllib.request.urlopen function. Therefore move the code around and the accompanying comment for #2352.
Reported by ondrosik on 2012-05-17 09:24
When running NVDA without administration privileges, update process fail. With "error downloading update"
Step by step:
i tried this with administration privileges successfully. It will be better to inform muser that it is necessary to log on as an administrator or show the password dialog as for example during copiing settings to logon screen.
The text was updated successfully, but these errors were encountered: