Page MenuHomePhabricator

Errors in Parsoid v2 entry point
Closed, ResolvedPublic0 Estimated Story Points


We are just starting to do some mass testing through restbase, and noticed failures from Parsoid when using the v2 GET entry point. Test cases:

Event Timeline

GWicke raised the priority of this task from to Needs Triage.
GWicke updated the task description. (Show Details)
GWicke changed Security from none to None.
GWicke added a project: Parsoid.
GWicke edited subscribers, added: Arlolra; removed: Aklapper.
GWicke added a subscriber: GWicke.

Ah, this is from stripping at !topLevel pipelines. Fixing.

Arlolra triaged this task as High priority.Dec 2 2014, 9:52 PM

Change 177142 had a related patch set uploaded (by Arlolra):
Fix failures from v2 endpoint


Change 177142 merged by jenkins-bot:
Fix failures from v2 endpoint

From the commit message,

Although this resolves the crashes, I'm unsatisfied with it as a
proper fix to the underlying issue. There are many places throughout
the codebase where we serialize and then parse document fragments
that should be instrumented to store and unpack data-* attributes.

FWIW, I haven't seen any significant number of crashes since.

@Arlolra, I think this issue can be closed. I reported the remaining crashers in a separate page (all long lists that time out). Can't find it right now though.

Edit: T88915

@GWicke The crashers are fixed but the robustness mentioned in T76518#804047 hasn't. Should I close this an open another then?