Closed Bug 480480 Opened 14 years ago Closed 14 years ago
Finding cluster with api/register/chknode has trailing newline
$ curl https://auth.services.mozilla.com/0.3/api/register/chknode/harikaried sj-weave01.services.mozilla.com $ Notice the next prompt $ starts on its own line. This is causing problems with the generated clusterURL because now it's... https://sj-weave01.services.mozilla.com / Not sure what's going on for the server, but we can work around it on the client by just doing res.data.trim()
Pushed a workaround for now. I'll leave the bug open for figuring out why the newline is there to begin with. http://hg.mozilla.org/labs/weave/rev/dce3e928fc11
FYI, we have plans to change what the server returns to be a URL instead of a domain name.
Adding Jeremy so that he can fix the current servers.
Newline has been removed.
Marking fixed per comment #4 and testing with curl shows no newline. I suppose we can just leave the workaround for now?
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Yeah, it doesn't really matter either way. We'll want to parse full URLs in the future, probably.
http://hg.mozilla.org/labs/weave/rev/561603ff8747 Remove trailing newline bug 480480 work-around: trim().
Component: Weave → General
Product: Mozilla Labs → Weave
Target Milestone: -- → ---
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.