The GitHub Actions job "Required Checks" on 
texera.git/feat/6929-byo-s3-storage-foundation has failed.
Run started by GitHub user mengw15 (triggered by mengw15).

Head commit for run:
75660d6a4e16f22dbf6c84230f9bc701d6b5acfd / mengw15 
<[email protected]>
fix(storage): locate URI keys by raw segment in decodeURI as well

warehouseFromURI was changed to split the raw path so a percent-encoded slash
cannot forge segments, but decodeURI -- which finds `wid`/`eid` by searching the
same segments -- was left splitting the decoded path. A URI whose warehouse name
contained `%2F` therefore read as `/wh/a/wid/999/wid/1/...` there, and the key
search returned the injected `wid` rather than the real one, putting one
execution's data under another's storage key. Python's decode_uri already split
the raw path, so the two languages also disagreed about which execution a URI
identified.

Split the raw path here too. Legitimate URIs are unaffected: the characters the
serialized port identity contributes -- '(', ')', ',', '=' -- are all legal in a
path segment and are never percent-encoded, so raw and decoded are identical for
anything the factory writes.

Report URL: https://github.com/apache/texera/actions/runs/30400230635

With regards,
GitHub Actions via GitBox

Reply via email to