Page MenuHomePhabricator

Redact key from error messages.
AbandonedPublic

Authored by Samwilson on Nov 1 2018, 1:44 AM.

Details

Reviewers
MusikAnimal
MaxSem
aezell
kaldari
Commits
rWSOCR9c61f71a9ecb: Switch to 'document text detection' instead of 'text detection'
rWSOCR529bbf4f1399: Redact key from error messages.
Patch without arc
git checkout -b D1124 && curl -L https://phabricator.wikimedia.org/D1124?download=true | git apply
Summary

Bug: T208374

Diff Detail

Repository
rWSOCR Wikisource OCR
Branch
document-text-detection
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 3283
Build 5444: arc lint + arc unit

Event Timeline

Samwilson created this revision.
src/Ocr.php
77

I don't like that stack trace will be lost here.

  • Include previous exception
Samwilson added inline comments.
src/Ocr.php
77

Yeah, me neither. I meant to at least put the previous exception. Done now.

I'm pretty sure there's a better way of doing what we want here, but I didn't want to spend too long on this. :)

Shall we re-write in Symfony?

This revision is now accepted and ready to land.Nov 1 2018, 10:40 PM
  • Switch to 'document text detection' instead of 'text detection'

Given that we are trying to minimize risky changes, this feels like something we should wait on.

While our testing says this is OK, our users expect the tool to behave in the way it currently does. Changing that behavior right now seems like a change we'd want to communicate more broadly before it happens.

Good point!

And I've stuffed up arc diff in someway too... I'm not sure why this has got commits from earlier in it. We'll move to Gerrit before any more changes anyway...