Skip to content

Fix analytics timeout #155

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

Merged
merged 1 commit into from
Nov 26, 2014
Merged

Conversation

rosen-vladimirov
Copy link
Contributor

Change dependency of xmlhttprequest to use Telerik fork.

Update common lib, where the following change is applied: EqatecMonitor.js call setTimeout of XMLHttpRequest with default value - 10000 ms. Add SetRequestTimeout method to EqatecMonitor which sets global timeout variable (currently this method is not used).

The timeout is required in order to prevent console hanging when analytics cannot send the request.

Fixes #153

Change dependency of xmlhttprequest to use Telerik fork.

Update common lib, where the following change is applied: EqatecMonitor.js call setTimeout of XMLHttpRequest with default value - 10000 ms. Add SetRequestTimeout method to EqatecMonitor which sets global timeout variable (currently this method is not used).

The timeout is required in order to prevent console hanging when analytics cannot send the request.
@ns-bot
Copy link

ns-bot commented Nov 25, 2014

Test PASSed.

@Fatme
Copy link
Contributor

Fatme commented Nov 26, 2014

👍

rosen-vladimirov added a commit that referenced this pull request Nov 26, 2014
@rosen-vladimirov rosen-vladimirov merged commit e9520b3 into master Nov 26, 2014
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/fix-analytics-timeout branch November 26, 2014 12:21
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.

CLI becomes unresponsive when the analytics is on and the user is not logged in ironport
3 participants