Repository navigation
BOT-3EN: snekbox hits a recusion exceded exception in pathlib when parsing lots of nested directories and 500s #172
Description
Summary
Sending this code:
# https://discord.com/channels/267624335836053506/267659945086812160/1084230562622996601
import os
x = ""
for _ in range(1000):
x += "a/"
os.mkdir(x)
open(f"{x}hello.txt", "w").write("test")Got this error:
# https://discord.com/channels/267624335836053506/267659945086812160/1084230566007799818
ClientResponseError: 500, message='Internal Server Error', url=URL('http://snekbox.default.svc.cluster.local/eval')
Which, according to grafana, is snekbox dying somewhere in pathlib while calling scandir.
Sentry Issue: BOT-3EN
Metadata
Metadata
Assignees
Labels
No labels
Activity
ionite34 commentedon Mar 11, 2023
Related to Cpython#102613. We should probably switch to
globinstead of pathlib for now.Add unit test for deeply nested path file parsing
ionite34 commentedon Mar 12, 2023
Also related to python/cpython#100282, could remove check after fix upstream
zmievsa commentedon Mar 22, 2023
@ionite34 Barney Gale has just merged #100282!