Closed Bug 480480 Opened 14 years ago Closed 14 years ago

Finding cluster with api/register/chknode has trailing newline

Categories

(Cloud Services :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Mardak, Unassigned)

Details

$ 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
QA Contact: weave → general
You need to log in before you can comment on or make changes to this bug.