front_end_builds 0.2.0 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 1542985a803ac72002fd437fc975726ee7dac9cc
-  data.tar.gz: 2caa84a2a6c32e72ec08ff5273738778c809eaa1
+SHA256:
+  metadata.gz: c79e8434a0ff0de302875b4a6e5659570622e6977514a2d7b6263eead44c0331
+  data.tar.gz: 378adb746ed9d7b0582ceb3d4243673c40552614797c945b4e540e7201f6dc85
 SHA512:
-  metadata.gz: 1af52583c3a5ff0874f0033ca30bc1cfedce894a2d9eaeb9719bc766476c22aa4d47fab82f17c0d30844b9db872c7e28ec4087c49a4619af3748bbf226e4ab15
-  data.tar.gz: 9b3bc4d6543bfa161d35d1f130fcfb56fef52918a92f82b4d558ee5b4f866d84ee93b3a686acd21ef93cee77c46754a86129ba5b5f7ed167d8fd5bc3f833ce63
+  metadata.gz: '098a7f9411dc70118a4ec676fbd0f7a9c4ecdffe9324f04b380a409d044b7321faf073cd093451f14b0efe15d56ceda896e1d28442f59a5240d74ae2d77cafbc'
+  data.tar.gz: 6f96701912242573b2f4337eab6144341f60fe0a5ccd88d16ba1e8698c8a376e50293f2eed1b61cfec033b78a5ec692d32e3d7f107f830a6a00da2504a05f782

data/Rakefile

@@ -25,7 +25,7 @@ end
 namespace :admin do
   task :build do
     Dir.chdir('admin') do
-      sh 'ember build  --environment=production'
+      sh 'ember build production'
     end
 
     # Copy the dist to public

data/app/controllers/front_end_builds/admin_controller.rb

@@ -15,7 +15,7 @@ module FrontEndBuilds
       html = html.sub('BASEURL/', baseURL)
       html = html.sub("baseURL: ''", "baseURL: '#{baseURL}'")
 
-      render text: html
+      render plain: html
     end
 
   end

data/app/controllers/front_end_builds/application_controller.rb

@@ -1,11 +1,6 @@
 module FrontEndBuilds
   class ApplicationController < ActionController::Base
 
-    def use_params(param_method)
-      v = Rails::VERSION::MAJOR
-      send("#{param_method}_rails_#{v}")
-    end
-
     # Public: A quick helper to create a respond_to block for
     # returning json to the client. Used because `respond_with`
     # is no longer included in Rails.
@@ -20,6 +15,5 @@ module FrontEndBuilds
     def error!(errors, status = :unprocessable_entity)
       respond_with_json({ errors: errors }, status: status)
     end
-
   end
 end

data/app/controllers/front_end_builds/apps_controller.rb

@@ -2,10 +2,15 @@ require_dependency "front_end_builds/application_controller"
 
 module FrontEndBuilds
   class AppsController < ApplicationController
-    before_filter :set_app , :only => [:show, :destroy, :update]
+    before_action :set_app , :only => [:show, :destroy, :update]
 
     def index
