rails to_json remove quote

Question

I am working the the LogMeIn Central API and in the body of my request I need to send some Json. I have this:

  host_ids = LmiHost.all.collect {|lmi| lmi.host_id}.join ', '
  create_servicetag_report_request.body = {hostIds: host_ids, fields: 'ServiceTag'}.to_json

This turns the body into

{\"hostIds\":\"5888, 6225, 214752\",\"fields\":\"ServiceTag\"}

how can i remove the

\"

from this section:

\"5888, 6225, 214752\"

it is not suppose to have quotes around it.

I am using Ruby on Rails

JSON strings are supposed to be wrapped by double-quotes - are you sure the API is expecting JSON, as opposed to a comma-separated list of IDs? — Brian, Dec 20 '16 at 20:31
positive, see here http://developer.logmein.com/api/hostactions/#system-inventory — Devon Quick, Dec 20 '16 at 20:38

whodini9 · Accepted Answer · 2016-12-21T19:12:16.627

3

The reason to_json adds the \" (escaped quotations) is because it is converting hostIds as a string. In your rails console try this to see the difference.

{"hostids":[0,1,2,3]}.to_json
=> "{\"hostids\":[0,1,2,3]}"

{"hostids":"[0,1,2,3]"}.to_json
=> "{\"hostids\":\"[0,1,2,3]\"}"

This can be seen another way by trying: puts [1,2,3,4] vs puts "[1,2,3,4]"

Ultimately I would refer to the LMI Central API to figure out exactly how multiple hostIds can be sent.

edited Dec 21 '16 at 19:12

answered Dec 20 '16 at 20:32

whodini9

1,434
13
18

`host_ids` is a string because the OP explicitly made it one. using `join ', '` – engineersmnky Dec 20 '16 at 20:37
Right, and this is why it is displaying as such. If OP wants valid json, a list of ids can only be a comma separated string or an array/array of hashes – whodini9 Dec 20 '16 at 20:39
1

He just added the link to their API Doc it says: "hostIds": [, , ...], so it should be an Array not a String. – whodini9 Dec 20 '16 at 20:40
Yeah I commented on it. I was more or less mentioning your phrasing of *"It looks like hostIds is a string."* – engineersmnky Dec 20 '16 at 20:41
1

It looks exactly like it :) – whodini9 Dec 20 '16 at 20:42
thanks guys, that makes sense. going to test it as a array in a few minutes and ill post back – Devon Quick Dec 20 '16 at 20:45

score 2 · Answer 2 · answered Dec 20 '16 at 20:28

2

You can use JSON.parse(create_servicetag_report_request.body[:hostIds]) to parse it.

answered Dec 20 '16 at 20:28

ydaniju

400
4
11

score 0 · Answer 3 · answered Dec 20 '16 at 20:52

Just remove the join part from this line:

host_ids = LmiHost.all.collect { |lmi| lmi.host_id }.join ', '

join joins your array of ids into a comma separated string. This doesn't seems to be what you want.

Btw. you can shorten .collect { |lmi| lmi.host_id } to map(&:host_id):

create_servicetag_report_request.body = {
  hostIds: LmiHost.all.map(&:host_id), 
  fields:  'ServiceTag'
}.to_json

rails to_json remove quote

3 Answers3