These are chat archives for exceptionless/Discuss

29th
Feb 2016
Jamie Clarke
@jamie94bc
Feb 29 2016 07:58
Happy Leap Day everyone! :frog:
Blake Niemyjski
@niemyjski
Feb 29 2016 12:39
likewise :)
Blake Niemyjski
@niemyjski
Feb 29 2016 12:59
Wonder if any old software is going to have an issue today
lol
Sander Rijken
@srijken
Feb 29 2016 13:04
some new software just did :P
Blake Niemyjski
@niemyjski
Feb 29 2016 13:06
yeah :)
Blake Niemyjski
@niemyjski
Feb 29 2016 13:28
@ejsmith @pwelter34 how is logging coming along? We really should update https://github.com/exceptionless/Foundatio.Samples and https://github.com/exceptionless/Foundatio.Repositories as well
Eric J. Smith
@ejsmith
Feb 29 2016 13:42
We are switched over. It was a PITA.
Blake Niemyjski
@niemyjski
Feb 29 2016 13:42
just kinda worried
you guys have been fixing some good bugs and the only way we can get them is by upgrading
Eric J. Smith
@ejsmith
Feb 29 2016 13:42
Yeah, we should update those.
Blake Niemyjski
@niemyjski
Feb 29 2016 13:43
I just logged into my personal vultr vm
Eric J. Smith
@ejsmith
Feb 29 2016 13:43
Yep it's worth it.
Blake Niemyjski
@niemyjski
Feb 29 2016 13:43
applied 16 updates, a free cpu upgrade and complete reboot in under 3 minutes
thing is so insanely fast
32$ a moth for a quad core 3.6ghz cpu, 90gb ssd
Eric J. Smith
@ejsmith
Feb 29 2016 13:47
That's a pretty good deal.
Blake Niemyjski
@niemyjski
Feb 29 2016 13:48
yeah
I split it with a old developer friend
so it’s 16$ a month
brb
Blake Niemyjski
@niemyjski
Feb 29 2016 14:49
<script src="//cdn.rawgit.com/exceptionless/Exceptionless.JavaScript/v1.3.2/dist/exceptionless.min.js?apiKey=YOUR_API_KEY"></script>
<script src="//cdn.jsdelivr.net/fingerprintjs2/1.1.1/fingerprint2.min.js"></script>
    <script type="text/javascript">
        new Fingerprint2().get(function(uniqueHash, components) {
            var client = exceptionless.ExceptionlessClient.default;
            client.config.setUserIdentity(uniqueHash);
            client.submitFeatureUsage('PageView');
        });
    </script>
wonder if I need to do on page load with that baby
that should give me unique anon users without cookies
Jamie Clarke
@jamie94bc
Feb 29 2016 14:50
that's cool!
Blake Niemyjski
@niemyjski
Feb 29 2016 14:51
need to put this in the wiki :)
I noticed that I had to go through the source to figure this out
so we have some work todo
Eric J. Smith
@ejsmith
Feb 29 2016 14:53
Hmmm... seems a bit overkill. If we just store a cookie or an id in local storage, it would be better because you can use that and know it won't change unlike this fingerprint.
Blake Niemyjski
@niemyjski
Feb 29 2016 14:53
yeah
but until we have something like that
..
Eric J. Smith
@ejsmith
Feb 29 2016 14:54
How hard is it to add an id somewhere? Do it in your own code.
Says that takes up to 600ms to run.
Blake Niemyjski
@niemyjski
Feb 29 2016 14:55
??600ms
Eric J. Smith
@ejsmith
Feb 29 2016 14:55
It's interesting, but just silly. Store an id.
Blake Niemyjski
@niemyjski
Feb 29 2016 15:09
yeah I could
but then I have to have startup logic to persist it and get it
guess I can see how easy it is to do it
Blake Niemyjski
@niemyjski
Feb 29 2016 15:14
wow
there is no way I can do that
@frankebersoll
our storage settings is queue based
and it’s really hard to get a stored key out of it

private static getSavedServerSettings(config: Configuration): ISettingsWithVersion {
let item = config.storage.settings.get()[0];
if (item && item.value && item.value.version && item.value.settings) {
return item.value;
}

return { version: 0, settings: {} };

}

