chicken-janitors
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Chicken-janitors] #1070: [spiffy] [PATCH] Add mime type for json


From: Chicken Trac
Subject: Re: [Chicken-janitors] #1070: [spiffy] [PATCH] Add mime type for json
Date: Fri, 22 Nov 2013 13:51:31 -0000

#1070: [spiffy] [PATCH] Add mime type for json
--------------------------+-------------------------------------------------
  Reporter:  RvdH         |       Owner:  sjamaan  
      Type:  enhancement  |      Status:  new      
  Priority:  minor        |   Milestone:  someday  
 Component:  unknown      |     Version:  4.8.x    
Resolution:               |    Keywords:  mime json
--------------------------+-------------------------------------------------

Comment(by sjamaan):

 Thanks for taking the time to make a patch.  However, I wonder how often
 people will be serving static files with a .json extension.  That doesn't
 make much sense (json makes more sense as a data exchange format).

 I'd prefer if the default extension->type mapping stayed as short and
 simple as possible.  This ensures the spiffy library doesn't become too
 bloated with string data and doesn't make searching the list too slow.
 Adding just one more type to the default list won't be bad, but adding one
 type one hundred times will :)  Of course, anyone's free to parameterize
 it in their own Spiffy setup.

 Do you use static .json-files? Does it cause problems when they're served
 up as application/octet-stream?

 I'm not 100% decided yet, but the advantage of adding it to the default
 list isn't clear to me yet. If there's a large body of software or
 emerging common practice out there that uses this extension, I wouldn't
 mind adding it.

-- 
Ticket URL: <http://bugs.call-cc.org/ticket/1070#comment:1>
Chicken Scheme <http://www.call-with-current-continuation.org/>
Chicken Scheme is a compiler for the Scheme programming language.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]