Where are you seeing these JSON parse errors? These files are used on the
server-side and the parsers are fairly forgiving.

-Stanton

Sent from a mobile device. Please forgive brevity or typos.
On Nov 18, 2014 3:39 PM, "Ichiro Furusato" <ichiro.furus...@gmail.com>
wrote:

> Hi,
>
> I've recently started working with Shindig for a project and have found
> that I am getting JSON parsing errors on unmodified files in the Shindig
> source. On investigating I found that these files contained Apache license
> boilerplate in the form of "/* */" style comments, which is invalid JSON
> as according to ECMA-404 (the JSON standard).
>
> The files containing comments are:
>
>   ./config/gadget-admin.json
>   ./config/oauth2.json
>   ./config/oauth.json
>   ./content/sampledata/canonicaldb.json
>
>
> ./java/gadgets/src/test/resources/org/apache/shindig/gadgets/oauth2/oauth2_test.json
>
> The last test file will not be a problem in production, but the other
> four show up in the distribution jars. The first three are found in
> shindig-gadgets-2.5.1.jar, the fourth in shindig-social-api-2.5.1.jar.
>
> Given I'm building via Maven and won't have control over the incoming
> jars (which will contain these files) I'm wondering how others have gotten
> around this problem, and if it's possible to simply remove the comments
> in the source files (which would seem to be the simplest solution).
> License boilerplate is meant to protect the intellectual property of the
> project -- it doesn't have to be included in every file, and certainly
> not in configuration files (which arguably have little IP to protect).
>
> I suppose I'm also curious as to why this problem apparently isn't
> affecting delivery of the project (i.e., it obviously passes its tests,
> the mvn -Prun demo works). I'm still new to working with Shindig so
> perhaps I'm missing something here...
>
> Cheers,
>
> Ichiro
>

Reply via email to