actor ids and timer names

[aktxyz]

encoding of actor ids and timer names?

I have noticed if I use some characters (like ?) in an actor id or timer name, things silently fail.

This kind of makes sense since these values eventually make their way into URL paths header to dapr sidecars.

But it seems like these values should either be encoded, or an exception should be thrown for invalid values.

[aktxyz]

maybe beef up the ArgumentVerifier with some more scenarios?

ArgumentVerifier.ThrowIfNullOrEmpty(pubsubName, nameof(pubsubName));

[rynowak]

This makes sense - we should be url-encoding Actor Ids and probably names as well so this works end to end. As long as the Dapr sidecar will accept a %-encoded actor name or Id then we should tolerate it as well. We don't want the SDK to be stricter than the runtime itself is.

We'd happily accept a PR for this.