These are chat archives for exceptionless/Discuss

24th
Nov 2015
benmaina
@benmaina
Nov 24 2015 13:51
@niemyjski Am here
@niemyjski so my exceptionless in production is starting but once on the signup page i cannot sign up due to the following error on the email field:"A user already exists with this email address".
I am usin g exceptionless version 3.1.182 and elasticsearch 1.7.3
Blake Niemyjski
@niemyjski
Nov 24 2015 13:55
hey
this could be because elastic search couldn’t be talked to
do you have team viewer or a screen share application
@benmaina :)
benmaina
@benmaina
Nov 24 2015 14:00
i have team viewer i can hook you up so that you connect
Blake Niemyjski
@niemyjski
Nov 24 2015 14:00
ok
feel free to message me the details
benmaina
@benmaina
Nov 24 2015 14:01
one moment i give you the ID
Blake Niemyjski
@niemyjski
Nov 24 2015 14:01
ok
benmaina
@benmaina
Nov 24 2015 14:06
@niemyjski ID:238 571 638 PWD:48hup9
you can connect now
Blake Niemyjski
@niemyjski
Nov 24 2015 14:08
hey
I have audo going, not sure if you can hear me or not
reguardless lets take a look
benmaina
@benmaina
Nov 24 2015 14:08
i can still see what you are doing so we can go on
i have set the exceptionless to run on port 8090 on iis
check on localhost spelling on the browser you have just input
Blake Niemyjski
@niemyjski
Nov 24 2015 14:14
should be good to go, the urls you enter into config need to match that of iis
benmaina
@benmaina
Nov 24 2015 14:15
should be on the web config right?
Blake Niemyjski
@niemyjski
Nov 24 2015 14:15
if you are accessing this remotely your configs should have the domain instead of localhost:8090 in the app.config and web.config
both configs matter
the app.config.js baseurl points to the server end point
and the web.config baseurl points to the user interface url (for link generation)
if you want to signup, I’ll make sure everything is working while I’m on
Blake Niemyjski
@niemyjski
Nov 24 2015 14:20
you don’t want to edit that one
benmaina
@benmaina
Nov 24 2015 14:23
just restarted iis with the saved settings and still the same error when signing up on email
Blake Niemyjski
@niemyjski
Nov 24 2015 14:23
@benmaina can you create an account quick?
I can only see your one screen
benmaina
@benmaina
Nov 24 2015 14:23
still not yet...look at the screen now
Blake Niemyjski
@niemyjski
Nov 24 2015 14:24
yeah. you need to fill out an email address and password
benmaina
@benmaina
Nov 24 2015 14:25
let me login remotely from another machine and not this one so that i tell if its able to
Blake Niemyjski
@niemyjski
Nov 24 2015 14:25
ok
once your signed up remotely lets sign into it on this machine
benmaina
@benmaina
Nov 24 2015 14:27
its cool now from other machines
Blake Niemyjski
@niemyjski
Nov 24 2015 14:28
ok
if you wish to setup logging of exceptionless itself I can help you do that if you’d like
benmaina
@benmaina
Nov 24 2015 14:31
I think from here i am now cool.Thanks alot for the support
Blake Niemyjski
@niemyjski
Nov 24 2015 14:32
no problem! please let us know if you have any feedback
Also, I’d recommend changing your team viewer password as it’s in general chat :)
benmaina
@benmaina
Nov 24 2015 14:33
@niemyjski already uninstalled..
haha
Blake Niemyjski
@niemyjski
Nov 24 2015 14:33
:)
what are you looking to log in your projects?
exceptions/logs/feature usages/xyz?
Frank Ebersoll
@frankebersoll
Nov 24 2015 17:02
@niemyjski I can happily add inheritance handling to our stringify method
do we need to use that all the time? or only for additional exception data?
Blake Niemyjski
@niemyjski
Nov 24 2015 17:03
that would be great
I think all the time
to be consistent we don’t know where we are calling stringify from
I don’t know if we have any stringify tests that check that or other simple data types.
probably need them if we are going to flatten cause we can’t error when flatting a string
I’m working on data exclusions in the .net client to be on pairity with the js client :)
Frank Ebersoll
@frankebersoll
Nov 24 2015 17:06
okay
should i fix it in main?
Blake Niemyjski
@niemyjski
Nov 24 2015 17:11
yeah
Blake Niemyjski
@niemyjski
Nov 24 2015 17:19
exceptionless/Exceptionless.Net@896432e
:)
Frank Ebersoll
@frankebersoll
Nov 24 2015 17:20
nice one!
should we flatten deeply?
or only first level?
Blake Niemyjski
@niemyjski
Nov 24 2015 17:21
how slow is this going to make it is my question
Frank Ebersoll
@frankebersoll
Nov 24 2015 17:21
deep would be harder to do because of possible cycles
we do it first-level :-) k?
Blake Niemyjski
@niemyjski
Nov 24 2015 17:21
yeah
I just remembered I never implemented the depth support to the stringify method.
Frank Ebersoll
@frankebersoll
Nov 24 2015 17:24
you mean max-depth?
that's right, i saw exceptions with very deep additional data pushing the layout out of its bounds on the right edge
it did not look pretty.
it was tables in tables. tables in tables in tables.
like a matryoshka doll
Blake Niemyjski
@niemyjski
Nov 24 2015 17:27
yeah
in the dot net client we do max depth
                json = serializer.Serialize(info.Data, exclusions, info.MaxDepthToSerialize.HasValue ? info.MaxDepthToSerialize.Value : 5, info.IgnoreSerializationErrors);