before we could do config.storage.get(maxitems)
now we moved it to storage having different containers (.queue and .settings).
but no way to get named things out of them
so makes it really hard for me to use the storage myself
wonder what would be a good way around that
would be nice if we had scoped keys and a single collection..
that way queue items would be prefixed and settings likewise and then we could use a get(“namespace*”) or get(“id”)
Blake Niemyjski
@niemyjski
Feb 29 2016 15:24
guess could just use local storage + polyfill
Eric J. Smith
@ejsmith
Feb 29 2016 15:43
polyfill to fallback to cookies is rediculously small.
we should have different defaults for settings storage and queue storage.
settings should default to being persisted.
and queue defaults to in memory.
Blake Niemyjski
@niemyjski
Feb 29 2016 15:44
yeah
Blake Niemyjski
@niemyjski
Feb 29 2016 17:31
@ejsmith the foundatio nuget package is still broken
the version needs to be fixed
Eric J. Smith
@ejsmith
Feb 29 2016 17:31
what?
I’m using it
what is broken?
Eric J. Smith
@ejsmith
Feb 29 2016 17:50
@niemyjski ?
Blake Niemyjski
@niemyjski
Feb 29 2016 18:06
@ejsmith
the nuget package is broken
look at it
Eric J. Smith
@ejsmith
Feb 29 2016 18:07
how about you tell me what is broken?
Blake Niemyjski
@niemyjski
Feb 29 2016 18:07
just pulled latest and same thing
Eric J. Smith
@ejsmith
Feb 29 2016 18:07
because I’m using it.
Blake Niemyjski
@niemyjski
Feb 29 2016 18:07
net45 contains pdb and net 451 contains dll
target 4.5 and try to use it
Eric J. Smith
@ejsmith
Feb 29 2016 18:07
ok, so that doesn’t keep you from using it, but yes it should be fixed.
Blake Niemyjski
@niemyjski
Feb 29 2016 18:07
it does
because our package targets 4.5
so nuget picks 4.5 instead of 4.5.1
and then won’t compile due to no reference
Eric J. Smith
@ejsmith
Feb 29 2016 18:08
well, we might not be able to target 4.5 any more.
Blake Niemyjski
@niemyjski
Feb 29 2016 18:08
brb making lunch
Eric J. Smith
@ejsmith
Feb 29 2016 18:08
since all the packages were upgraded to 4.5.1
Blake Niemyjski
@niemyjski
Feb 29 2016 18:08
that’s fine but it needs to be fixed then
Eric J. Smith
@ejsmith
Feb 29 2016 18:25
should be fixed now
Blake Niemyjski
@niemyjski
Feb 29 2016 18:29
thanks :)
Blake Niemyjski
@niemyjski
Feb 29 2016 18:40
@ejsmith workitem handlers need to be updated
I think they should take a logger factory
and then we get can get rid of the enablelogging flag
Eric J. Smith
@ejsmith
Feb 29 2016 18:40
yeah
Blake Niemyjski
@niemyjski
Feb 29 2016 18:40
also what is a one time work item handler
want me to to make that change?
Eric J. Smith
@ejsmith
Feb 29 2016 18:41
sure
I’m in the middle of something
then going to lunch
Blake Niemyjski
@niemyjski
Feb 29 2016 18:41
I was just going to update this foundatio repos change quick (not updating anything else) just want to keep track of everything else that is going on)
Eric J. Smith
@ejsmith
Feb 29 2016 18:42
ok
didn’t you add something to es config to keep the amount of logs limited?
Blake Niemyjski
@niemyjski
Feb 29 2016 18:43
whats up with all the onetimeworkitemhandlers
bunch of duplicated types
nope
Eric J. Smith
@ejsmith
Feb 29 2016 18:43
we have 4gb of logs
remember some guy created an issue
I think the 1 time handlers are a WIP
not working
Blake Niemyjski
@niemyjski
Feb 29 2016 18:55
ok
yeah there was an issue for them
but I didn’t apply it
because it really changes it up
the format
and it’s easy to wipe them out if we do have an issue
Blake Niemyjski
@niemyjski
Feb 29 2016 19:49
Seems like I can’t update those.. and they were brought in by the xunit project I believe.
Screen Shot 2016-02-29 at 1.38.50 PM.png
Blake Niemyjski
@niemyjski
Feb 29 2016 20:03
InMemoryMetricsClient: An in memory metrics implementation. This metrics implementation is only valid for the lifetime of the process. It's worth noting that this metrics client also has the ability to display the metrics to a TextWriter on a timer or by calling DisplayStats(TextWriter).
where is display stats
Eric J. Smith
@ejsmith
Feb 29 2016 20:42
got rid of it… we have the ability to view metric stats now.
Blake Niemyjski
@niemyjski
Feb 29 2016 21:08
how?
should update the readme
        // 008, Accoona, Arachmo, boitho, B-O-T, Cerberian Drtrs, Charlotte, Charlotte, Covario IDS, DataparkSearch,FindLinks,Holmes,htdig,
        // ia_archiver,ichiro,igdeSpyder,L.webis,Larbin,LinkWalker,lwp-trivial,mabontland,Mediapartners-Google,Mnogosearch,mogimogi,Morning Paper,
        // MVAClient,NetResearchServer,NewsGator,NG-Search,NutchCVS,Nymesis,oegp,Orbiter,Peew,Pompos,PostPost,PycURL,Qseero,Radian6,SBIder
        // ScoutJet,Scrubby,SearchSight,semanticdiscovery,ShopWiki,silk,Snappy,Sqworm,StackRambler,Teoma,TinEye,truwoGPS,updated,Vagabondo,Vortex
        //voyager,VYU2,webcollage,Websquash.com,wf84,WomlpeFactory,yacy,Yahoo,YandexImages,Yeti,yoogliFetchAgent,Zao, ZyBorg
        project.Configuration.Settings[SettingsDictionary.KnownKeys.UserAgentBotPatterns] = "*bot*,*crawler*,*spider*";
