475207 - allow cross-site XMLHttpRequest for tinderbox quickparse.txt output

Status: RESOLVED FIXED

(Webtools Graveyard :: Tinderbox, defect)

Description

[Justin Dolske [:Dolske]]

Bug 393401 allowed this for the JSON output. I'd like to do the same thing for the quickparse.txt output. [I'd rather load the quickparse.txt format for my task, it's only ~2K vs. 250K for the JSON.]

This is apparently going to be an Apache/IT config change, but I thought I'd file it here first.

This header needs to be sent in the quickparse.txt HTTP reply:

Access-Control-Allow-Origin: *

[Note that the spec has changed since bug 393401]

Comment 1

[Justin Dolske [:Dolske]]

Attachment: Patch v.1

Patch to implement a .htaccess that adds the needed header.

Comment 2

[Justin Dolske [:Dolske]]

Attachment: Patch v.2

Comment 3

[Reed Loden [:reed]]

Comment on attachment 358663 [details] [diff] [review]
Patch v.2

This isn't valid. Put the <FilesMatch> block under the <Directory /var/www/html/tinderbox/*/> section.

Comment 4

[Justin Dolske [:Dolske]]

Attachment: Patch v.3

Comment 5

[Justin Dolske [:Dolske]]

Attachment: Patch v.4

WITH FEELING.

Comment 6

[Reed Loden [:reed]]

Comment on attachment 358665 [details] [diff] [review]
Patch v.4

>+			Header set Access-Control-Allow-Origin "*"

Seems like this should be tabbed once more, but I don't care.

r=me. A+++++++++ would review again.

Comment 7

[Justin Dolske [:Dolske]]

Checking in webtools/tinderbox/INSTALL;
  new revision: 1.7; previous revision: 1.6

Comment 8

[(not currently active) Ted Mielczarek]

What, you couldn't fix the JSON output to match the spec while you were screwing around in there?

Comment 9

[cls]

patch v.4 checked into tbox1_20080527_cls_branch