Allowing users to authenticate with SoundCloud

pyro2927 · pyro2927 · commit c575e18f2c02 · 2014-10-25T12:58:20.000-05:00
soundcloud_id, username, and name added to User model. Using Omniauth
with Devise, ignoring SSL certs while in dev mode
diff --git a/.gitignore b/.gitignore
@@ -29,4 +29,7 @@ config/secrets.yml
 # if using bower-rails ignore default bower_components path bower.json files
 /vendor/assets/bower_components
 *.bowerrc
-bower.json
+bower.json
+
+# dotenv
+.env
diff --git a/Gemfile b/Gemfile
@@ -15,15 +15,20 @@ gem 'jquery-rails', '~> 3.1.0'
 gem 'responders', '~> 1.0.0'
 gem 'slim-rails', '~> 2.1.4'
 
+#authentication
+gem 'omniauth-soundcloud', '~> 1.0.1'
+
 group :test, :development do
   gem 'factory_girl_rails'
   gem 'pry-rescue'
   gem 'simplecov', :require => false
   gem 'minitest-reporters'
   gem 'faker', '~> 1.4.2', require: false
+  gem 'minitest-spec-context'
 end
 
 group :development do
+  gem 'dotenv-rails', '~> 1.0.2'
   gem 'html2slim'
   gem 'thin'
   gem 'quiet_assets'
@@ -34,7 +39,7 @@ group :development do
   gem 'binding_of_caller'
   gem 'spring'
   gem 'byebug'
-  # gem 'foreman'
+  gem 'foreman'
   # gem 'guard', :require => false
   # gem 'guard-livereload', :require => false
   # gem 'rack-livereload'
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -63,6 +63,9 @@ GEM
       thread_safe (~> 0.1)
       warden (~> 1.2.3)
     docile (1.1.5)
+    dotenv (1.0.2)
+    dotenv-rails (1.0.2)
+      dotenv (= 1.0.2)
     erubis (2.7.0)
     eventmachine (1.0.3)
     excon (0.40.0)
@@ -74,6 +77,8 @@ GEM
       railties (>= 3.0.0)
     faker (1.4.3)
       i18n (~> 0.5)
+    faraday (0.9.0)
+      multipart-post (>= 1.2, < 3)
     fog (1.24.0)
       fog-brightbox
       fog-core (~> 1.23)
@@ -109,7 +114,11 @@ GEM
     fog-xml (0.1.0)
       fog-core
       nokogiri (~> 1.5, >= 1.5.11)
+    foreman (0.63.0)
+      dotenv (>= 0.7)
+      thor (>= 0.13.6)
     formatador (0.2.5)
+    hashie (3.3.1)
     hike (1.2.3)
     hpricot (0.8.6)
     html2slim (0.1.0)
@@ -122,6 +131,7 @@ GEM
       railties (>= 3.0, < 5.0)
       thor (>= 0.14, < 2.0)
     json (1.8.1)
+    jwt (1.0.0)
     kgio (2.9.2)
     mail (2.6.1)
       mime-types (>= 1.16, < 3)
@@ -134,13 +144,32 @@ GEM
       builder
       minitest (>= 5.0)
       ruby-progressbar
+    minitest-spec-context (0.0.3)
     multi_json (1.10.1)
+    multi_xml (0.5.5)
+    multipart-post (2.0.0)
     net-scp (1.2.1)
       net-ssh (>= 2.6.5)
     net-ssh (2.9.1)
     newrelic_rpm (3.9.6.257)
     nokogiri (1.6.3.1)
       mini_portile (= 0.6.0)
+    oauth2 (0.9.4)
+      faraday (>= 0.8, < 0.10)
+      jwt (~> 1.0)
+      multi_json (~> 1.3)
+      multi_xml (~> 0.5)
+      rack (~> 1.2)
+    omniauth (1.2.2)
+      hashie (>= 1.2, < 4)
+      rack (~> 1.0)
+    omniauth-oauth2 (1.1.2)
+      faraday (>= 0.8, < 0.10)
+      multi_json (~> 1.3)
+      oauth2 (~> 0.9.3)
+      omniauth (~> 1.2)
+    omniauth-soundcloud (1.0.1)
+      omniauth-oauth2 (~> 1.1.0)
     orm_adapter (0.5.0)
     pg (0.17.1)
     pry (0.10.1)
@@ -241,12 +270,16 @@ DEPENDENCIES
   byebug
   cancan (~> 1.6.10)
   devise (~> 3.4.0)
+  dotenv-rails (~> 1.0.2)
   factory_girl_rails
   faker (~> 1.4.2)
+  foreman
   html2slim
   jquery-rails (~> 3.1.0)
   minitest-reporters
+  minitest-spec-context
   newrelic_rpm
+  omniauth-soundcloud (~> 1.0.1)
   pg (~> 0.17.1)
   pry
   pry-rails
