Last modified: 2014-06-07 01:01:54 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68139, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 66139 - VisualEditor: Link inspector should catch 400 errors from sending overly-long queries to the API
VisualEditor: Link inspector should catch 400 errors from sending overly-long...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Normal major
: VE-deploy-2014-06-12
Assigned To: Alex Monk
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2014-06-04 20:42 UTC by James Forrester
Modified: 2014-06-07 01:01 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description James Forrester 2014-06-04 20:42:08 UTC 
This results in a quite bad outcome for the user, totally inexplicably. Instead we should just catch these errors and state them as invalid links (or have a different message?).
Comment 1 Alex Monk 2014-06-06 20:17:22 UTC 
Gerrit change #138016 should make it so that these show invalid title warnings. Though I haven't got CirrusSearch installed on my dev wiki so haven't tested it against the request_too_long Cirrus error.
Comment 2 Gerrit Notification Bot 2014-06-06 20:17:37 UTC 
Change 138016 had a related patch set uploaded by Alex Monk:
Don't sent invalid titles from the link inspector to the API

https://gerrit.wikimedia.org/r/138016
Comment 3 Gerrit Notification Bot 2014-06-07 01:01:54 UTC 
Change 138016 merged by jenkins-bot:
Don't send invalid titles from the link inspector to the API

https://gerrit.wikimedia.org/r/138016
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links