mux-rails 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9daf9dfa3ace722030cdfa1020cb3682771167ef24cbb91394fa59b6007a3243
-  data.tar.gz: d2f33afce8d1b13649bbee69e38912dc59cbbd82ccae9f7d926178e83b62180f
+  metadata.gz: 7322b49a720b1e55df32f6916ae9cdaa626a6fbeb1e93755728d19ce6a8e51df
+  data.tar.gz: 96c0caac8cb6f8f248ef12c8cb13a6e5166c2c9626297677d22e5bdb1ca37fd4
 SHA512:
-  metadata.gz: cd03523ecd957b317f4f7cc5cd2ce8cf2e07d3068124347b283f2ed1432cd3aa2b026cfec68621d5c1569431aaf52c6c798c50e0361e6119ecccead2067c3da9
-  data.tar.gz: 6a725b3b197c054f75abc3e7114b243d6058b6d1b395925099d00e7e372b537c3eb9145ff9b7432503812266b28d49707258a989aac87147ba6d5f05c0096ebd
+  metadata.gz: 4e50040a885429872de68f92d0d3045e7bd3fd03dfd9e9a458261df833abca8960557bb22966192341f5ebee8d68bcf67ede5c4bd8a6075eb67d37b7f11491c9
+  data.tar.gz: 2df97d06b227ab78e0e0cc98318a0101abf103f6dd4a030bbb5248d8fd31d534a2246bf22a77f6a6a573aeac5526772e570c0c677a0547ca62b7e84ed2a7d302

data/README.md

@@ -19,6 +19,8 @@ gem 'mux-rails'
 mount Mux::Engine, at: "/mux" # provide a custom path
 ```
 
+Make sure to configure the webhooks at mux.com to point to `https://yourwebsite.com/mux/events`.
+
 ## Usage
 
 ### Setup Mux tokens
@@ -71,25 +73,23 @@ Using a subscriber with a block we can listen to incoming events from Mux and do
 
 Mux::Notifications.subscribe "video.asset.created" do |event|
   # handle asset created
-  # event.object.id == mux_asset_id
-  # event.data.playback_ids.first.id
+  # event.id == mux_asset_id
 end
 
 Mux::Notifications.subscribe "video.asset.ready" do |event|
   # handle asset ready
-  # event.object.id == mux_asset_id
-  # event.data.playback_ids.first.id
+  # event.id == mux_asset_id
 end
 
 Mux::Notifications.subscribe "video.asset.deleted" do |event|
   # handle asset deleted
