Should this library use json-iterator?

[errordeveloper]

Kubernetes uses github.com/json-iterator/go instead of encoding/json, should this library switch also?

[dims]

sure!

[neolit123]

/assign
although need to find bandwidth.
i'm yet to check #15
but i suspect discrepancy between the current backends: yaml2 vs encoding/json.

json-iter does support case sensitive JSON.

[neolit123]

/kind feature
/priority important-longterm

[errordeveloper]

json-iter does support case sensitive JSON.

You mean it can detect unknown keys in case-sensitive manner?

[neolit123]

You mean it can detect unknown keys in case-sensitive manner?

yes. i have a PR in the works to enable json-iter.
whether it makes it in is another question as it requires a minor breaking change.

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[fejta-bot]

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

[neolit123]

initial reaction was that there is no good justification for this change:
#19
/lifecycle frozen

[neolit123]

Looks like there is desire to switch k/k away from json iter.

kubernetes/kubernetes#105030

It suggests using a fork of stdlib, that supports the missing valudation features.