Page MenuHomeSoftware Heritage

T435-c
ClosedPublic

Authored by jbertran on Jun 10 2016, 4:06 PM.

Diff Detail

Repository
rDWAPPS Web applications
Branch
T435
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 175
Build 265: Software Heritage Python tests
Build 264: arc lint + arc unit

Event Timeline

jbertran retitled this revision from to T435-c.
jbertran updated this object.
jbertran edited the test plan for this revision. (Show Details)

Hey,

You should somehow mark what fields failed to decode and got escaped; in the current implementation there is no way to distinguish between

some_input = {
            'a': 'something',
            'b': 'someone',
            'c': b'a name \xff',
            'd': b'an email \xff',
}

and

some_input = {
            'a': 'something',
            'b': 'someone',
            'c': b'a name \\xff',
            'd': b'an email \\xff',
}
jbertran edited edge metadata.

Track and show which swh fields generate decoding errors

olasd added a reviewer: olasd.
This revision is now accepted and ready to land.Jun 13 2016, 2:20 PM
This revision was automatically updated to reflect the committed changes.