478680 - Make preview do something sensible for non-HTML / text files

Status: RESOLVED INVALID

(Skywriter Graveyard :: Server, defect, P3)

Description

[Dion Almaer]

---
I've just run a simple hello.php file with Bespin

<?php
echo('hello world!');
?>
---

Apache 500'd so it appears that it didn't potentially get to the python server itself?

Comment 1

[Reed Loden [:reed]]

What do you mean by "preview"? You mean actually _run_ the PHP or just display the text?

Comment 2

[Dion Almaer]

View the text. We obviously don't want to run arbitary PHP code :)

Comment 3

[Reed Loden [:reed]]

Ok, good... just double checking! :)

Comment 4

[Gordon P. Hemsley [:GPHemsley]]

There's an assignee set, so these should be ASSIGNED. If you're listed as the assignee, and you don't plan on working on this bug, please remove yourself from the assignee field and set this back to NEW.

Comment 5

[Gordon P. Hemsley [:GPHemsley]]

This is a mass migration from Mozilla Labs :: Bespin to Bespin :: Server.

Comment 6

[Ben Galbraith]

We should either disable preview for non-HTML and plain text or explicitly convert mime types to plain text for everything non-HTML

Comment 7

[Kevin Dangoor]

ACETRANSITION

The Skywriter project has merged with Ajax.org's Ace project (the full server part of which is their Cloud9 IDE project). Background on the change is here:

http://mozillalabs.com/skywriter/2011/01/18/mozilla-skywriter-has-been-merged-into-ace/

The bugs in the Skywriter product are not necessarily relevant for Ace and quite a bit of code has changed. For that reason, I'm closing all of these bugs. Problems that you have with Ace should be filed in the Ace issue tracker at GitHub:

https://github.com/ajaxorg/ace/issues