-  # event.object.id == mux_asset_id
-  # event.data.playback_ids.first.id
+  # event.id == mux_asset_id
 end
 ```
 
 The block is passed an event which is simply the incoming JSON (take a peek
-in [fixtures](test/fixtures) for examples) wrapped in an OpenStruct for ease of access.
+in [fixtures](test/fixtures) for examples) wrapped in [`Mux::Event`](lib/mux/event.rb) class based on
+`Dry::Struct`.
 
 ## License
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/Rakefile

@@ -1,31 +1,33 @@
+# frozen_string_literal: true
+
 begin
-  require 'bundler/setup'
+  require "bundler/setup"
 rescue LoadError
-  puts 'You must `gem install bundler` and `bundle install` to run rake tasks'
+  puts "You must `gem install bundler` and `bundle install` to run rake tasks"
 end
 
-require 'rdoc/task'
+require "rdoc/task"
 
 RDoc::Task.new(:rdoc) do |rdoc|
-  rdoc.rdoc_dir = 'rdoc'
-  rdoc.title    = 'Mux::Rails'
-  rdoc.options << '--line-numbers'
-  rdoc.rdoc_files.include('README.md')
-  rdoc.rdoc_files.include('lib/**/*.rb')
+  rdoc.rdoc_dir = "rdoc"
+  rdoc.title    = "Mux::Rails"
+  rdoc.options << "--line-numbers"
+  rdoc.rdoc_files.include("README.md")
+  rdoc.rdoc_files.include("lib/**/*.rb")
 end
 
 APP_RAKEFILE = File.expand_path("test/dummy/Rakefile", __dir__)
-load 'rails/tasks/engine.rake'
+load "rails/tasks/engine.rake"
 
-load 'rails/tasks/statistics.rake'
+load "rails/tasks/statistics.rake"
 
-require 'bundler/gem_tasks'
+require "bundler/gem_tasks"
 
-require 'rake/testtask'
+require "rake/testtask"
 
 Rake::TestTask.new(:test) do |t|
-  t.libs << 'test'
-  t.pattern = 'test/**/*_test.rb'
+  t.libs << "test"
+  t.pattern = "test/**/*_test.rb"
   t.verbose = false
 end
 

data/app/controllers/mux/application_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Mux
   class ApplicationController < ActionController::Base
     protect_from_forgery with: :exception

data/app/controllers/mux/events_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Mux
   class EventsController < ActionController::Base
     # QUESTION: Is this the right way to handle it?
@@ -11,10 +13,10 @@ module Mux
     end
 
     private
-
-    def event
-      event = JSON.parse(request.body.read, object_class: OpenStruct)
-      Mux::Event.new(event)
-    end
+      def event
+        Mux::Event.new(
+          JSON.parse(request.body.read)
+        )
+      end
   end
 end

data/lib/mux/client.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "mux_ruby"
 
 module Mux
@@ -9,7 +11,7 @@ module Mux
         request.input = url
         request.playback_policy = MuxRuby::PlaybackPolicy.build_from_hash(playback_policy)
         response = assets_api.create_asset(request)
-        return response.data.id
+        response.data.id
       end
 
       def destroy_asset(asset_id, options = {})
@@ -22,10 +24,9 @@ module Mux
       end
 
       private
-
-      def assets_api
-        @assets_api ||= MuxRuby::AssetsApi.new
-      end
+        def assets_api
+          @assets_api ||= MuxRuby::AssetsApi.new
+        end
     end
   end
 end

data/lib/mux/engine.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Mux
   class Engine < ::Rails::Engine
     isolate_namespace Mux

data/lib/mux/event.rb

@@ -1,12 +1,59 @@
+# frozen_string_literal: true
+
 module Mux
-  class Event
-    # QUESTION: Is there security concerns here? This implementation seems very naive ;-)
-    def initialize(payload)
-      @event = OpenStruct.new(payload)
+  class Event < Dry::Struct
+    module Types
+      include Dry.Types()
+    end
+
+    transform_keys(&:to_sym)
+
+    attribute :id, Types::String
+    attribute :type, Types::String
+    attribute :created_at, Types::JSON::Time
+
+    attribute :object do
+      attribute :id, Types::Coercible::String
+      attribute :type, Types::Coercible::String
+    end
+
+    attribute :environment do
+      attribute :id, Types::Coercible::String
+      attribute :name, Types::Coercible::String
     end
 
-    def method_missing(method, *args, &block)
-      @event.send(method, *args, &block)
+    attribute :data do
+      attribute :id, Types::Coercible::String
+      attribute :status, Types::Coercible::String
+      attribute :created_at, Types::Coercible::Integer
+
+      attribute? :aspect_ratio, Types::Coercible::String
+      attribute? :duration, Types::Coercible::Float
+      attribute? :master_access, Types::Coercible::String
+      attribute? :max_stored_frame_rate, Types::Coercible::Float
+      attribute? :max_stored_resolution, Types::Coercible::String
+      attribute? :mp4_support, Types::Coercible::String
+
+      attribute? :playback_ids, Types::Array do
+        attribute :id, Types::Coercible::String
+        attribute :policy, Types::Coercible::String
+      end
+
+      attribute? :tracks, Types::Array do
+        attribute :id, Types::Coercible::String
+        attribute :type, Types::Coercible::String
+
+        attribute :duration, Types::Coercible::Float
+
+        # when type = "video"
+        attribute? :max_width, Types::Coercible::Integer
+        attribute? :max_height, Types::Coercible::Integer
+        attribute? :max_frame_rate, Types::Coercible::Float
+
+        # when type = "audio"
+        attribute? :max_channels, Types::Coercible::Integer
+        attribute? :max_channel_layout, Types::Coercible::String
+      end
     end
   end
 end

data/lib/mux/notifications.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support/notifications"
 
 module Mux

data/lib/mux/rails.rb

@@ -1,5 +1,15 @@
+# frozen_string_literal: true
+
+require "dry-struct"
+require "dry-types"
+require "mux_ruby"
+
 require "mux/engine"
+require "mux/rails/version"
+
+module Mux
+end
 
-require "mux/notifications"
-require "mux/event"
 require "mux/client"
+require "mux/event"
+require "mux/notifications"

data/lib/mux/rails/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Mux
   module Rails
-    VERSION = "0.2.0"
+    VERSION = "0.2.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mux-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Asger Behncke Jacobsen
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-19 00:00:00.000000000 Z
+date: 2021-04-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -38,6 +38,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.8.0
+- !ruby/object:Gem::Dependency
+  name: dry-struct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: dry-types
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.4'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.4'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +108,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails_config
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A Rails Engine for integrating with Mux.
 email:
 - a@asgerbehnckejacobsen.dk