Anders Bjerner

Awesome 👍 I have thought about this before, and IIRC I just handled this locally in my value converters rather than submitting a fix to core (silly me). Anyways, it...

`CharRangeIndexDetectJsonBad` is okay with a string starting with `{` and ending with `[` - or starting with `[` and ending with `}`, which wouldn't be a valid JSON string (hence...

@mikecp yes, I unfortunately made a small typo in my "correct" example. The last `{` should be a `}` instead. This then also matches what @patrickdemooij9 now has committed. ```csharp...

@AndyButland just for reference, he's my reply to you from Slack the other day: > I've only looked at the PR, and not the code, but I think it looks...

@AndyButland I see that the `SecretEncryptor` class expects a secret key - eg. from config/AppSettings. Any thoughts on how this should be generated? (I'm aware this might differ if the...

It might be worth splitting this up in iterations. Eg. I think the encryption part has broader uses than just what we're doing here, so could be useful to have...

Probably a good idea, but a few things to consider: - Not all packages have assets (but probably doesn't hurt anyways if they are RCLs) - If RCLs become officially...

@alanwthom thanks for reporting 👍 I just tried spinning up a test site for 8.18, and I haven't been able to reproduce the error. I haven't tested with Azure Blob...

No worries ;) I couldn't reproduce the error with Azure blob storage either. I was waiting for some more time to test it, but good to hear you found the...

Hi @jonroberts24 Do you by any chance have some more information about the issue? I don't recall an error message like that being part of this package. Could you please...