[interchange] Improve application/json POST support

David Christensen david at endpoint.com
Tue Dec 30 15:28:36 UTC 2014


> On Dec 29, 2014, at 5:26 PM, Peter <peter at pajamian.dhs.org> wrote:
> 
> On 12/30/2014 10:02 AM, David Christensen wrote:
>> +use JSON ();
> 
> ...
> 
>> +					$CGI::json_ref = JSON::from_json($$CGI::post_ref);
> 
> You've added in JSON as a requirement here, but it's not (last time I
> checked) part of core perl and it's not listed as a module in
> Bundle::Interchange (or even Bundle::InterchangeKitchenSink).  Please
> add in code that makes this optional so we don't break existing IC sites
> when they upgrade.

I'd checked, and I thought as of 5.14 it is core (though maybe that is the PP JSON).  Despite that, there are still plenty of supported perls where this is not the case.  I'll make the code a no-op in case JSON is not available.

> Mike, should JSON be a part of KitchenSink?

+1 (though it looks like a moot issue, as it's already done).

Regards,

David
--
David Christensen
End Point Corporation
david at endpoint.com
785-727-1171






More information about the interchange-cvs mailing list