including a URL in defaultPadText in settings.json causes things to fail

I tried to add a URL into the defaultPadText in the settings.json and restart etherpad, but it fails:

[2011-11-29 08:23:25.941] [ERROR] console - There is a syntax error in your settings.json file

URLs should be valid JSON, so I'm not sure I understand why this is happening. I tried to escape the slashes, but it made no difference.

[JohnMcLear]

And if you remove the URL it works fine?

Can you paste a copy of what breaks and what doesn't?

It's possible the : is the problem.

Thanks

-----Original Message-----
From: Riseup Labs [mailto:reply@reply.github.com]
Sent: 29 November 2011 16:24
To: John McLear
Subject: [etherpad-lite] including a URL in defaultPadText in settings.json causes things to fail (#249)

I tried to add a URL into the defaultPadText in the settings.json and restart etherpad, but it fails:

[2011-11-29 08:23:25.941] [ERROR] console - There is a syntax error in your settings.json file

URLs should be valid JSON, so I'm not sure I understand why this is happening. I tried to escape the slashes, but it made no difference.

---

Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/issues/249
This email and its attachments may be confidential and are intended solely for the use of the individual to whom it is addressed. Any views or opinions expressed are solely those of the author and do not necessarily represent those of the organisation from which this email originated. If you are not the intended recipient of this email and its attachments, you must take no action based upon them, nor must you copy or show them to anyone. Please contact the sender if you believe you have received this email in error. This email was sent by School Email - Safe Webmail and Hosted Email for Schools

Yes, when I remove the URL it works. I tried a number of different things to try and get it to work with the URL, but each one failed.

I dont have any additional information about what breaks and what doesn't, the only information I have is what appears in the logs when things are restarted, which is what I included above.

If there is a way to get more details on what breaks, I'm happy to generate it, I just am not sure I understand how I can.

[JohnMcLear]

Default settings.json has:

http://j.mp/ep-lite\n

so it should be fine if you break the /'s

-----Original Message-----
From: Riseup Labs [mailto:reply@reply.github.com]
Sent: 29 November 2011 22:30
To: John McLear
Subject: Re: [etherpad-lite] including a URL in defaultPadText in settings.json causes things to fail (#249)

Yes, when I remove the URL it works. I tried a number of different things to try and get it to work with the URL, but each one failed.

I dont have any additional information about what breaks and what doesn't, the only information I have is what appears in the logs when things are restarted, which is what I included above.

If there is a way to get more details on what breaks, I'm happy to generate it, I just am not sure I understand how I can.

---

Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/issues/249#issuecomment-2951597
This email and its attachments may be confidential and are intended solely for the use of the individual to whom it is addressed. Any views or opinions expressed are solely those of the author and do not necessarily represent those of the organisation from which this email originated. If you are not the intended recipient of this email and its attachments, you must take no action based upon them, nor must you copy or show them to anyone. Please contact the sender if you believe you have received this email in error. This email was sent by School Email - Safe Webmail and Hosted Email for Schools

hm, shouldn't that be:

http://j.mp/ep-lite\n

if you were actually escaping the /'s?

It appears that if I escape ALL the backslashes, then it causes the error, but by escaping just the backslashes after the http: makes it work... that seems a bit strange.

[JohnMcLear]

Yep, a bit strange, we will try to find a way to provide better support for this issue in the future.

[JohnMcLear]

I would suggest we write something that attempts to parse the settings.json when the init script is initiated so users get better debug output from the init script.

[marcelklehr]

Issue should be gone, since we now use v8 to parse settings.json.