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

RESOLVED INVALID

Status

Skywriter
Server
P3
normal
RESOLVED INVALID
9 years ago
7 years ago

People

(Reporter: Dion Almaer, Assigned: Kevin Dangoor)

Tracking

Trunk
x86
Mac OS X

Details

(Reporter)

Description

9 years ago
---
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?
(Reporter)

Updated

9 years ago
Assignee: nobody → kdangoor
Priority: -- → P3
Whiteboard: server
What do you mean by "preview"? You mean actually _run_ the PHP or just display the text?
(Reporter)

Comment 2

9 years ago
View the text. We obviously don't want to run arbitary PHP code :)
Ok, good... just double checking! :)
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.
Status: NEW → ASSIGNED
This is a mass migration from Mozilla Labs :: Bespin to Bespin :: Server.
Component: Bespin → Server
Product: Mozilla Labs → Bespin
QA Contact: bespin → server
Whiteboard: server

Updated

9 years ago
Target Milestone: Future → ---

Comment 6

9 years ago
We should either disable preview for non-HTML and plain text or explicitly convert mime types to plain text for everything non-HTML
Summary: Previewing a php file gives a 500 error → Make preview do something sensible for non-HTML / text files
(Assignee)

Comment 7

7 years ago
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
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.