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

Timeout while tracking multiple events at the same time #27

Closed
brownsugar opened this issue Dec 4, 2016 · 3 comments
Closed

Timeout while tracking multiple events at the same time #27

brownsugar opened this issue Dec 4, 2016 · 3 comments
Labels
Milestone

Comments

@brownsugar
Copy link

There is a problem while I tried to track some events.

private void TrackSomething() {
    Piwik piwik = new Piwik(1, "XXXX");
    piwik.doTrackEvent("XX", "XX", "XX"); // OK
    piwik.doTrackEvent("XX", "XX", "XX"); // OK
    piwik.doTrackEvent("XX", "XX", "XX"); // Timeout
}

And I found there's weird bug at sendRequest() funtion,
it stucked at the last line of the function,
after I changed it to:

using (HttpWebResponse response = (HttpWebResponse)request.GetResponse()) {
    return response;
}

It works fine now.

@opcon
Copy link
Contributor

opcon commented Jan 8, 2017

What platform are you running your application on?

I encountered this issue with my application on Mono, but not on Windows.

I think this is partially an issue on Mono's side, where it can't handle many undisposed HttpWebResponses. I fixed it by disposing the HttpWebResponses, but in my code rather than Piwik's

@brownsugar
Copy link
Author

Hi, I'm on Windows, and I tested on several Windows platforms too.

@julienmoumne julienmoumne changed the title Timeout while tracking multiple events at the same time. [API Change] Timeout while tracking multiple events at the same time Jan 11, 2017
@julienmoumne
Copy link
Member

fixed in #30

@julienmoumne julienmoumne added this to the v3.0.0 milestone Jan 11, 2017
@julienmoumne julienmoumne changed the title [API Change] Timeout while tracking multiple events at the same time Timeout while tracking multiple events at the same time Mar 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants