Closed Bug 1035345 Opened 12 years ago Closed 10 years ago

The validator returns an error when escaping the results takes too long

Categories

(Marketplace Graveyard :: Developer Pages, defect, P3)

Avenir
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: 316k, Unassigned)

Details

(Whiteboard: [marketplace-transition])

Attachments

(1 file)

6.84 MB, application/zip
Details
Attached file man-html.zip
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:30.0) Gecko/20100101 Firefox/30.0 (Beta/Release) Build ID: 20140605174243 Steps to reproduce: I submitted a .zip packaged app with > 2000 HTML files. Every time I try to resubmit it, the validator gives me the same error. The zip file is attached. Actual results: La validation de votre application a renvoyé 1 erreur. Une erreur est survenue pendant le contact du serveur. Aka : The validation returned 1 error. Error while contacting the server. I suspect the connexion timed out because of the large amount of files to analyze. Expected results: Since there was no javascript in the HTML files, the validation should have returned no error, or at least, a report should have been produced.
This gives the same error as AMO when the validation takes too long on the escaping step (bug 957158). The fix for AMO is being deployed on Wednesday so if that fixes AMO let's port it to marketplace.
Assignee: nobody → mstriemer
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
This is fixed in olympia's PR https://github.com/mozilla/olympia/pull/127. This should be ported over to zamboni so the HTML escaping is performed in the celery task.
bug 957158 is now on production and it does indeed solve this problem. I'll look into porting it over.
Summary: The validator returns an error without any validation report when too much html files are sent in a packaged app. → The validator returns an error when escaping the results takes too long
Assignee: mstriemer → nobody
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Whiteboard: [marketplace-transition]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: