railsでjsonを取得することができません

Question

前提・実現したいこと

cisco MerakiのScanning APIを使いPOSTされたjson形式のデータをRailsで取得したいです。

しかし、そのjsonを取得できません
rails側でエラーは出てないのですが、Merakiダッシュボードの方でエラーが表示されてます。

当方初心者なためそもそもrailsでjsonの取得のコーディングがあってるか怪しいです。

発生している問題・エラーメッセージ

Merakiダッシュボードで表示されてるエラー内容

”Did not match validator: http://wex01.cloud24.jp/personal_logs”

#app/controllers/personal_logs_controller.rb
validator ='xxxxxxxxxxxxxxxx'
secret = 'xxxxxxxxxx'

class PersonalLogsController < ApplicationController
  skip_before_action :verify_authenticity_token
  before_action :set_personal_log, only: [:show, :edit, :update, :destroy]

  def presenceapi
    if request.post?
      map = JSON.parse(params[:data]).with_indifferent_access
      if map[:secret] == secret
        map[:probing].each do |p|
          PersonalLog.create(ap_mac: p[:ap_mac], client_mac: p[:client_mac],
                          last_seen: DateTime.parse(p[:last_seen]),
                          rssi: p[:rssi])
        end
        render :text => '' and return
      else
        Rails::logger.warn("*** MERAKI req with bad secret '#{map[:secret]}'")
        rails 'access denied'
      end
    else
      render :text => MERAKI_VALIDATOR
    end
  end
end

#config/routes.rb
Rails.application.routes.draw do
  match 'personal_logs', :to => 'personal_logs#presenceapi', :as => :personal_logs, :via => [:get, :post]
end

Answer 1

Rails 5.1以降render :text =>''は廃止されました

render :plain => MERAKI_VALIDATOR

を利用してください

Comments

[kozica]

回答ありがとうございます！
上記の通り textからplainに変更して実行したところ
Validated http://wex01.cloud24.jp/personal_logs
と表示されました。
これは成功したということでしょうか？

[kozica]

また、昨日も回答して頂き本当にありがとうございます！！

[kozica]

rails側で下記エラーが出力されました
```
Started POST "/personal_logs" for 127.0.0.1 at 2018-07-26 10:07:07 +0900
(0.2ms) SET NAMES utf8, @@SESSION.sql_mode = CONCAT(CONCAT(@@sql_mode, ',STRICT_ALL_TABLES'), ',NO_AUTO_VALUE_ON_ZERO'), @@SESSION.sql_auto_is_null = 0, @@SESSION.wait_timeout = 2147483
LoadError (Unable to autoload constant PersonalLog, expected /home/ユーザ名/workspace/working/webex/app/models/personal_log.rb to define it):
```
app/models/personal_log.rbに問題があるようですが、ここには何も書いてないのですが、何か記述が必要なのでしょうか？

[asm]

エラーとしては、「定数PersonalLog(たぶんクラス)が見つからない」
なので、
「PersonalLog」でgrep検索して誰が参照してて、どこにあるのか
を把握することが対処の第一歩

[kozica]

$grep PersonalLog
を実行したところ、処理が多いせいなのかずっと処理中なようで結果が得られてないです。
他に調べたところ、PersonalLogの記述ミスがないか調べてみます

[asm]

git grep -w PersonalLog

gitのgrepコマンドの方が検索対象絞れて速いかもしれませんね

[kozica]

下記結果になりました。
$ sudo git grep -w PersonalLog
fatal: Not a git repository (or any of the parent directories): .git

[asm]

ありゃgitで管理してるわけじゃなかったのですね
申し訳ございません

Answer 2

cisco Merakiはよく知らないのですが、Merakiダッシュボードに表示される（はずの）Validatorを、GETリクエストの際に返してやる必要があるようです。
「Did not match validator」というメッセージの意味はそういうことだと思われます。

書かれているコードの場合、presenceapiメソッドの最後の~~render :text => ''~~render :plain => ''の''を、空文字ではなくValidatorを文字列として記述するようにしてみるとよさそうです。
※確かに別回答でasmさんが書かれてたようにrender :plainが正しそうですね。

---

初心者の方がいきなりいろいろやろうとすると難しそうなので、まずは単純にPOSTで送られてきたJSONデータを保存するところまでする、という想定で、イチから行うよう整理してみます。

rails newから始めると、だいたい以下のような流れになるかと思います。

rails new scanning-app
cd scanning-app
rails g scaffold Event data:text
rails db:migrate

ここでscaffoldで作っているのは、dataというカラムにJSONのオブジェクトのうち、data:というキーの値を全部テキストとして保存する、という想定です。リクエストを受け取るURLが/eventsだそうなので、モデルはEvent、コントローラーはEventsControllerにしています。

こんな感じで用意して、app/controller/events_controller.rbは、

class EventsController < ApplicationController
  skip_before_action :verify_authenticity_token

  VALIDATOR = "xxxxx"

  def index
    render :plain => VALIDATOR
  end


  def create
    @event = Event.new(data: params[:data].to_s)

    if @event.save
      render :plain => "OK"
    else
      render :plain => "NG"
    end
  end
end

みたいにします。VALIDATORは正しい値を入れてください。

ローカルで確認する場合は、