-      apps = App.includes(:recent_builds)
+      # this should be the most recent 10 builds for each app
+
+      # DO NOT use `App.includes(:recent_builds)`
+      # b/c it mucks up the grouping logic and only gives the most
+      # recent 10 for ALL apps not 10 per app
+      apps = App.all
 
       respond_with_json({
         apps: apps.map(&:serialize),
@@ -23,7 +28,7 @@ module FrontEndBuilds
     end
 
     def create
-      @app = FrontEndBuilds::App.new( use_params(:app_create_params) )
+      @app = FrontEndBuilds::App.new( app_create_params )
 
       if @app.save
         respond_with_json(
@@ -39,7 +44,7 @@ module FrontEndBuilds
     end
 
     def update
-      if @app.update_attributes( use_params(:app_update_params) )
+      if @app.update_attributes( app_update_params )
 
         respond_with_json(
           { app: @app.serialize },
@@ -61,7 +66,7 @@ module FrontEndBuilds
         )
       else
         respond_with_json(
-          {errors: @app.errors},
+          { errors: @app.errors },
           status: :unprocessable_entity
         )
       end
@@ -73,31 +78,18 @@ module FrontEndBuilds
       @app = FrontEndBuilds::App.find(params[:id])
     end
 
-    def app_create_params_rails_3
-      params[:app].slice(:name)
-    end
-
-    def app_create_params_rails_4
+    def app_create_params
       params.require(:app).permit(
         :name
       )
     end
 
-    def app_update_params_rails_3
-      params[:app].slice(
-        :name,
-        :require_manual_activation,
-        :live_build_id
-      )
-    end
-
-    def app_update_params_rails_4
+    def app_update_params
       params.require(:app).permit(
         :name,
         :require_manual_activation,
         :live_build_id
       )
     end
-
   end
 end

data/app/controllers/front_end_builds/bests_controller.rb

@@ -11,17 +11,17 @@ module FrontEndBuilds
   class BestsController < ApplicationController
     include Rails.application.routes.url_helpers
 
-    before_filter :find_front_end, only: [:show]
+    before_action :find_front_end, only: [:show]
 
     def show
       if @front_end
         respond_to do |format|
-          format.html { render text: @front_end.with_head_tag(meta_tags) }
+          format.html { render plain: @front_end.with_head_tag(meta_tags) }
           format.json { render json: { version: @front_end.id } }
         end
       else
         # TODO install instructions, user needs to push build
-        render text: "not found", status: 404
+        render plain: "not found", status: 404
       end
     end
 
@@ -33,10 +33,10 @@ module FrontEndBuilds
         csrf_param: request_forgery_protection_token,
         csrf_token: form_authenticity_token,
         front_end_build_version: @front_end.id,
-        front_end_build_params: use_params(:build_search_params).to_query,
+        front_end_build_params: build_search_params.to_h.to_query,
         front_end_build_url: front_end_builds_best_path(
-            use_params(:build_search_params).merge(format: :json)
-          )
+          build_search_params.merge(format: :json)
+        )
       }
 
       tags
@@ -48,14 +48,10 @@ module FrontEndBuilds
     end
 
     def find_front_end
-      @front_end = FrontEndBuilds::Build.find_best(use_params(:build_search_params))
+      @front_end = FrontEndBuilds::Build.find_best(build_search_params)
     end
 
-    def build_search_params_rails_3
-      params.slice(:app_name, :id, :branch, :sha, :job)
-    end
-
-    def build_search_params_rails_4
+    def build_search_params
       params.permit(:app_name, :id, :branch, :sha, :job)
     end
   end

data/app/controllers/front_end_builds/builds_controller.rb

@@ -2,7 +2,7 @@ require_dependency "front_end_builds/application_controller"
 
 module FrontEndBuilds
   class BuildsController < ApplicationController
-    before_filter :set_app!, only: [:create]
+    before_action :set_app!, only: [:create]
 
     def index
       builds = FrontEndBuilds::Build.where(app_id: params[:app_id])
@@ -12,7 +12,7 @@ module FrontEndBuilds
     end
 
     def create
-      build = @app.builds.new(use_params(:build_create_params))
+      build = @app.builds.new(build_create_params)
 
       if build.verify && build.save
         build.setup!
@@ -22,7 +22,7 @@ module FrontEndBuilds
         build.errors[:base] << 'No access - invalid SSH key' if !build.verify
 
         render(
-          text: 'Could not create the build: ' + build.errors.full_messages.to_s,
+          plain: 'Could not create the build: ' + build.errors.full_messages.to_s,
           status: :unprocessable_entity
         )
       end
@@ -45,7 +45,7 @@ module FrontEndBuilds
 
       if @app.nil?
         render(
-          text: "No app named #{params[:app_name]}.",
+          plain: "No app named #{params[:app_name]}.",
           status: :unprocessable_entity
         )
 
@@ -64,11 +64,7 @@ module FrontEndBuilds
       ]
     end
 
-    def build_create_params_rails_3
-      params.slice(*_create_params)
-    end
-
-    def build_create_params_rails_4
+    def build_create_params
       params.permit(*_create_params)
     end
   end

