JustAnotherArchivist

@myfingerhurt I'm not sure about the other things you mentioned, but about this: > `mitmweb mitmweb -s mitm_anatomy.py 2>&1 > /tmp/mitm.log` did save both stderr and stdout to `mitm.log` but...

As a datapoint, I was running a very similar command to the above, and its memory usage kept growing until it OOM'd after 12 million URLs at about 28 GB...

Any update on this?

I think it's worth going for full compatibility even though that requires a significant refactor. I wanted to look into this a while ago, but it got blocked by the...

@bwoodsend The downside is that with such a notation, you will always need to load the *entire* log into memory for decoding (without special trickery) rather than looping over the...

#270 and colleagues addressed leaks during encoding, not decoding. I don't see any relevant commits that might have changed anything on decoding.

I commented on this before, but it's in a resolved comment, so repeating (and expanding) for visibility... I'm not sure the surrogates or other weird characters in `indent` are anything...

The tests are failing now due to the negative `indentLength`, I think. The buffer size calculations all use that variable, and when it's negative, it won't correctly reserve buffer space...

I just tested this with current versions (CPython 3.10.1 and ujson 5.4.0). On my machine, the two have virtually identical timings. So it's better now but still not great since...

Yeah, this was known internally for a while. The negative sign triggers the number detection code, but it never verifies that it has seen at least one digit. The same...