Closed X3Technology closed 5 years ago
@mathisjay are you saying the files are too big to...
well, for starters, the 51degrees.dat file is over 100MB and the GitHub limit is 100MB...
But, there is a file size limit on the module installer also.
I didnt try putting it directly in the /install folder, that might work.
Also, I wasnt sure about how to get the manifest to put the file in the AppData folder.
Get Outlook for Androidhttps://aka.ms/ghei36
From: David Poindexter notifications@github.com Sent: Wednesday, July 10, 2019 7:51:29 PM To: DNNCommunity/Dnn.WebAnalytics Cc: Jay Mathis; Mention Subject: Re: [DNNCommunity/Dnn.WebAnalytics] Installation process (#1)
@mathisjayhttps://github.com/mathisjay are you saying the files are too big to...
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/DNNCommunity/Dnn.WebAnalytics/issues/1?email_source=notifications&email_token=AAMDENJQV4ZYIUR7WHEO6BTP6ZYYDA5CNFSM4H4NEEUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVCHCA#issuecomment-510272392, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAMDENKZHXUEXLJIQTQRQ7DP6ZYYDANCNFSM4H4NEEUA.
The 51Degrees file probably shouldn't be part of the install or check-in at all. Instead, it should be integrated with their API and then the API would download it after install.
I don’t understand that… The code just expects the .dat file to be there. If there is some magic sauce to make that happen automatically, then I am all for it.
From: Will Strohl notifications@github.com Sent: Thursday, July 11, 2019 4:36 PM To: DNNCommunity/Dnn.WebAnalytics Dnn.WebAnalytics@noreply.github.com Cc: Jay Mathis jmathis@x3technology.com; Mention mention@noreply.github.com Subject: Re: [DNNCommunity/Dnn.WebAnalytics] Installation process (#1)
The 51Degrees file probably shouldn't be part of the install or check-in at all. Instead, it should be integrated with their API and then the API would download it after install.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/DNNCommunity/Dnn.WebAnalytics/issues/1?email_source=notifications&email_token=AAMDENLNOTPAMTL22W2CJCDP66KSJA5CNFSM4H4NEEUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZX46SA#issuecomment-510644040, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAMDENIMLNFMFC7RRKXDKXDP66KSJANCNFSM4H4NEEUA.
@mathisjay are you using THIS via NuGet or something else?
Here is the .NET API - https://51degrees.com/Developers/Documentation/APIs/NET-V32
Here is the Cloud API - https://51degrees.com/support/documentation/cloud-api
Also, I know and work with the folks at 51Degrees, so I can get them to help. :)
Thanks for providing those links, @david-poindexter.
@mathisjay The file was only (manually) put there originally as a short-cut - so-to-speak. At the time, there was no need or desire to properly implement the API that 51Degrees has, because the file was already there and we never thought the code would become public in any way. (So, it wasn't productized for other use cases/websites.)
If we do properly implement it to use their API, it will lower the footprint, prevent the need for committing the data, and also regularly receive updated mobile information.
thats awesome. im all for it
Get Outlook for Androidhttps://aka.ms/ghei36
From: Will Strohl notifications@github.com Sent: Friday, July 12, 2019 4:38:04 PM To: DNNCommunity/Dnn.WebAnalytics Cc: Jay Mathis; Mention Subject: Re: [DNNCommunity/Dnn.WebAnalytics] Installation process (#1)
Thanks for providing those links, @david-poindexterhttps://github.com/david-poindexter.
@mathisjayhttps://github.com/mathisjay The file was only (manually) put there originally as a short-cut - so-to-speak. At the time, there was no need or desire to properly implement the API that 51Degrees has, because the file was already there and we never thought the code would become public in any way. (So, it wasn't productized for other use cases/websites.)
If we do properly implement it to use their API, it will lower the footprint, prevent the need for committing the data, and also regularly receive updated mobile information.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/DNNCommunity/Dnn.WebAnalytics/issues/1?email_source=notifications&email_token=AAMDENINDB5AHUNGEKJJ6WLP7DTSZA5CNFSM4H4NEEUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ22DAY#issuecomment-511025539, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAMDENJOUPJJFKY2YYKO5CDP7DTSZANCNFSM4H4NEEUA.
I guess I'm a bit late to the party, but I added an msbuild-packaging script today. I ran into a missing file though: the .mmdb file. Been trying to find it here https://github.com/maxmind but maybe anyone has a link to it?
I was in the package I was handed from Shaun
I think it come from here: https://dev.maxmind.com/geoip/geoip2/geolite2/
From: Stefan Kamphuis notifications@github.com Sent: Thursday, July 25, 2019 9:17 AM To: DNNCommunity/Dnn.WebAnalytics Dnn.WebAnalytics@noreply.github.com Cc: Jay Mathis jmathis@x3technology.com; Mention mention@noreply.github.com Subject: Re: [DNNCommunity/Dnn.WebAnalytics] Installation process (#1)
I guess I'm a bit late to the party, but I added an msbuild-packaging script today. I ran into a missing file though: the .mmdb file. Been trying to find it here https://github.com/maxmind but maybe anyone has a link to it?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/DNNCommunity/Dnn.WebAnalytics/issues/1?email_source=notifications&email_token=AAMDENL2KDEB3KNAFTHBMK3QBGRUPA5CNFSM4H4NEEUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2ZN6YQ#issuecomment-515039074, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAMDENPRYTMVCVJ6Q34NCFTQBGRUPANCNFSM4H4NEEUA.
Thanks for providing those links, @david-poindexter.
@mathisjay The file was only (manually) put there originally as a short-cut - so-to-speak. At the time, there was no need or desire to properly implement the API that 51Degrees has, because the file was already there and we never thought the code would become public in any way. (So, it wasn't productized for other use cases/websites.)
If we do properly implement it to use their API, it will lower the footprint, prevent the need for committing the data, and also regularly receive updated mobile information.
It would be nice however, to be able to use the downloaded file as an option. I know of at least one client where the server is not able to access external API's for security reasons.
This module requires several large data files to be located in the App_Data folder and also a .config file to be located in the web root. The data files are much too large for the module installer process. And, anyways I am not sure how to work them into the module manifest.
Here are files that need to be located in the App_Data folder: 51Degrees.dat - provides device information for requests made to the website GeoIP2-City.mmdb - provides geocoding service for IP addresses
This file needs to be the web root: 51Degrees.mobi.config