🦋 Changeset detected

Latest commit: 8542698

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

#1100 Bundle Size — 1.92MiB (~+0.01%).

8542698(current) vs cc834ce main#1098(baseline)

Bundle metrics   2 changes 1 regression

	Current #1100	Baseline #1098
Initial JS	1.92MiB(~+0.01%)	1.92MiB
Initial CSS	0B	0B
Cache Invalidation	100%	0%
Chunks	1	1
Assets	1	1
Modules	1049	1049
Duplicate Modules	0	0
Duplicate Code	0%	0%
Packages	158	158
Duplicate Packages	4	4

Bundle size by type   1 change 1 regression

	Current #1100	Baseline #1098
JS	1.92MiB (~+0.01%)	1.92MiB

Bundle analysis report Branch brynn/doc-1675-allow-json-primit... Project dashboard

@marclave we may want to consider this but it will have significant downstream impacts as "AnyObject" is not the true valid return type of Content-Type: 'application/json'

I am also a little confused about the original issue as it looks like their request was using Content-Type: 'text/plain' in which case no parsing should be going on.

Going to close this for now until @geoffgscott gets a chance to chat with @marclave @hanspagel about how we want to proceed

@hanspagel We should take a look at how to get this in safely

I think we should focus on the shift to the new proxy first (because it affects basically all users) and get this PR here in after we’ve migrated to the new proxy.

I think I fixed this here today: #1090

I saw a few duplicate lines that looked like the result of a merge conflict.