Rails: Twilio :: REST :: RestError - PullRequest
       0

Rails: Twilio :: REST :: RestError

0 голосов
/ 28 января 2020

Я добавил twilio gem в свое приложение для проверки телефонных номеров. И это настройки у меня:

Контроллер:

class UsersController < ApplicationController
  before_action :set_user, only: [:show, :edit, :update, :destroy]
  before_action :set_client, only: [:create, :verify]
  before_action :current_user, only: [:verify]

  # GET /users
  # GET /users.json
  def index
    @users = User.all
  end

  # GET /users/1
  # GET /users/1.json
  def show
  end

  # GET /users/new
  def new
    @user = User.new
  end

  # GET /users/1/edit
  def edit
  end

  # POST /users
  # POST /users.json
  def create
    channel = user_params['channel']
    @user = User.new(user_params.except('channel', 'displayed_phone_number'))

    respond_to do |format|
      if @user.save
        start_verification(@user.phone_number, channel)
        session[:user_id] = @user.id
        format.html { redirect_to verify_url, notice: 'User was successfully created.' }
        format.json { render :show, status: :created, location: @user }
      else
        format.html { render :new }
        format.json { render json: @user.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /users/1
  # PATCH/PUT /users/1.json
  def update
    respond_to do |format|
      if @user.update(user_params)
        format.html { redirect_to @user, notice: 'User was successfully updated.' }
        format.json { render :show, status: :ok, location: @user }
      else
        format.html { render :edit }
        format.json { render json: @user.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /users/1
  # DELETE /users/1.json
  def destroy
    @user.destroy
    respond_to do |format|
      format.html { redirect_to users_url, notice: 'User was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  def verify
    if request.post?
      is_verified = check_verification(@current_user.phone_number, params['verification_code'])
      if is_verified
        @current_user.verified = true
        @current_user.save
        respond_to do |format|
          format.html { redirect_to main_index_url, notice: 'User was successfully verified.' }
        end
      else
        respond_to do |format|
          format.html { redirect_to verify_url, notice: 'The code was invalid.' }
        end
      end
    else
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_user
      @user = User.find(params[:id])
    end

    def set_client
      @client = Twilio::REST::Client.new(ENV['TWILIO_ACCOUNT_SID'], ENV['TWILIO_AUTH_TOKEN'])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def user_params
      params.require(:user).permit(:username, :password, :password_confirmation, :phone_number)
    end

    def start_verification(to, channel='sms')
      channel = 'sms' unless ['sms', 'voice'].include? channel
      verification = @client.verify.services(ENV['VERIFICATION_SID'])
                                   .verifications
                                   .create(:to => to, :channel => channel)
      return verification.sid
    end

    def check_verification(phone, code)
      verification_check = @client.verify.services(ENV['VERIFICATION_SID'])
                                         .verification_checks
                                         .create(:to => phone, :code => code)
      return verification_check.status == 'approved'
    end
end

Форма:

<%= form_with(model: user, local: true) do |form| %>
  <% if user.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(user.errors.count, "error") %> prohibited this user from being saved:</h2>

      <ul>
      <% user.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <div class="field">
    <%= form.label :username %>
    <%= form.text_field :username %>
  </div>

  <div class="field">
    <%= form.label :password %>
    <%= form.password_field :password %>
  </div>

  <div class="field">
    <%= form.label :password_confirmation %>
    <%= form.password_field :password_confirmation %>
  </div>

  <div class="field">
    <%= form.label :displayed_phone_number %>
    <%= form.text_field :displayed_phone_number, type: 'tel' %>
  </div>

  <div class="field">
    <label for="channel">Verification Method</label>
    <label><input type="radio" name="channel" value="sms" checked>SMS</label>
    <label><input type="radio" name="channel" value="voice">Call</label>
  </div>

  <div class="actions">
    <%= form.submit 'Sign Up' %>
  </div>
<% end %>


<script>
  var input = document.querySelector("#user_displayed_phone_number");
  window.intlTelInput(input, {
    hiddenInput: "phone_number",
    preferredCountries: ["us", "gb", "co", "de"]
  });
</script>

Модель:

class User < ApplicationRecord
  has_secure_password

  validates :username, presence: true, uniqueness: true
  validates :phone_number, presence: true, uniqueness: true
end

Форма подтверждения:

<% title 'Verify' %>

<header><h1>Verify</h1></header>

<p id="notice"><%= notice %></p>

<%= form_tag(verify_url) do %>
  <div class="field">
    <%= label_tag :verification_code %>
    <%= text_field_tag :verification_code %>
  </div>

  <div class="actions">
    <%= submit_tag 'Verify' %>
  </div>
<% end %>

Маршруты:

Rails.application.routes.draw do
  get 'main/index'

  resources :users, only: [:new, :create, :show, :index, :destroy]
  resources :sessions, only: [:new, :create, :destroy]

  get 'verify', to: 'users#verify', as: 'verify'
  post 'verify', to: 'users#verify'
  get 'register', to: 'users#new', as: 'register'
  get 'login', to: 'sessions#new', as: 'login'
  get 'logout', to: 'sessions#destroy', as: 'logout'

  root 'main#index'
end

Когда я проверяю это, я получаю телефонный звонок или смс с кодом подтверждения. Я ввожу его в форму подтверждения и все работает нормально, номер телефона проверяется. Но когда я ввожу неправильный код или совсем не код ... вместо того, чтобы получить уведомление The code was invalid., я получаю эту ошибку:

Twilio::REST::RestError in UsersController#verify
[HTTP 400] 60200 : Unable to create record Invalid parameter: Code https://www.twilio.com/docs/errors/60200

, которая от контроллера пользователя и этот метод:

def check_verification(phone, code)
  verification_check = @client.verify.services(ENV['VERIFICATION_SID']).verification_checks.create(:to => phone, :code => code)
  return verification_check.status == 'approved'
end

И ошибка в этой строке:

verification_check = @client.verify.services(ENV['VERIFICATION_SID']).verification_checks.create(:to => phone, :code => code)

Есть идеи, почему это происходит?

1 Ответ

0 голосов
/ 29 января 2020

Twanio developer evangelist здесь.

В этом сообщении об ошибке указано, что параметр кода недействителен. Я предполагаю, что вы получите это в случае не посылки кода. Поэтому я бы, вероятно, встроил в ваш метод check_verification ранний возврат, если код пустой (в любом случае он не будет правильным).

    def check_verification(phone, code)
      return false if code.blank?
      verification_check = @client.verify.services(ENV['VERIFICATION_SID'])
                                         .verification_checks
                                         .create(:to => phone, :code => code)
      return verification_check.status == 'approved'
    end

Что касается отправки неправильного кода, вы получаете то же самое? возникла ошибка и то же сообщение?

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...