These are chat archives for exceptionless/Discuss

8th
Dec 2015
Blake Niemyjski
@niemyjski
Dec 08 2015 19:41
@theit8514 just let me know when it’s ready
and I’ll review and accept it
theit8514
@theit8514
Dec 08 2015 20:03
@niemyjski I created the pull request here csnover/TraceKit#39
Blake Niemyjski
@niemyjski
Dec 08 2015 20:12
ok
@theit8514 just noticed looking at the diff the @? is marked as optional
but no tests for it.. I’ll accept it but if you come across any use cases where this breaks down let me know
do you have a try catch block that would reproduce that stack trace? I’d love to run it and take a look
@theit8514 do you need a new release pushed?
theit8514
@theit8514
Dec 08 2015 20:21
Sorry, chasing fires here. Not really a high priority for me right now, we're still in beta
Blake Niemyjski
@niemyjski
Dec 08 2015 20:21
ok
I’ll try and push it soon
I’ll try and pull it down and do more testing / see what else I could include with it
if you need it before I push it let me know and I’ll do it asap.
theit8514
@theit8514
Dec 08 2015 21:11
@niemyjski - Sample gist for the error https://gist.github.com/theit8514/e4f201f3a2717045f44b
In the plugin, I cause an error (dataSource undefined) which causes the stack trace to contain that .plugin/e.fn[c]/< method.
Blake Niemyjski
@niemyjski
Dec 08 2015 21:13
thanks!
really appreciate it
Frank Ebersoll
@frankebersoll
Dec 08 2015 21:47
hey! long time no see :-)
Frank Ebersoll
@frankebersoll
Dec 08 2015 21:55
i really like your feature comparison in the wiki
Eric J. Smith
@ejsmith
Dec 08 2015 22:10
hey!
Blake Niemyjski
@niemyjski
Dec 08 2015 22:10
:)
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:13
i've successfully kept away from javascript the last few days
Blake Niemyjski
@niemyjski
Dec 08 2015 22:13
yeah
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:13
those were good days.
Blake Niemyjski
@niemyjski
Dec 08 2015 22:13
been on vacation?
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:16
no, just sick at home again.
Blake Niemyjski
@niemyjski
Dec 08 2015 22:16
sad
I hope you feel better
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:18
really sad. particularly because I call in sick in office and then contribute to open source at home :-)
Blake Niemyjski
@niemyjski
Dec 08 2015 22:19
haha
hope they don’t check your github profile
p.s., I have some really fun and cool javascript stuff incoming
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:19
they absolutely could do that
i'm excited. what about?
Blake Niemyjski
@niemyjski
Dec 08 2015 22:20
exceptionless/Exceptionless#166
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:20
hah! nice
wanted to come back to you about something like that!
gimme a second
Blake Niemyjski
@niemyjski
Dec 08 2015 22:20
ok
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:20
found it on reddit somewhere...
theit8514
@theit8514
Dec 08 2015 22:21
:point_up: December 8, 2015 3:12 PM Some Safari tests use this gecko regex, and they have stacks which don't include a @ at the beginning. Requiring the @ fails several tests: https://gist.github.com/theit8514/f11d1a6f96c3e5f17f0e
he has some ideas there to create "context" around log messages
i like that.
Blake Niemyjski
@niemyjski
Dec 08 2015 22:23
@theit8514 ok :)
that’s pretty cool
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:26
it's something I often miss in production apps when there are intermittent errors that cannot be reproduced in development
single messages are not enough, you need more context
Make the script load async from cdn and collect any runtime errors. We need to promote this as the best option so we can rev the script faster. We should move bower and npm install methods to a side note.
could you elaborate on that a little?
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:31
the last thing i can remember from before becoming sick was looking into javascript bundling :-|
in retrospect, that might have had negative impact on my health
Blake Niemyjski
@niemyjski
Dec 08 2015 22:34
hehe
yeah, so we’d load our script async like google anayltics
and push any errors that happen to an array and process them when the whole framework has been loaded
Frank Ebersoll
@frankebersoll
Dec 08 2015 22:35
would we use some existing loader mechanism?
i mean, there is browserify
have you looked into that?
Blake Niemyjski
@niemyjski
Dec 08 2015 22:39
a little bit