diff --git a/app/controllers/omniauth_callbacks_controller.rb b/app/controllers/omniauth_callbacks_controller.rb
@@ -0,0 +1,12 @@
+class OmniauthCallbacksController < Devise::OmniauthCallbacksController
+  def soundcloud
+    @user = User.from_omniauth(request.env["omniauth.auth"])
+    if @user.persisted?
+      flash[:notice] = "Signed in!"
+      sign_in_and_redirect @user, event: :authentication
+    else
+      flash[:alert] = "Error logging in"
+      redirect_to '/'
+    end
+  end
+end
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -1,6 +1,17 @@
 class User < ActiveRecord::Base
   # Include default devise modules. Others available are:
-  # :confirmable, :lockable, :timeoutable and :omniauthable
+  # :confirmable, :lockable, :timeoutable
   devise :database_authenticatable, :registerable,
-         :recoverable, :rememberable, :trackable, :validatable
+         :recoverable, :rememberable, :trackable, :validatable, :omniauthable, :omniauth_providers => [:soundcloud]
+
+  def self.from_omniauth(auth)
+    user = User.where(soundcloud_id: auth.uid.to_s).first_or_create
+    user.username = auth.extra.raw_info.username
+    user.name = auth.info.name
+    user.password = auth.credentials.token
+    user.email = user.soundcloud_id + "@soundcloud.com"
+    user.save
+    user
+  end
+
 end
diff --git a/config/environments/development.rb b/config/environments/development.rb
@@ -27,4 +27,5 @@
   # This option may cause significant delays in view rendering with a large
   # number of complex assets.
   config.assets.debug = true
+  OpenSSL::SSL::VERIFY_PEER = OpenSSL::SSL::VERIFY_NONE
 end
diff --git a/config/initializers/omniauth.rb b/config/initializers/omniauth.rb
@@ -0,0 +1,3 @@
+Rails.application.config.middleware.use OmniAuth::Builder do
+  provider :soundcloud, ENV['SOUNDCLOUD_CLIENT_ID'], ENV['SOUNDCLOUD_SECRET'], {provider_ignores_state: true}
+end
diff --git a/config/routes.rb b/config/routes.rb
@@ -1,5 +1,5 @@
 Apollo::Application.routes.draw do
-  devise_for :users
+  devise_for :users, path_names: {sign_in: "login", sign_out: "logout"}, controllers: {omniauth_callbacks: "omniauth_callbacks"}
   # The priority is based upon order of creation: first created -> highest priority.
   # See how all your routes lay out with "rake routes".
 
diff --git a/db/migrate/20141025165208_add_info_to_users.rb b/db/migrate/20141025165208_add_info_to_users.rb
@@ -0,0 +1,7 @@
+class AddInfoToUsers < ActiveRecord::Migration
+  def change
+    add_column :users, :soundcloud_id, :string
+    add_column :users, :username, :string
+    add_column :users, :name, :string
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20141025161136) do
+ActiveRecord::Schema.define(version: 20141025165208) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -29,6 +29,9 @@
     t.inet     "last_sign_in_ip"
     t.datetime "created_at"
     t.datetime "updated_at"
+    t.string   "soundcloud_id"
+    t.string   "username"
+    t.string   "name"
   end
 
   add_index "users", ["email"], name: "index_users_on_email", unique: true, using: :btree
diff --git a/test/models/user_test.rb b/test/models/user_test.rb
@@ -4,4 +4,23 @@ class UserTest < ActiveSupport::TestCase
   # test "the truth" do
   #   assert true
   # end
+  describe "user creation" do
+    context "from omniauth" do
+      before :each do
+        OmniAuth.config.test_mode = true
+        OmniAuth.config.mock_auth[:soundcloud] = OmniAuth::AuthHash.new({
+          :provider => 'soundcloud',
+          :uid => '123545',
+          :credentials => {:token => 'asdfasdf'},
+          :info => {:nickname=>"jose",:image=>"http://myimageurl.com",:name=>"billy jean"},
+          :extra => {:raw_info=>{:country=>"Kazakstan",:city=>"Kazak",:uri=>"http://fakesoundcloud.com/userfaker"}}
+        })
+        auth = OmniAuth.config.mock_auth[:soundcloud]
+        @user = User.from_omniauth(auth)
+      end
+      it "creates a new user from omniauth hash" do
+        assert @user.valid?
+      end
+    end
+  end
 end
diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -1,6 +1,7 @@
 ENV["RAILS_ENV"] ||= "test"
 require File.expand_path('../../config/environment', __FILE__)
 require 'rails/test_help'
+require 'minitest-spec-context'
 
 class ActiveSupport::TestCase
   ActiveRecord::Migration.check_pending!

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+Rails.application.config.middleware.use OmniAuth::Builder do`
	`2`	`+ provider :soundcloud, ENV['SOUNDCLOUD_CLIENT_ID'], ENV['SOUNDCLOUD_SECRET'], {provider_ignores_state: true}`
	`3`	`+end`