Page MenuHomePhabricator

ContentTranslation publishing error handling is too generic
Closed, ResolvedPublic1 Estimated Story Points

Description

In the publishing workflow the only error that is handled meaningfully is the request for CAPTCHA. Other errors, for example abuse filter, are just silently skipped. The user will often just see a "Publishing error" without further useful debugging information. It may even happen that the article would be indicated as published even if the page was not actually created (see T87510 for an example).

To reproduce, define an abuse filter that filters all Russian letters ([а-я]), and try to type and publish something in Russian.

An error must not be silently skipped. It must at least be logged to the console and to EventLogging, and the user must not see that an article was published if it wasn't actually published.

Event Timeline

Amire80 raised the priority of this task from to Medium.
Amire80 updated the task description. (Show Details)
Amire80 added a project: ContentTranslation.
Amire80 added a subscriber: Amire80.
gerritbot added a subscriber: gerritbot.

Change 190685 had a related patch set uploaded (by Amire80):
Log unhandled publishing error to the console

https://gerrit.wikimedia.org/r/190685

Patch-For-Review

Change 190685 merged by jenkins-bot:
Log unhandled publishing error to the console

https://gerrit.wikimedia.org/r/190685

KartikMistry claimed this task.

(It's not resolved.)

Amire80 raised the priority of this task from Medium to High.Feb 18 2015, 4:28 PM
Amire80 updated the task description. (Show Details)

Change 192786 had a related patch set uploaded (by Santhosh):
Publish module refactoring for title collision

https://gerrit.wikimedia.org/r/192786

Change 192786 merged by jenkins-bot:
Publish module refactoring for title collision

https://gerrit.wikimedia.org/r/192786

Change 204019 had a related patch set uploaded (by Santhosh):
Fix a js error when publishing fails because of unknown reasons

https://gerrit.wikimedia.org/r/204019

Change 204019 merged by jenkins-bot:
Fix a js error when publishing fails because of unknown reasons

https://gerrit.wikimedia.org/r/204019

Arrbee moved this task from In Progress to Done on the LE-Sprint-85 board.