51degrees Issue Tracker Rss Feed

Commented Issue: Memory Leak [9643]

davidke2 — Thu, 20 Sep 2012 17:27:32 GMT

We are having an issue with a memory leak after putting on version 2.1.10.2. We rolled back to ver 2.1.4.9. Is there a known issue with 2.1.10.2 or maybe we can configure something differently? Enclosed is the graph of recorded private bytes on 2 servers. The red line is from the server where we completely removed the dll at 12:30.

We are using the Lite data only.

We are running IIS6 on Windows 2003 server and .NET 4.0.

Our web.config.
<sectionGroup name="fiftyOne">
<section name="log" type="FiftyOne.Foundation.Mobile.Configuration.LogSection, FiftyOne.Foundation" requirePermission="false" allowDefinition="Everywhere" restartOnExternalChanges="false" allowExeDefinition="MachineToApplication"/>
<section name="redirect" type="FiftyOne.Foundation.Mobile.Configuration.RedirectSection, FiftyOne.Foundation" requirePermission="false" allowDefinition="Everywhere" restartOnExternalChanges="false" allowExeDefinition="MachineToApplication"/>
<section name="detection" type="FiftyOne.Foundation.Mobile.Detection.Configuration.DetectionSection, FiftyOne.Foundation" requirePermission="false" allowDefinition="Everywhere" restartOnExternalChanges="false" allowExeDefinition="MachineToApplication"/>
</sectionGroup>

<fiftyOne>
<log logFile="D:\WebActivity\MobileRedirectLog.txt" logLevel="Info"/>
</fiftyOne>
Comments: ** Comment from web user: davidke2 **

James,

Thanks for the good response. We do have high volume web servers. Also we are running 2.1.10.3 not
2.1.10.2 as in the first post.

I checked the code from ver 2.1.4.9 and it already had both items 1 and 2 set the same as ver 2.1.10.3. I am wondering why we saw the dramatic memory consumption with the new version? Maybe something else changed or could it be that the new ver detects more devices and it excercises the cache more?

Thanks
David

Commented Issue: Memory Leak [9643]

jwrosewell — Thu, 20 Sep 2012 09:22:24 GMT

Thank you for the comprehensive information. There are two areas to investigate.

1. StoreSegmentResults was introduced to improve performance at the expense of memory consumption. Setting this to false will reduce the memory demand over time as less data will be cached. However detection times will increase marginally as more CPU will be used.

2. Look for this line "private static readonly Cache<IDictionary> _cache = new Cache<IDictionary>(60);" if Factory.cs. This tells the cache to store previous matches for 60 minutes. Reducing the number from 60 will reduce memory consumption, particularly on a web site with a lot of traffic.

We are working on an option to replace the C# provider with a C provider which will give us direct control over the memory management and also allow working data to be compiled into the assembly. See http://sourceforge.net/projects/fiftyone-c/ for more information on this project.

Please let me know if the 1st two options help you.

Thanks,

James

Created Issue: Memory Leak [9643]

davidke2 — Tue, 18 Sep 2012 16:51:12 GMT

Closed Issue: High memory usage [9607]

jwrosewell — Mon, 03 Sep 2012 17:18:56 GMT

First, this is a great project. Thanks much.

I'm finding though that the memory demands are huge. From a profiler: 8 Meg of string memory, 1 Meg of byte[]. 5.5 meg for namespaces.

This is pushing my .net4 MVC3 app over the common limit for shared hosting of a number of hosting companies (unfortunately 98% of the memory space is being used by the libraries, not my object -- this being the only external library, the rest from MS).

Is there any way to reduce the memory footprint?
Comments:

See comments on forum.

Commented Issue: High memory usage [9607]

jwrosewell — Mon, 03 Sep 2012 17:16:35 GMT

Here is the response from our forums.

"There is a constant called StoreSegmentResults in the DetectionConstants.cs class which could be set to false. This will reduce memory consumption slightly by not caching some of the matching data but will increase CPU usage marginally.

An alternative which may be less relevant in a shared hosting environment would be to use the C API. C enables the working data to be embedded into the compiled executable reducing the memory foot print, startup time, etc. However this solution is more technically complex than the NuGet install.

Our approach is to ensure the solution is both fast and accurate. We don't include data for devices which we've not seen in the past two years."

Commented Issue: High memory usage [9607]

pkuykendall — Tue, 28 Aug 2012 22:17:12 GMT

No need to reply here. Found your forums and added a comment to an existing thread on the same topic http://51degrees.mobi/Support/Forum/tabid/65/forumid/2/postid/1799/scope/posts/Default.aspx#1799 .

Created Issue: High memory usage [9607]

pkuykendall — Tue, 28 Aug 2012 21:08:34 GMT

Commented Issue: [2.1.4.8] Wrong detection [9481]

dotneter — Fri, 06 Jul 2012 10:11:52 GMT

