by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 14:03
    448333839 starred exceptionless/Exceptionless
  • 13:41
    niemyjski synchronize #649
  • 13:41

    niemyjski on discard-join

    WIP stack soft deletes. Fixed issues with resolving App… (compare)

  • 11:32
    CLAassistant commented #671
  • 11:32

    dependabot-preview[bot] on nuget

    (compare)

  • 11:32
    dependabot-preview[bot] closed #667
  • 11:32
    dependabot-preview[bot] commented #667
  • 11:32
    dependabot-preview[bot] labeled #671
  • 11:32
    dependabot-preview[bot] opened #671
  • 11:32

    dependabot-preview[bot] on nuget

    Bump Stripe.net from 37.10.0 to… (compare)

  • 09:06
    lichaoqiang starred exceptionless/Exceptionless
  • Jul 12 05:33
    huangweiboy2 starred exceptionless/Exceptionless
  • Jul 11 11:52
    niemyjski closed #670
  • Jul 11 11:52
    niemyjski commented #670
  • Jul 11 11:52
    niemyjski labeled #670
  • Jul 11 09:37
    emejibka opened #670
  • Jul 11 02:49

    ejsmith on master

    Upgrade ES to 7.8.0 (compare)

  • Jul 10 14:00
    niemyjski edited #530
  • Jul 10 14:00
    niemyjski edited #530
Blake Niemyjski
@niemyjski
like, when you file a patent you do existing research work.. when you are doing logging for an entire framework you do existing research
Eric J. Smith
@ejsmith
but there are like 100 extension methods that sit on top of ILogger and there is zero reason to include those in the logging contract.
Blake Niemyjski
@niemyjski
existing research would point that no one is doing what you are doing..
or what everyone else is using
too me that is a huge red flag
Eric J. Smith
@ejsmith
I don’t freakin’ care about the Log in the method names.
Blake Niemyjski
@niemyjski
like a few small changes would make it a heck of a lot easier for people to port to it
those shouldn’t be ther
well even that
only the contract
Eric J. Smith
@ejsmith
I want a shared contract that is tiny that has a chance of being broadly accepted.
Blake Niemyjski
@niemyjski
yeah
Eric J. Smith
@ejsmith
ILogger is 1 method
Blake Niemyjski
@niemyjski
if you want those extensions there then put them in a different namespace
or different assembly
Eric J. Smith
@ejsmith
and is proven to be performant
those extension methods have a bunch of dependencies.
now I gotta force those dependencies on my users
when all I freakin’ wanted was to write some simple log messages.
don’t understand why it’s so hard to understand
Blake Niemyjski
@niemyjski
you mean you didn’t want to do install-package logging.kitchensink
Eric J. Smith
@ejsmith
I want to install Foundatio and not have it bring the kitchen sink down with it or force me into using shit I don’t want to use.
Blake Niemyjski
@niemyjski
you know what this reminds me of
Azure/azure-storage-net#97
Eric J. Smith
@ejsmith
so as a lib developer my only choice is to source import the logging code so I don’t force people to use stuff they don’t want to use.
Blake Niemyjski
@niemyjski
omg another microsoft library
lol
Jamie Clarke
@jamie94bc
you guys happy with the PR?
only concern is the overhead of creating another dictionary
Eric J. Smith
@ejsmith
sorry, need to stop venting over here and check it. :-)
Blake Niemyjski
@niemyjski
yeah it looks good
well
actually
that won’t work I don’t think
Eric J. Smith
@ejsmith
the getunscoped could be simpler I think.
Blake Niemyjski
@niemyjski
protected string GetUnscopedCacheKey(string scopedKey) {
int i = scopedKey.IndexOf(':');
        if (-1 < i && i < (scopedKey.Length - 1)) {
            return scopedKey.Substring(i + 1);
        }

        return scopedKey;
    }
yeah
it should look at the Scope property which has the scope
Eric J. Smith
@ejsmith
just do Substring(Scope.Length + 1)
Blake Niemyjski
@niemyjski
because the passed in scope could have :a:b:c:
yeah
Jamie Clarke
@jamie94bc
yeah that's why i was looking for the first index
Blake Niemyjski
@niemyjski
so
Eric J. Smith
@ejsmith
but your scanning the keys when you know what the scope is.
Blake Niemyjski
@niemyjski
we should also add a null check to the constuctor and do
Jamie Clarke
@jamie94bc
but that's easier! :)
Blake Niemyjski
@niemyjski
Scope = scope?.Trim() ?? String.Empty
might be null currently
Eric J. Smith
@ejsmith
not sure if we should even allow it to be created without a scope.
Blake Niemyjski
@niemyjski
guess we probably should throw an argument exception if you try using a scoped cache client with no scope