guess our default depth is 5
think it should be even smaller than that in js?
like 2
or 3
three might be good if we are doing a really small depth probably wouldn’t hurt to flatten it all but guess we should just flatten the first and it would be easy to flatten them all later if needed
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:12
in the meantime... i created a PR for my little refactoring. no actual build process improvements, yet. we need to discuss those a little more
Blake Niemyjski
@niemyjski
Nov 24 2015 18:12
ok
just let me know when you want to discus it
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:14
okay, there are multiple things :-)
first: test.
i tried it out and removed everything. karma, chrome, jasmine.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:16
did it make it easier to test?
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:16
i added: mocha, chai.
syntax is marginally different, now it's expect(something).to.be.null instead of .to.be(null)
Blake Niemyjski
@niemyjski
Nov 24 2015 18:16
ok
that’s fine if it makes testing easier :)
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:17
test time is down to 1 second for all tests.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:17
dang
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:17
you don't even wait, it's immediate.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:17
that will be nice
could even have it run tests on watch now
hehe
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:18
better: you can set breakpoints and debug into tests. in the IDE.
yes, that's what i have my BDD watcher task for.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:19
awe sweet
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:19
there's one problem: it will be much more fun if the build itself is faster.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:19
merge immediately :)
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:19
and we have no incremental builds yet - i don't know if tsproject supports them.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:20
the thing with 36, we’ve made some commits since then, we gotta make sure it merges properly.
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:20
i mean, it has to spit out the whole thing every time.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:20
tsproject does I think
if it doesn’t we could create an issue
that guy is really good about it
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:20
my question is, do we really need it during development time?
build, watch and watch-test should do as little work as possible.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:22
no, I think we are fine
although, do we have to transpile for tests?
I think that’s a yes
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:22
we need an additional publish task that does packaging, final tests and then pushes it
at least, that's what i imagine right now
yes, we need to transpile, and with source maps so.
because it will make debugging much easier. i've enabled source maps in karma, so we get stack traces with actual typescript references
no more searching in js files
it also enables stepping into typescript. i didn't get that with our current solution, sadly
Blake Niemyjski
@niemyjski
Nov 24 2015 18:25
yeah that will be really nice
well we have the build step
which publish relies on
so we should be fine. unless all of that changed
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:26
build takes about 20 seconds on my machine now
with gulp-tsb and incremental builds i got it down to 0.5 seconds when a file changes.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:27
well we don’t have to run the linter
that should make it faster no?
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:27
so, build could be only what is needed to run and debug stuff. and publish would do minifying, packaging and whatnot
yes, but even that could be included in watch
and even before building, because, i don't want to build if my source code has errors
i'll try to get incremental builds working with tsproject
but first, i fix stringify
#36 has been rebased onto all commits that have happened in the meantime
should be ready for merge
Blake Niemyjski
@niemyjski
Nov 24 2015 18:34
ok
did you sync?
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:37
?
Blake Niemyjski
@niemyjski
Nov 24 2015 18:38
push your changes I dont’ see the rebase or I guess it doesn’t show up in github?
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:39
i have rewritten the changes so they include your commits
Blake Niemyjski
@niemyjski
Nov 24 2015 18:40
ok, do you think we should merge it now or after our release?
it’s your call
if it’s going to make our lives easier lets do it now
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:40
this should be non-breaking, as a recompile lead to no changes in dist
Blake Niemyjski
@niemyjski
Nov 24 2015 18:41
ok
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:41
only things to change in dist have been maps, which makes sense
Blake Niemyjski
@niemyjski
Nov 24 2015 18:42
ok
merged
give me a few minutes
once I push these nuget packages I can focus just on the js client
@all thanks for your help getting https://github.com/exceptionless/Exceptionless.Net/releases/tag/v3.2.0 out the door!
Frank Ebersoll
@frankebersoll
Nov 24 2015 18:43
okay, i'm on stringify then.
Blake Niemyjski
@niemyjski
Nov 24 2015 18:54
I’m on it, looking at your changes :)
pulling latest hehe
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:05
i put my messing with the build into... ehm... features/betterBuild
would be interesting to know if that works for you. i constantly get 0.8 seconds for 72 test cases
Blake Niemyjski
@niemyjski
Nov 24 2015 19:07
ok
want me to pull that down first?
I’m on that branch now
let me check
3.2.1424
830ms
dangggg
we had that one test that did a timeout or something
did you update it?
I think that was taking the 10 seconds to be honest not sure
awe that is awesome, it highlights the slow tests
[TsProject] Compiling Project Files...
[TsProject] src/typings/jasmine/jasmine.d.ts(9,18): error TS2300: Duplicate identifier 'describe'.
[TsProject] src/typings/jasmine/jasmine.d.ts(11,18): error TS2300: Duplicate identifier 'xdescribe'.
[TsProject] src/typings/jasmine/jasmine.d.ts(13,18): error TS2300: Duplicate identifier 'it'.
[TsProject] src/typings/jasmine/jasmine.d.ts(14,18): error TS2300: Duplicate identifier 'it'.
[TsProject] src/typings/jasmine/jasmine.d.ts(17,18): error TS2300: Duplicate identifier 'xit'.
[TsProject] src/typings/jasmine/jasmine.d.ts(18,18): error TS2300: Duplicate identifier 'xit'.
[TsProject] src/typings/mocha/mocha.d.ts(37,13): error TS2300: Duplicate identifier 'describe'.
[TsProject] src/typings/mocha/mocha.d.ts(38,13): error TS2300: Duplicate identifier 'xdescribe'.
[TsProject] src/typings/mocha/mocha.d.ts(43,13): error TS2300: Duplicate identifier 'it'.
[TsProject] src/typings/mocha/mocha.d.ts(44,13): error TS2300: Duplicate identifier 'xit'.
Blake Niemyjski
@niemyjski
Nov 24 2015 19:12
do you get that as well?
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:12
just throw away your jasmine typings
tsd doesn't remove them on its own.
i don't know about timeouts - didn't get any. i don't know how to specify the timeout for single tests either, but we should focus on having fast tests anyway.
Blake Niemyjski
@niemyjski
Nov 24 2015 19:17
yeah, I fixed it.
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:18
what do you think about it generally?
Blake Niemyjski
@niemyjski
Nov 24 2015 19:18
like it
I think it was eventplugin manager timeout test or it was a timeout checking test
but if it’s all green fine by me
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:19
source maps not working yet, but we will get there.
Blake Niemyjski
@niemyjski
Nov 24 2015 19:19
seems much faster and higher weight
only thing I really want and it’s not going to happen anytime soon is for webstorm to detect file paths and make them clickable
would be sweet to click to go to test failure, I think vs code has it
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:20
yes, has it since the latest update.
Blake Niemyjski
@niemyjski
Nov 24 2015 19:20
light weight*
so when our normal files change, we need to build before test
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:21
yes, i like that too. don't ask me what karma / jasmine was doing, maybe spinning of a process for each test or so
but i hope we don't need that.
Blake Niemyjski
@niemyjski
Nov 24 2015 19:21
wonder if we can hash the files and if they changed we trigger a rebuild
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:21
yes, but we can make that faster, too.
Blake Niemyjski
@niemyjski
Nov 24 2015 19:21
but that’s something in the future
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:21
i got that going in my .Relay project already
Blake Niemyjski
@niemyjski
Nov 24 2015 19:21
:)
so do you have to do anythign special to debug?
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:22
rebuild needs about 0.5 seconds there.
Blake Niemyjski
@niemyjski
Nov 24 2015 19:22
so fast :)
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:22
well, only for changed files while watching
but that will be okay during development
no need to hash anything
Blake Niemyjski
@niemyjski
Nov 24 2015 19:23
yeah
that’s a good tradeoff
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:23
so do you have to do anythign special to debug?
?
Blake Niemyjski
@niemyjski
Nov 24 2015 19:23
just wanted to try it out wondered if you had to do anything differently
trying it now
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:24
i debug directly in vscode. how would you like to do it?
for me, it's setting a breakpoint and hitting F5
Blake Niemyjski
@niemyjski
Nov 24 2015 19:25
let me try that
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:25
what does WebStorm provide there?
Blake Niemyjski
@niemyjski
Nov 24 2015 19:25
I use webstorm a lot because I really like how powerful navigation and things like that are
not much I can show you via team viewer
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:25
can it debug node programs?
Blake Niemyjski
@niemyjski
Nov 24 2015 19:26
yep
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:26
try those arguments:
Blake Niemyjski
@niemyjski
Nov 24 2015 19:28
don’t see anything
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:29
program to debug: node_modules/.bin/_mocha
arguments: dist/temp/exceptionless-spec.js
Blake Niemyjski
@niemyjski
Nov 24 2015 19:30
can you commit your .vscode launch.json file?
would be cool to get this all configured for vscode
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:31
yes, i wanted to do that. give me a second
Blake Niemyjski
@niemyjski
Nov 24 2015 19:31
ok
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:38
Pizza
see you after Dinner
pushed it.
maybe you can use the vscode settings somehow for getting it working in webstorm
gulp test
Blake Niemyjski
@niemyjski
Nov 24 2015 19:39
ok
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:39
then add a breakpoint in the exceptionless-spec.js
Blake Niemyjski
@niemyjski
Nov 24 2015 19:39
ok
trying now
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:39
then launch the "Test" debug configuration
Blake Niemyjski
@niemyjski
Nov 24 2015 19:40
how did you do that
shortcut keys
when I press f5 it just does the launch node
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:47
Go to the debug area on the left. Select "Test" from the dropdown
Blake Niemyjski
@niemyjski
Nov 24 2015 19:53
ok
looking
thinking we should default it to test
the other one for the node app could be in the express folder
thoughts?
pushed
break points don’t seem to be mapping properly
I’ve always had problems with that
Frank Ebersoll
@frankebersoll
Nov 24 2015 19:57
pizza :)
Blake Niemyjski
@niemyjski
Nov 24 2015 19:58
haha
eat up
I had summer sausage for lunch
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:12
thinking we should default it to test
not a problem. we just need to put this configuration to the top, i think
break points don’t seem to be mapping properly
i think that the paths in the .map files are incorrect
Blake Niemyjski
@niemyjski
Nov 24 2015 20:14
well
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:14
i already got that working, just not yet in our project
Blake Niemyjski
@niemyjski
Nov 24 2015 20:14
they would be pointing to dist/temp and our breakpoints are set in /
I think that’s the issue
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:19
they need to point back to the actual sources
ehm....
exceptionless-spec.js.map contains only source code, no mappings
not cool.
Blake Niemyjski
@niemyjski
Nov 24 2015 20:21
{"version":3,"file":"submitSync.js","sourceRoot":"/source/","sources":["submitSync.ts"],
that sourceroot should be /src/ no?
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:21
right.
even more, it needs to be relative
or absolute to your workstation root
that's another problem that would be solved by an additional publish-build
Blake Niemyjski
@niemyjski
Nov 24 2015 20:22
relative would be best
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:23
during debugging, we need source maps that support our local file structure
Blake Niemyjski
@niemyjski
Nov 24 2015 20:23
yeah
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:23
after publishing, we need to support loading via the web
that might be two different things.
i'm back on master now, finishing stringily.
stringify*
Blake Niemyjski
@niemyjski
Nov 24 2015 20:28
kk
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:36
ah, another thing!
the new npm seems to deduplicate modules into the root node_modules directory, wherever possible
Blake Niemyjski
@niemyjski
Nov 24 2015 20:37
yeah
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:37
i guess it does this by downloading the "lowest common denominator" version of a module that is used by other modules
Blake Niemyjski
@niemyjski
Nov 24 2015 20:37
it seems much slower tho
there is a github issue for that
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:38
i like this behavior, but I would guess it has problems with our current package.json
as we specify versions explicitly, it has to use that exact version
Blake Niemyjski
@niemyjski
Nov 24 2015 20:39
yeah
I do that because I’ve been burned by newer packages
plus the build should never change ever
it should know the specific packages
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:39
maybe it has more possibilities for optimization if we use ^version, as the default does
Blake Niemyjski
@niemyjski
Nov 24 2015 20:40
yeah
thought about that
guess we’d know when we are building that as we are commiting the files manually
we don’t do it for our spa app because we auto deploy and need to know the changes immediately
need to know that nothing ever gets updated and deployed without us changing it*
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:41
i understand
i really don't know about the implications
Blake Niemyjski
@niemyjski
Nov 24 2015 20:42
yeah me either
kinda funny..
it’s impossible to be a fullstack developer and that term.. because there are a trillion of things that you need to know the nitty gritty details of and it’s impossible
how many things do we use every day that we have a gist of how it works or a good understanding
but knowing the implications is hard without serious work for something trivial
I’ve been spending a lot of free time to read up on things I don’t know and I still feel 10yr behind
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:46
i guess we're pretty close to being full stack developers
i have seen different :-)
both of us know what we don't know and actually think about it. i have seen too many people doing things while having absolutely no clue whatsoever
and they can't even be blamed, because everybody started like this
but they were writing business critical applications and trying to tell me that they don't synchronize their code because, c'mon, this race condition will only happen in one of a hundred times.
Frank Ebersoll
@frankebersoll
Nov 24 2015 20:51
they "fixed" it with Thread.Sleep
sigh sorry.
Blake Niemyjski
@niemyjski
Nov 24 2015 21:00
back talking to sander :)
lol
Frank Ebersoll
@frankebersoll
Nov 24 2015 21:01
?
Blake Niemyjski
@niemyjski
Nov 24 2015 21:01
it’s all good
was in a tab and forgot about the main channel /sorry
Frank Ebersoll
@frankebersoll
Nov 24 2015 21:01
tell me if i'm ranting too much :-) might happen occasionally
Blake Niemyjski
@niemyjski
Nov 24 2015 21:01
nope, your good
I rant a lot too
yeah I need to get better at not judging stuff and being negative
sometimes that happens and I’ve been trying really hard not too jump to decisions to fast