Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 12:20
    IgorFedchenko commented #3973
  • 12:17
    IgorFedchenko commented #3973
  • 11:58
    IgorFedchenko synchronize #3973
  • 11:33
    IgorFedchenko commented #3973
  • 11:25
    IgorFedchenko synchronize #3973
  • 07:04
    dependabot-preview[bot] labeled #3996
  • 07:04
    dependabot-preview[bot] opened #3996
  • 07:04

    dependabot-preview[bot] on nuget

    Bump Microsoft.NET.Test.Sdk fro… (compare)

  • 07:04
    dependabot-preview[bot] labeled #3995
  • 07:04

    dependabot-preview[bot] on nuget

    Bump BenchmarkDotNet from 0.10.… (compare)

  • 07:04
    dependabot-preview[bot] opened #3995
  • Oct 17 22:42
    Aaronontheweb commented #3944
  • Oct 17 22:41
    Aaronontheweb commented #3973
  • Oct 17 21:17
    IgorFedchenko commented #3973
  • Oct 17 21:08
    IgorFedchenko commented #3973
  • Oct 17 19:59
    IgorFedchenko synchronize #3973
  • Oct 17 19:34
    IgorFedchenko synchronize #3973
  • Oct 17 16:12
    Aaronontheweb commented #3993
  • Oct 17 15:51
    dependabot-preview[bot] synchronize #3991
  • Oct 17 15:51

    dependabot-preview[bot] on nuget

    Bump Microsoft.Extensions.Depen… (compare)

Natan Vivo
@nvivo
I'm getting an error on config loading where double quotes work, but single quotes gives an error. Shouldn't both work the same?
Natan Vivo
@nvivo

this works:

ConfigurationFactory.ParseString(@"akka { loggers = [""CustomLogger, Assembly""] }")

but this:

ConfigurationFactory.ParseString(@"akka { loggers = ['CustomLogger, Assembly'] }")

fails with "Logger specified in config cannot be found: "'CustomLogger, Assembly".

Roger Johansson
@rogeralsing
Ill have a look at it, could be a parser bug inside array
Roger Johansson
@rogeralsing
@nvivo hehe single quotes are not part of neither the json nor hocon spec.. so 'CustomLogger will be parsed as what hocon calls an unquoted string
agree that it would be useful, but I guess we should stick to the specs
Bartosz Sypytkowski
@Horusiath
when I run persistence tests (all at once) I've got random timeout exceptions all the time. ofc when I try to run them one by one to check, everything is ok
Roger Johansson
@rogeralsing
I came to think of that turbo button we had back in the 286/386 days :)
What if you run a subset of test? Maybe there are some specific tests that mess things up for others?
Bartosz Sypytkowski
@Horusiath
any domain subset of Akka.Persistence.Tests also cause this, only when I run them one by one, they pass
Roger Johansson
@rogeralsing
Even if two by two?
Possibly the threadpool is saturated, but that shouldnt be visible with a few tests at a time
Stefan Sedich
@stefansedich
should hostname casing matter? If I try use akka.tcp://import@foo:9998/user/foo
and my helios tcp hostname is FOO
the host will drop the message
Natan Vivo
@nvivo
@rogeralsing, I based that on the HOCON page that states "a JSON superset, that is, all valid JSON should be valid and should result in the same in-memory data that a JSON parser would have produced."
single quotes are not part of json spec?
wow.. that got me
it really isn't. who would have thought? =)
Natan Vivo
@nvivo
@rogeralsing nevermind. I agree, stick to the specs.
if only we read them... =)
Stefan Sedich
@stefansedich
that got me too yesterday @nvivo
jcwrequests
@jcwrequests
@nvivo I gave up on that stuff a long time ago and started using embedded text files for things like configuration and sql queries. If you were using VB.NET you could also use XML literals to solve the problem of readability. Anyway who needs specs :smile: .
Stefan Sedich
@stefansedich
I went with base config in web.config and some overrides it seems to work nicely.
Roger Johansson
@rogeralsing
I have to check that, URI's are case sensitive according to spec, dunno about akka behavior in JVM
Stefan Sedich
@stefansedich
yeah right, didn't think the host was case sensitive though?
"the scheme and host are case-insensitive and therefore should be normalized to lowercase. For example, the URI HTTP://www.EXAMPLE.com/ is equivalent to http://www.example.com/."
Roger Johansson
@rogeralsing
ah, a bug on our side then
Stefan Sedich
@stefansedich
I am happy to fix it, I will create an issue and take it on.
Roger Johansson
@rogeralsing
:+1:
Stefan Sedich
@stefansedich
ok cool created akkadotnet/akka.net#750 will get it done soon!
ah damn can't assign to myself?
Roger Johansson
@rogeralsing
Should be possible, Ill look into team settings
Stefan Sedich
@stefansedich
I could just be blind :D
Stefan Sedich
@stefansedich
@rogeralsing do you think the system should be case insensitive too on an Address?
same as protocol
Stefan Sedich
@stefansedich
yeah cool system name is case sensitive in JVM missed that one.
Roger Johansson
@rogeralsing
@stefansedich I found the problem, you were not in any of the access groups, added you. you should be able to assign the task now
Stefan Sedich
@stefansedich
sweet
yeah cool found the issue, there are tests around the casing of a hostname when using ActorSelection which are fine as that parses into a Uri and gets the host which gets lower cased for us.
the HeliosTransport creates the Address instance but does not lowercase.
easy fix should be just to always lower case the host when creating an Address instance to be safe.
Stefan Sedich
@stefansedich
how adverse are we to the boilerplate comments on things like this?
/// <summary>
/// Gets the host.
/// </summary>
I would rather remove them for cleanliness.
Roger Johansson
@rogeralsing
Ye, I'm to blame for most of them, had a period of Ghost doccing everything in my way :P
Stefan Sedich
@stefansedich
hehe yeah gathered it was ghost doc
Roger Johansson
@rogeralsing
just remove if they are nonsense
Stefan Sedich
@stefansedich
coolies I will give Address some love
my 1 line change made me feel bad have to do something useful