data/app/controllers/front_end_builds/pubkeys_controller.rb

@@ -8,8 +8,7 @@ module FrontEndBuilds
     end
 
     def create
-      pubkey = FrontEndBuilds::Pubkey
-        .new( use_params(:pubkey_create_params) )
+      pubkey = FrontEndBuilds::Pubkey.new(pubkey_create_params)
 
       if pubkey.save
         respond_with_json(
@@ -36,11 +35,7 @@ module FrontEndBuilds
 
     private
 
-    def pubkey_create_params_rails_3
-      params[:pubkey].slice(:name, :pubkey)
-    end
-
-    def pubkey_create_params_rails_4
+    def pubkey_create_params
       params.require(:pubkey).permit(
         :name,
         :pubkey

data/app/models/front_end_builds/app.rb

@@ -6,7 +6,7 @@ module FrontEndBuilds
                       :live_build_id
     end
 
-    belongs_to :live_build, class_name: 'FrontEndBuilds::Build'
+    belongs_to :live_build, class_name: 'FrontEndBuilds::Build', optional: true
     has_many :builds, class_name: 'FrontEndBuilds::Build'
 
     if ActiveRecord::VERSION::MAJOR < 4

data/db/migrate/20141010162405_create_front_end_builds_builds.rb

@@ -2,9 +2,9 @@ class CreateFrontEndBuildsBuilds < ActiveRecord::Migration
   def change
     create_table :front_end_builds_builds do |t|
       t.references :app
-      t.string :sha
-      t.string :job
-      t.string :branch
+      t.string :sha, limit: 191
+      t.string :job, limit: 191
+      t.string :branch, limit: 191
       t.text :html
       t.boolean :fetched, default: false
       t.boolean :active, default: false

data/db/migrate/20141010165726_create_front_end_builds_apps.rb

@@ -1,8 +1,8 @@
 class CreateFrontEndBuildsApps < ActiveRecord::Migration
   def change
     create_table :front_end_builds_apps do |t|
-      t.string :name
-      t.string :api_key
+      t.string :name, limit: 191
+      t.string :api_key, limit: 191
 
       t.timestamps
     end

data/db/migrate/20150124215337_create_front_end_builds_pubkeys.rb

@@ -1,7 +1,7 @@
 class CreateFrontEndBuildsPubkeys < ActiveRecord::Migration
   def change
     create_table :front_end_builds_pubkeys do |t|
-      t.string :name, null: false
+      t.string :name, null: false, limit: 191
       t.text :pubkey, null: false
 
       t.timestamps null: false

data/lib/front_end_builds/engine.rb

@@ -6,7 +6,7 @@ module FrontEndBuilds
 
     config.generators do |g|
       g.test_framework :rspec, fixture: false
-      g.fixture_replacement :factory_girl, dir: 'spec/factories'
+      g.fixture_replacement :factory_bot, dir: 'spec/factories'
       g.assets false
       g.helper false
     end

data/lib/front_end_builds/utils/ssh_pubkey_convert.rb

@@ -3,6 +3,9 @@
 #
 # https://github.com/mytestbed/omf/blob/master/omf_common/lib/omf_common/auth/ssh_pub_key_convert.rb
 #
+# Support for DSA keys was removed from this code as the FEB app doesn't support DSA keys
+# See PubKey#verify
+#
 
 module FrontEndBuilds
   module Utils
@@ -66,27 +69,21 @@ module FrontEndBuilds
           (nstr, bytes) = unpack_string(bytes, n)
 
           key = OpenSSL::PKey::RSA.new
-          key.n = OpenSSL::BN.new(nstr, 2)
-          key.e = OpenSSL::BN.new(estr, 2)
-          key
-        elsif keytype == 'ssh-dss'
-          (n, bytes) = unpack_u32(bytes)
-          (pstr, bytes) = unpack_string(bytes, n)
-          (n, bytes) = unpack_u32(bytes)
-          (qstr, bytes) = unpack_string(bytes, n)
-          (n, bytes) = unpack_u32(bytes)
-          (gstr, bytes) = unpack_string(bytes, n)
-          (n, bytes) = unpack_u32(bytes)
-          (pkstr, bytes) = unpack_string(bytes, n)
 
-          key = OpenSSL::PKey::DSA.new
-          key.p = OpenSSL::BN.new(pstr, 2)
-          key.q = OpenSSL::BN.new(qstr, 2)
-          key.g = OpenSSL::BN.new(gstr, 2)
-          key.pub_key = OpenSSL::BN.new(pkstr, 2)
+          # support SSL 2
+          if Gem::Version.new(OpenSSL::VERSION) < Gem::Version.new('2.0.0')
+            key.n = OpenSSL::BN.new(nstr, 2)
+            key.e = OpenSSL::BN.new(estr, 2)
+          else
+            # params are n, e, d
+            key.set_key(OpenSSL::BN.new(nstr, 2), OpenSSL::BN.new(estr, 2), nil)
+          end
+
           key
         else
-          nil
+          # anything non-RSA is not supported
+          # this part edited by TED
+          raise "Unsupported key type: #{keytype}"
         end
       end
     end

data/lib/front_end_builds/version.rb

@@ -1,3 +1,3 @@
 module FrontEndBuilds
-  VERSION = "0.2.0"
+  VERSION = '1.0.3'
 end

metadata

@@ -1,18 +1,33 @@
 --- !ruby/object:Gem::Specification
 name: front_end_builds
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 1.0.3
 platform: ruby
 authors:
 - Ryan Toronto
 - Sam Selikoff
-autorequire: 
+- John Hirbour
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-11-13 00:00:00.000000000 Z
+date: 2020-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: sqlite3
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+- !ruby/object:Gem::Dependency
+  name: bundler-audit
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -26,21 +41,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec-rails
+  name: byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec-its
+  name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -54,7 +69,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: factory_girl_rails
+  name: factory_bot_rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,6 +110,76 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rb-readline
+  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'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  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'
+- !ruby/object:Gem::Dependency
+  name: rspec-its
+  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'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  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'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  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'
 - !ruby/object:Gem::Dependency
   name: shoulda-matchers
   requirement: !ruby/object:Gem::Requirement
@@ -109,6 +194,48 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 2.7.0
+- !ruby/object:Gem::Dependency
+  name: sprockets
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.7.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.7.2
+- !ruby/object:Gem::Dependency
+  name: sprockets-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.2.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.2.1
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  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'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -127,6 +254,7 @@ description: Rails engine to manage front end builds and deployments
 email:
 - rt@ted.com
 - sam@ted.com
+- gohn@ted.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -172,13 +300,12 @@ files:
 - public/front_end_builds/assets/fontawesome-webfont.woff2
 - public/front_end_builds/assets/vendor-23f9bdd567ccab65909edb0f4ac32394.css
 - public/front_end_builds/assets/vendor-7c6d3f715703c96d6dce5d24fd762eb6.js
-- public/front_end_builds/crossdomain.xml
 - public/front_end_builds/robots.txt
 homepage: http://github.com/tedconf/front_end_builds
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -193,9 +320,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.2.2
-signing_key: 
+rubyforge_project:
+rubygems_version: 2.7.8
+signing_key:
 specification_version: 4
 summary: Summary of FrontEndBuilds.
 test_files: []

data/public/front_end_builds/crossdomain.xml

@@ -1,15 +0,0 @@
-<?xml version="1.0"?>
-<!DOCTYPE cross-domain-policy SYSTEM "http://www.adobe.com/xml/dtds/cross-domain-policy.dtd">
-<cross-domain-policy>
-    <!-- Read this: www.adobe.com/devnet/articles/crossdomain_policy_file_spec.html -->
-
-    <!-- Most restrictive policy: -->
-    <site-control permitted-cross-domain-policies="none"/>
-
-    <!-- Least restrictive policy: -->
-    <!--
-    <site-control permitted-cross-domain-policies="all"/>
-    <allow-access-from domain="*" to-ports="*" secure="false"/>
-    <allow-http-request-headers-from domain="*" headers="*" secure="false"/>
-    -->
-</cross-domain-policy>