comments are the ones not covered by the wildcards I’m going to start setting for new projects and in this update work item job that’s triggered via admin endpoint
see any in there you care about
or should I just add them all
tineye, newsgator and yahoo stand out to me personally
Eric J. Smith
@ejsmith
Feb 29 2016 21:19
well, hopefully adding all of those won’t block any legit traffic.
since we use wildcards
Blake Niemyjski
@niemyjski
Feb 29 2016 21:26
I’m thinking about only doing the first three
unless you think we should add all of the above
Eric J. Smith
@ejsmith
Feb 29 2016 21:34
I think if they are unique bot agents that won’t accidentally remove valid traffic...
then we should add them as defaults
just worried that some will be too generic
Blake Niemyjski
@niemyjski
Feb 29 2016 21:53
you can look through that list
guess I can add all of those :\
just think that it’s a huge list
Eric J. Smith
@ejsmith
Feb 29 2016 21:55
yeah, can probably find a smaller list some where.
Blake Niemyjski
@niemyjski
Feb 29 2016 21:55
well
I think the three above and maybe yahoo
would cover like 95%
or higher
Eric J. Smith
@ejsmith
Feb 29 2016 22:25
ok
do the big ones
Blake Niemyjski
@niemyjski
Feb 29 2016 22:44
off the top of your heard
head
do you know if we are releasing locks if an error occurs inside of a job?
Eric J. Smith
@ejsmith
Feb 29 2016 22:45
hope so
not sure
Blake Niemyjski
@niemyjski
Feb 29 2016 22:45
looks like it should be
via using block
but I wonder
I triggered this job and hit an exception
and then my work item job isn’t being rehit
it has a default delay
I am seeing an issue where the queue job appears to stop working.
gotta try and figure that out.
Blake Niemyjski
@niemyjski
Feb 29 2016 23:31
hmm
whose going to be pull request #200?
@ejsmith exceptionless/Exceptionless#199
this fixes a few issues of which bill ran into and should also help with deduping
well.. noise..
Blake Niemyjski
@niemyjski
Feb 29 2016 23:38
if you like it merge it :)
@ejsmith are you going to be on tonight?
We can review my stats stuff
and other pull request stuff :)
Eric J. Smith
@ejsmith
Feb 29 2016 23:45
@niemyjski why are you throttling the work item to once every 15 minutes?