railsでAPIを用いてJSON形式のデータを取得したい

Question

前提・実現したいこと

さくらインターネットのAPIを用いてデータをjson形式で取得したいのですが、httpのレスポンスは200番で返ってきているのにうまくデータを取得することができません。お力添えをお願いいたします。

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

message: HTTP ERROR: code=200 message=OK

該当のソースコード

require 'net/http'
require 'net/https'
require 'uri'
require 'json'

class AddressController < ApplicationController
    def show

      
      params = URI.encode_www_form({token: 'token'})
      # URIを解析し、hostやportをバラバラに取得できるようにする
      uri = URI.parse("https://api.sakura.io/datastore/v1/channels?#{params}")

  
      # 新しくHTTPセッションを開始し、結果をresponseへ格納
      https = Net::HTTP.new(uri.host, uri.port) do |https|
        https.use_ssl = true
        https.verify_mode = OpenSSL::SSL::VERIFY_NONE
        response = https.start
        # 接続時に待つ最大秒数を設定
        https.open_timeout = 5
        # 読み込み一回でブロックして良い最大秒数を設定
        https.read_timeout = 10
        # ここでWebAPIを叩いている
        # Net::HTTPResponseのインスタンスが返ってくる
        #https.start(uri.request_uri)
      end


      # 例外処理の開始
      begin
        print "response=" 
        puts response
        puts uri
        # responseの値に応じて処理を分ける
        case response
        # 成功した場合
        when Net::HTTPSuccess
          # responseのbody要素をJSON形式で解釈し、hashに変換
          @result = JSON.parse(response.body)
          # 表示用の変数に結果を格納
          @id = @result["results"][0]["id"]
          @channel = @result["results"][0]["channel"]
          @datatime = @result["results"][0]["datatime"]
          @value_str = @result["results"][0]["value_str"]
          @value_num = @result["results"][0]["value_num"]
          @type = @result["results"][0]["type"]
        # 別のURLに飛ばされた場合
        when Net::HTTPRedirection
          @message = "Redirection: code=#{response.code} message=#{response.message}"
        # その他エラー
        else
          @message = "HTTP ERROR: code=#{response.code} message=#{response.message}"
        end
      # エラー時処理
      rescue IOError => e
        @message = e.message
      rescue TimeoutError => e
        @message = e.message
      rescue JSON::ParserError => e
        @message = e.message
      rescue => e
        @message = e.message
    end
  end
end

試したこと

デバッグとしてcase responseの前にresponseとuriの中身を表示してみました。
内容は以下の通りです
response=#ActionDispatch::Response:0x000000000db58a30
https://api.sakura.io/datastore/v1/channels?token=token

補足情報（FW/ツールのバージョンなど）

ruby 2.6.3p62
rails 6.0.0

Answer 1

なんというか、面白いバグ作りましたね。

rubyのブロック内で定義した変数はブロック外に持ち出せません。
そのため、

response = https.start

と

@result = JSON.parse(response.body)

時点のresponseは別物、後者は正確にはRailsがContorollerに用意しているresponseメソッドです。

とりあえずの対処療法ですと、

response = nil
https = Net::HTTP.new(uri.host, uri.port) do |https|
# 省略

と、ブロックの前に変数を定義するとよいでしょう。

追記:
失礼しました。
よく、Net::HTTP.newについて調べたら、そもそもブロックを取らないメソッドでした。
そのため、該当箇所は完全に誤りでした。

      response = Net::HTTP.start(uri.host,
          use_ssl: true, verify_mode: OpenSSL::SSL::VERIFY_NONE,
          open_timeout: 5, # 接続時に待つ最大秒数を設定
          read_timeout: 10 # 読み込み一回でブロックして良い最大秒数を設定
        ){|https| https.get(uri.request_uri)}

Comments

[hokahomu]

ご回答ありがとうございます。
当方rails初心者な物でして、様々なwebサイトを参考にしてやってみたので大目に見てくださると助かります。
ご回答いただいた通りに変数を定義したのですが、undefined method `code' for nil:NilClass　とエラーが発生してしまいました。
お手数ですが、何か対処法はございますでしょうか。

[hokahomu]

追記ありがとうございます！
無事にうまくjsonのデータを表示することができました！
本当に助かりました！