rails server

でサーバを立てると、http://localhost:3000/で立ち上がるはずです。
これで、curlを使って（curlがなければインストールしておいてください）、

curl http://localhost:3000/events

とすると、VALIDATORの文字列が帰ってくるはずです。同様に、POSTについては、

curl -X POST -H "Content-Type: application/json" -d '{"secret":"xxxx","version":"2.0","type":"DevicesSeen","data":{"a":"bbb"}}' http://localhost:3001/events

みたいな感じで適当にJSONを作ってPOSTリクエストを行ってみると、data:の値が保存されることが開発環境で確認できるかと思います。

これをproductionのサーバに持っていって、動作するのを確認したあとで、それからsecretの値チェックとか、data:の値をさらに項目ごとに分けてDBに保存するようにする、といったことを試してみるとよいかと思います。

Comments

[kozica]

回答ありがとうございます。
仰られたようにpresenceapiメソッドの最後の下記のように変更しましたが、同じ「Did not match validator」と表示されてしましました。
render :text => MERAKI_VALIDATOR

[kozica]

初心者なりに原因を考えてみましたがどうでしょうか？
・def presenceapiメソッドはコントローラーに記述してますが、記述場所としてあってますか？
・presenceapi.html.erbにはなにか呼び出しなどの記述は必要ですか？
・models/personal_logのほうにpresenceapiメソッドを記述する必要はないですか？

また、routes.rbも書き直したので投稿を修正致します。

[kozica]

cisco merakiのScanning APIについての説明ページを記載しておきます。
返す形式としてjsonになってないからなのかなと思ったですがどうでしょうか
https://www.cisco.com/c/m/ja_jp/meraki/documentation/mr/monitoring-and-reporting/location-analytics.html

[takahashim]

https://files.mtstatic.com/site_5814/2616/0?Expires=1532572128&Signature=hV3eDGD2EIMDcA0UfGSOS2~8EX7FP0MYfCojrEDyUU-woTr45cme7v2j1tXu5lEu4u9uyRs1bHdt43ozHAwJRMHfK~q40LORUSahEjLv6d~rmQ4H5PV-e1r9stKNWHI6LMx~L8l1AyFCcCr61bFREgDm6ycE7EgYj5FaQVSXXP0_&Key-Pair-Id=APKAJ5Y6AV4GI7A555NA を見ると、最初に返す値はPOSTのJSONとは関係なさそうです

[kozica]

確かにそのようですね！
今は下記エラーと戦っております。
Started POST "/personal_logs" for 127.0.0.1 at 2018-07-26 10:07:07 +0900
(0.2ms) SET NAMES utf8, @@SESSION.sql_mode = CONCAT(CONCAT(@@sql_mode, ',STRICT_ALL_TABLES'), ',NO_AUTO_VALUE_ON_ZERO'), @@SESSION.sql_auto_is_null = 0, @@SESSION.wait_timeout = 2147483
LoadError (Unable to autoload constant PersonalLog, expected /home/ユーザ名/workspace/working/webex/app/models/personal_log.rb to define it):

[takahashim]

うーん、personal_logs_controller.rbですが、これってどうやって作られましたか？ scaffoldしたんでしょうか。
ActiveRecordのモデルとして、PersonalLogクラスではなくPresenceクラスを使いたいのは何か理由があるんでしょうか。

[kozica]

personal_logs_controller.rbはscaffoldして作ったと記憶してます。
PersonalLogクラスではなくPresenceクラスを使いたい理由はないので、変える必要があるならすぐに変えます。
当方初心者なためそこらへんの関係性もまだ完璧に把握できておりません、、、

[kozica]

ご指摘にあったとおりPresence.createをPersonalLog.createに変更したのですが、同じエラーが返って来てしましました。

[kozica]

エラーで指摘されている行をみてみると、
```
map = JSON.parse(params[:data]).with_indifferent_access
```
この行を指摘されてるのでこの辺に原因があるのかなと思っています。
models/personal_log.rbには何か記述することがあるのでしょうか？
今何も記述してないのでそれも原因かと考えられます

[kozica]

あ、申し訳ありません
エラー内容が違いました。
Processing by PersonalLogsController#presenceapi as HTML
Parameters: {"version"=>"2.0", "secret"=>.......
・
・
"rssi"=>17, "ipv6"=>nil, "manufacturer"=>"Rivet Networks"}]}}}
Completed 500 Internal Server Error in 8ms (ActiveRecord: 0.8ms)
TypeError (no implicit conversion of ActionController::Parameters into String):
app/controllers/personal_logs_controller.rb:12:in `presenceapi'
上記エラー内容となっております

[takahashim]

その辺から違っているのであれば根本的になんとかした方が良さそうですね…。
回答の方にあとで追記します。

[kozica]

ものすごく細かく説明して頂き本当にありがとうございます！（泣）
無事うまく動いてeventテーブルにデータが格納されるようになりました！

本当にありがとうございました！！

[takahashim]

無事に動いてよかったですね！
もっともこれは最初のとっかかりで、最後のところに書いた通り、dataの項目に分けて保存したり、secretやtypeのチェックをしたり等が必要になるかと思いますが、一回動くところまでいけば、続きの作業もやりやすくなるかと思います。がんばってください。