UserAgent
YandexMarket/1.9-2 (compatible; http://market.yandex.ru)
IsMobileDevice return true.
Also any random UserAgent like safdsa is detected as mobile device. Is there any way to change this behavior,
so IsMobileDevice return false if algorithm is not sure?
Comments: ** Comment from web user: dotneter **

Same problem with
Google-Site-Verification/1.0

Created Issue: [2.1.4.8] Wrong detection [9481]

dotneter — Wed, 04 Jul 2012 13:00:40 GMT

Closed Issue: Make FiftyOne.Foundation.Mobile.Redirection.RedirectModule.IsFirstTime method public? [9387]

jwrosewell — Wed, 20 Jun 2012 14:27:59 GMT

If this method was public then I could use it to do custom redirection in my project like this:

If Request.Browser.IsMobileDevice = True Then
If FiftyOne.Foundation.Mobile.Redirection.RedirectModule.IsFirstTime(HttpContext.Current) Then
Response.Redirect(MobilePageURL)
End If
End If

Is there a reason I should not be doing that? I have recompiled your dll and made the method public but I have not moved the code to a production environment.
Comments:

This change was made in 2.1.6.11.

Closed Issue: inheritInChildApplications cannot be set to false [8728]

jwrosewell — Wed, 20 Jun 2012 14:26:50 GMT

This error occurs because of the way the library accesses the "51Degrees.mobi.config" file.

Steps to reproduce:
1. Create a new MVC3 Application
2. Wrap the <system.web> tag with <location path="." inheritInChildApplications="false">
3. Feel the pain.

Steps to fix:
1. Navigate to FiftyOne.Foundation.Mobile.Configuration.Support.GetConfigurationSectionFromAltConfig
2. Change "fiftyOneConfig = ConfigurationManager.OpenMappedExeConfiguration(configFileMap, ConfigurationUserLevel.None);"
to
try
{
fiftyOneConfig = ConfigurationManager.OpenMappedExeConfiguration(configFileMap, ConfigurationUserLevel.None);
}
catch
{ // if a <location> wraps the system.web we'll end up here, so use the WebConfigurationManager
fiftyOneConfig = WebConfigurationManager.OpenWebConfiguration(Constants.CONFIG_FILENAME);
}
Comments:

Version 2.1.6.11 resolved this issue in our test environment.

Commented Issue: inheritInChildApplications cannot be set to false [8728]

jwrosewell — Wed, 20 Jun 2012 14:26:17 GMT

Version 2.1.6.11 resolved this issue in our test environment.

Commented Issue: inheritInChildApplications cannot be set to false [8728]

jwrosewell — Mon, 11 Jun 2012 12:52:53 GMT

We have made a change to the support.cs class which has addressed the problem as experienced in our test environment. We expect to release this to codeplex this week in the next versions beta.

Reopened Issue: inheritInChildApplications cannot be set to false [8728]

jwrosewell — Mon, 11 Jun 2012 12:11:05 GMT

Seeing comments this issue needs to be reopened.

Closed Issue: isTablet does not work [9335]

jwrosewell — Mon, 11 Jun 2012 12:09:08 GMT

I tried out to redirect my page, if it is visited by a tablet.

This could does not work (also tried it with property is_tablet):

<redirect firstRequestOnly="true"
mobileHomePageUrl="~/Mobile/index.html"
timeout="20" mobilePagesRegex="~/Mobile/|~/Widgets/|~/Produkte/GN/|~/facebook/">

<locations>


<location name="Tablet" url="~/Default.aspx">
<add property="IsTablet" matchExpression="true"/>
</location>
</locations>

</redirect>

<log logFile="~/App_Data/Log.txt"
logLevel="Info" />

<detection>
<xmlFiles>
<add name="xml" filePath="~/App_Data/51Degrees.mobi-Lite-2012.04.04.xml" enabled="true" />
</xmlFiles>
</detection>

Any idea?
Comments:

The issue relates to the version of device data in use. IsTablet is part of the Premium data set.

Commented Issue: isTablet does not work [9335]

jwrosewell — Mon, 11 Jun 2012 12:08:26 GMT

The premium data is licenced on a per instance basis. If you have a 100 web sites on the same server using just one copy of the data between them then you'll need one instance. If you have more than one instance/copy of the data then you'll need enough instance subscriptions for each one. Sorry for the delay in coming back to you. I will mark the issue as closed.

Commented Issue: Make FiftyOne.Foundation.Mobile.Redirection.RedirectModule.IsFirstTime method public? [9387]

jwrosewell — Mon, 11 Jun 2012 12:05:22 GMT

This change makes a lot of sense and will be incorporated into the next release which we expect to make later this week. Thank you for your feedback.

Commented Issue: inheritInChildApplications cannot be set to false [8728]

ArtemS — Fri, 11 May 2012 14:51:51 GMT

Hi,
I just got this issue too. This was closed in december, it's now may and recent version doesn't include this fix.

I would argue that this is pretty high impact, as it makes <location> tag unusable for anybody using 51degrees.

It also sounds like the code was changed from last year, and the section reported by original reporter is now used for any config load, not just alternate one. Constants.ConfigFileNames now includes regular web.config (not sure if it did before or not). I haven't actually debugged this, just a guess.

The error that you get is

InheritInChildApplications cannot be set to "false" if the location path is referring to machine.config or the root web.config.

which is very confusing, as it doesn't point at 51degrees being the issue.

Created Issue: Make FiftyOne.Foundation.Mobile.Redirection.RedirectModule.IsFirstTime method public? [9387]

v1rich — Wed, 09 May 2012 22:11:02 GMT

Commented Issue: inheritInChildApplications cannot be set to false [8728]

rsiera — Sat, 05 May 2012 12:48:24 GMT

51degrees is being used by Dotnetnuke since v6.1.5.

When upgrading Dotnetnuke to v6.1.5 I got this error. I debugged latest 51degrees source code (v2.1.4.7) and I can confirm that issue still isn't fixed.