Chat shows raw HTML output

Bug #676205 reported by XavierAntoviaque
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Humanity Project
Confirmed
High
Unassigned

Bug Description

On hackit.cx, when I click on "Start mission", the IM window appears, but it is filled with HTML code. It goes too fast to really see what it is about, and it hangs the browser, maybe an error message or a traceback. (cf screenshot)

Revision history for this message
XavierAntoviaque (xavier-antoviaque) wrote :
Changed in hackit:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Vlad Dragu (vlad-dragu)
milestone: none → alpha2.1
Vlad Dragu (vlad-dragu)
Changed in hackit:
status: Confirmed → Fix Committed
Changed in hackit:
status: Fix Committed → Fix Released
Revision history for this message
XavierAntoviaque (xavier-antoviaque) wrote :

I still have the issue - I merged the latest changes from your branch Vlad, and it still shows raw HTML output.

What is the issue exactly? Is it an exception raised during an Ajax call?

This bug is pretty nasty, it makes the browser hang completely - it would be good to add some error handling here (not just fixing the exception), in case another bug creates the same effect later on.

Changed in hackit:
status: Fix Released → Confirmed
Revision history for this message
david blanchard (david-blanchard) wrote :

ok - it didn't happen on my side

Revision history for this message
Vlad Dragu (vlad-dragu) wrote :

The html out put is an warning issues by OWA which tries to set a cookie after some content has been outputted to the browser.
I ssh-ed in hackit.cx and fixed it. Also commited to the repository.
As far as detecting filtering the errors, i don't think i can do it on the front end. The js just outputs whatever it receives from the ajax call. One way would be be just to change the error_reporting level for that ajax call

Changed in hackit:
status: Confirmed → Incomplete
Revision history for this message
XavierAntoviaque (xavier-antoviaque) wrote :

Have a look at JSON - passing values this way will make it easier to handle errors: http://php.net/manual/en/book.json.php

If you want I can postpone this part to alpha3.0 though.

Changed in hackit:
status: Incomplete → Confirmed
Changed in hackit:
milestone: alpha2.1 → alpha3.0
Changed in hackit:
assignee: Vlad Dragu (vlad-dragu) → nobody
milestone: alpha3.0 → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.