Moving v1.3.2 to the stable branch

* Update README.md

* Delete FUNDING.yml

* Changed discord as the primary way to get in touch with the comminuty

* Changed the discord link so that it uses a permanent invite link

* Changed discord as the primary way to get in touch with the comminuty (#1279)

* Changed discord as the primary way to get in touch with the comminuty

* Changed the discord link so that it uses a permanent invite link

* Fixed the version number being out dated from the actual latest release number.

* Updated the version number in Amber to reflect the actual release number (#1280)

* Changed discord as the primary way to get in touch with the comminuty

* Changed the discord link so that it uses a permanent invite link

* Fixed the version number being out dated from the actual latest release number.

* Fix the correct documentation url (#1278)

Co-authored-by: Philippe Nénert @papilip <me@papilip.me>

* Feature/version upgrade (#1282)

* Changed discord as the primary way to get in touch with the comminuty

* Changed the discord link so that it uses a permanent invite link

* Fixed the version number being out dated from the actual latest release number.

* Updated Amber version and default Granite version

* Feature/update readme contribution steps (#1283)

* Changed discord as the primary way to get in touch with the comminuty

* Changed the discord link so that it uses a permanent invite link

* Fixed the version number being out dated from the actual latest release number.

* Update Linux installation steps to reflect the actual steps needed to install from source

* Make default package support older and newer versions of node-sass

* Bump amber version and update Ameba version

* Added github actions for CI instead of TravisCI

* Amber version bump for these quality of life updates

* Fix naming warning raised by crystal compiler about positional names being mismatched

* removed the travisCI yml

* Fixed amber_spec cookie naming error that relied on old implicit encoding/decoding of cooke name & value

* Fixed areas that ameba caught for format improvements

* Changed ubuntu to an older version instead of latest

* Changed ci workflow to remove an extra sqlite3 dev library that may be causing issues

* Changed from uninstall to install to fix sqlite3 issue

* Changed installation of sqlite3 based on further reading

* Set apt install to automatically agree to prompts

* changed packed installed from full sqlite3 to sqlite3-dev

* Adds redis for the test suite

* Added redis dependencies to the apt install step

* Changed steps to stop using the supercharge redis installation and added a step to check that redis is running

* Trying redis as a service

* Tweaked the redis service settings to map the port for the host container to access redis using localhost:6379

* Added a step to check access to the redis service before the tests run

* redis-cli stopped existing so trying to install plain redis

* removed the port mapping to test if redis becomes accessible from the runner

* specified the host when testing the redis connection

* Added ports mapping back to the redis service container to see if this resolves the redis-cli not reaching the redis service

* Changed host from localhost to redis to see if that maps correctly

* Added the REDIS_URL with the hostname of the redis service and removed the redis connection verification step now that it's working

* Changed the empty? method to be more clear about 'any origin' being the actual question, and fixed the to_h method doing too much to return a hash

* Remove crystalshards.xyz link on template view

* Include the discord link into the generated home page templates

* Tweaked how the PG test data is configured

* fix to cli scaffold controller spec deprecated garnetspec parent class

* Updated the remaining two templates that used the old GarnetSpec parent class to use the new/correct parent class

* Updated docker file to use latest crystal image and updated stragling Amber version

* Changed node version check to look for v14 only

* Bump amber version and updated cli template to use latest release version instead of master branch

* added "/myapp" to be excluded from git tracking

* Added cli and cli.dwarf to the git ignore so less clean-up is required when working on the cli tool

* Changed PG to check for the current user and omit the password for a speedier way to get started

---------

Co-authored-by: Isaac Sloan <isaac@isaacsloan.com>
Co-authored-by: robacarp <robacarp@users.noreply.github.com>
Co-authored-by: Philippe Nénert <me+github@papilip.me>
Co-authored-by: Philippe Nénert @papilip <me@papilip.me>
Co-authored-by: Spenser Coke <work@spensercoke.com>
Co-authored-by: Rafael Revi <rafael.e.revi@gmail.com>

.ameba.yml

@@ -0,0 +1,3 @@
+Excluded:
+  - tmp/
+  - myapp/

.github/workflows/ci.yml

@@ -0,0 +1,46 @@
+on:
+  push:
+jobs:
+  amber-spec:
+
+    runs-on: ubuntu-latest
+
+    container:
+      image: crystallang/crystal
+
+    services:
+      redis:
+        image: redis
+        options: >-
+          --health-cmd "redis-cli ping"
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+        ports:
+          - 6379:6379
+      postgres:
+        image: postgres
+        env:
+          POSTGRES_PASSWORD: postgres
+        options: >-
+          --health-cmd pg_isready
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+        ports:
+          - 5432:5432
+
+    steps:
+      - name: Update & install sqlite3
+        run: apt update && apt install -y libsqlite3-dev redis
+
+      - name: Download source
+        uses: actions/checkout@v3
+
+      - name: Install shards
+        run: shards install
+
+      - name: Run tests
+        run: bin/amber_spec
+        env:
+          REDIS_URL: "redis://redis:6379"

.gitignore

@@ -10,4 +10,7 @@ lib/
 /tmp
 shard.lock
 package-lock.json
-node_modules
+node_modules
+/myapp
+cli
+cli.dwarf

Dockerfile

@@ -1,4 +1,4 @@
-FROM crystallang/crystal:1.0.0
+FROM crystallang/crystal:latest
 
 # Install Dependencies
 ARG DEBIAN_FRONTEND=noninteractive

shard.yml

@@ -1,6 +1,6 @@
 name: amber
 
-version: 1.2.1
+version: 1.3.2
 
 authors:
   - Amber Team and Contributors <amberframework.org>
@@ -52,7 +52,7 @@ dependencies:
 
   redis:
     github: stefanwille/crystal-redis
-    version: ~> 2.7.0
+    version: ~> 2.8.0
 
   shell-table:
     github: luckyframework/shell-table.cr
@@ -85,4 +85,4 @@ dependencies:
 development_dependencies:
   ameba:
     github: crystal-ameba/ameba
-    version: ~> 0.13.4
+    version: ~> 1.0.0

spec/amber/cli/commands/pipelines/pipelines_spec.cr

@@ -59,7 +59,7 @@ module Amber::CLI
           end
 
           pipe_plugs.each do |pipe_name, plugs|
-            output_plugs = output_lines.select { |line| line.includes?(pipe_name) }
+            output_plugs = output_lines.select(&.includes?(pipe_name))
             plugs.each_with_index do |plug, index|
               output_plug = output_plugs[index]
               (output_plug != nil).should be_true

spec/amber/cli/commands/routes/routes_spec.cr

@@ -24,15 +24,15 @@ module Amber::CLI
 
         expected = "Amber::Controller::Static"
         output.should contain expected
-        line = output_lines.find("") { |this_line| this_line.includes? expected }
+        line = output_lines.find("", &.includes?(expected))
         expectations = %w(get Amber::Controller::Static index static /*)
         expectations.each do |expectation|
           line.should contain expectation
         end
 
         expected = "HomeController"
         output.should contain expected
-        line = output_lines.find("") { |this_line| this_line.includes? expected }
+        line = output_lines.find("", &.includes?(expected))
         expectations = %w(get HomeController index web /)
         expectations.each do |expectation|
           line.should contain expectation
@@ -77,7 +77,7 @@ module Amber::CLI
 
         expected = "websocket"
         output.should contain expected
-        line = output_lines.find("") { |this_line| this_line.includes? expected }
+        line = output_lines.find("", &.includes?(expected))
         expectations = %w(websocket ElectricSocket web /electric)
         expectations.each do |expectation|
           line.should contain expectation

spec/amber/router/session/redis_store_spec.cr

@@ -152,7 +152,7 @@ module Amber::Router::Session
         cookie_store.delete("ses")
 
         cookie_store.to_h.keys.should eq %w(a b c)
-        cookie_store.to_h["c"].should eq "x"
+        cookie_store.to_h.dig("c").should eq "x"
       end
     end
 

spec/support/helpers/cli_helper.cr

@@ -40,7 +40,7 @@ module CLIHelper
   def expected_db_url(db_key, env)
     case db_key
     when "pg"
-      "postgres://postgres:password@localhost:5432/#{TEST_APP_NAME}_#{env}"
+      "postgres://#{`whoami`.strip("\n")}:@localhost:5432/#{TEST_APP_NAME}_#{env}"
     when "mysql"
       "#{db_key}://root@localhost:3306/#{TEST_APP_NAME}_#{env}"
     else

src/amber/cli/commands/exec.cr

@@ -32,7 +32,7 @@ module Amber::CLI
         _filename = if File.exists?(args.code)
                       args.code
                     elsif options.back.to_i(strict: false) > 0
-                      Dir.glob("./tmp/*_console.cr").sort.reverse[options.back.to_i(strict: false) - 1]?
+                      Dir.glob("./tmp/*_console.cr").sort.reverse![options.back.to_i(strict: false) - 1]?
                     end
 
         system("cp #{_filename} #{@filename}") if _filename

src/amber/cli/commands/pipelines.cr

@@ -88,7 +88,7 @@ module Amber::CLI
         match = line.match(PIPELINE_REGEX)
 
         if match && (pipes = match[1])
-          pipes = pipes.split(/,\s*/).map { |s| s.gsub(/[:\"]/, "") }
+          pipes = pipes.split(/,\s*/).map(&.gsub(/[:\"]/, ""))
           result << {pipes: pipes, plugs: [] of String}
         else
           raise BadRoutesException.new(FAILED_TO_PARSE_ERROR)
@@ -113,7 +113,7 @@ module Amber::CLI
         table.border_color = :dark_gray unless options.no_color?
 
         if options.no_plugs?
-          result.map { |pipes_and_plugs| pipes_and_plugs[:pipes] }.flatten.uniq.each do |pipe|
+          result.flat_map { |pipes_and_plugs| pipes_and_plugs[:pipes] }.uniq!.each do |pipe|
             row = table.add_row
             row.add_column(pipe)
           end

src/amber/cli/commands/plugin.cr

@@ -25,7 +25,7 @@ module Amber::CLI
 
         if Amber::Plugins::Plugin.can_generate?(args.name)
           template = Amber::Plugins::Plugin.new(args.name, "./src/plugins", options.args)
-          template.generate (options.uninstall? ? "uninstall" : "install")
+          template.generate(options.uninstall? ? "uninstall" : "install")
         end
       end
 

src/amber/cli/commands/routes.cr

@@ -126,7 +126,7 @@ module Amber::CLI
 
       private def print_routes_table_json
         puts routes.map { |route|
-          route.transform_keys { |key| key.to_s.downcase.gsub(' ', '_') }
+          route.transform_keys(&.to_s.downcase.gsub(' ', '_'))
         }.to_json
       end
 

src/amber/cli/helpers/process_runner.cr

@@ -79,7 +79,7 @@ module Sentry
     private def check_processes
       @processes.each do |task, procs|
         # clean up process list and restart if terminated
-        if procs.any?
+        if !procs.empty?
           procs.reject!(&.terminated?)
 
           if procs.empty?

src/amber/cli/recipes/scaffold/controller.cr

@@ -24,7 +24,7 @@ module Amber::Recipes::Scaffold
       @fields += %w(created_at:time updated_at:time).map do |f|
         Amber::CLI::Field.new(f, hidden: true, database: @database)
       end
-      @visible_fields = @fields.reject { |f| f.hidden }
+      @visible_fields = @fields.reject(&.hidden)
       field_hash
 
       @template = RecipeFetcher.new("scaffold", @recipe).fetch

src/amber/cli/recipes/scaffold/view.cr

@@ -21,7 +21,7 @@ module Amber::Recipes::Scaffold
         Amber::CLI::Field.new(f, hidden: true, database: @database)
       end
 
-      @visible_fields = @fields.reject { |f| f.hidden }
+      @visible_fields = @fields.reject(&.hidden)
 
       @template = RecipeFetcher.new("scaffold", @recipe).fetch
       unless @template.nil?

src/amber/cli/templates/api/controller/spec/controllers/{{name}}_controller_spec.cr.ecr

@@ -25,7 +25,7 @@ def <%= create_model_method %>
   model
 end
 
-class <%= class_name %>ControllerTest < GarnetSpec::Controller::Test
+class <%= class_name %>ControllerTest < GarnetSpec::SystemTest
   getter handler : Amber::Pipe::Pipeline
 
   def initialize

src/amber/cli/templates/app/config/environments/development.yml.ecr

@@ -19,7 +19,7 @@ redis_url: "redis://localhost:6379"
 when "mysql" -%>
 database_url: mysql://root@localhost:3306/<%= database_name %>_development
 <% when "pg" -%>
-database_url: postgres://postgres:password@localhost:5432/<%= database_name %>_development
+database_url: postgres://<%= `whoami`.strip("\n") -%>:@localhost:5432/<%= database_name %>_development
 <% when "sqlite" -%>
 database_url: sqlite3:./db/<%= database_name %>_development.db
 <% else

src/amber/cli/templates/app/config/environments/production.yml.ecr

@@ -19,7 +19,7 @@ redis_url: "redis://localhost:6379"
 when "mysql" -%>
 database_url: mysql://root@localhost:3306/<%= database_name %>
 <% when "pg" -%>
-database_url: postgres://postgres:password@localhost:5432/<%= database_name %>
+database_url: postgres://<%= `whoami`.strip("\n") %>:@localhost:5432/<%= database_name %>
 <% when "sqlite" -%>
 database_url: sqlite3:./db/<%= database_name %>.db
 <% else

src/amber/cli/templates/app/config/environments/test.yml.ecr

@@ -19,7 +19,7 @@ redis_url: "redis://localhost:6379"
 when "mysql" -%>
 database_url: mysql://root@localhost:3306/<%= database_name %>_test
 <% when "pg" -%>
-database_url: postgres://postgres:password@localhost:5432/<%= database_name %>_test
+database_url: <%= "postgres://postgres:#{ENV["POSTGRES_PASSWORD"]? || "password"}@localhost:5432/#{database_name}_test" %>
 <% when "sqlite" -%>
 database_url: sqlite3:./db/<%= database_name %>_test.db
 <% else

src/amber/cli/templates/app/package.json.ecr

@@ -20,8 +20,13 @@
     "file-loader": "^6.2.0",
     "html-webpack-plugin": "^5.3.1",
     "mini-css-extract-plugin": "^1.6.0",
+  <% if `node -v`.matches?(/v14/) -%>
     "node-sass": "^5.0.0",
     "sass-loader": "^11.0.1",
+  <% else -%>
+    "node-sass": "^7.0.0",
+    "sass-loader": "^13.0.2",
+  <% end -%>
     "webpack": "^5.36.2",
     "webpack-cli": "^4.6.0",
     "webpack-merge": "^5.7.3"

src/amber/cli/templates/app/shard.yml.ecr

@@ -18,8 +18,7 @@ targets:
 dependencies:
   amber:
     github: amberframework/amber
-    #version: <%= Amber::VERSION %>
-    branch: master
+    version: <%= Amber::VERSION %>
 
   granite:
     github: amberframework/granite

src/amber/cli/templates/app/src/views/home/index.ecr.ecr

@@ -4,8 +4,8 @@
     <p>Thank you for trying out the Amber Framework.  We are working hard to provide a super fast and reliable framework that provides all the productivity tools you are used to without sacrificing the speed.</p>
     <div class="list-group">
       <a class="list-group-item list-group-item-action" target="_blank" href="https://docs.amberframework.org">Getting Started with Amber Framework</a>
-      <a class="list-group-item list-group-item-action" target="_blank" href="https://github.com/veelenga/awesome-crystal">List of Awesome Crystal projects and shards</a>
-      <a class="list-group-item list-group-item-action" target="_blank" href="https://crystalshards.xyz">What's hot in Crystal right now</a>
+      <a class="list-group-item list-group-item-action" target="_blank" href="https://github.com/veelenga/awesome-crystal">List of Awesome C
+      <a class="list-group-item list-group-item-action" target="_blank" href="https://discord.gg/vwvP5zakSn">Join the Amber Discord!</a>
     </div>
   </div>
 </div>

src/amber/cli/templates/app/src/views/home/index.slang.ecr

@@ -5,4 +5,4 @@
     .list-group
       a.list-group-item.list-group-item-action target="_blank" href="https://docs.amberframework.org" Getting Started with Amber Framework
       a.list-group-item.list-group-item-action target="_blank" href="https://github.com/veelenga/awesome-crystal" List of Awesome Crystal projects and shards
-      a.list-group-item.list-group-item-action target="_blank" href="https://crystalshards.xyz" What's hot in Crystal right now
+      a.list-grouo-item.list-group-item-action target="_blank" href="https://discord.gg/vwvP5zakSn" Join the Amber Discord!

src/amber/cli/templates/error/spec/controllers/{{name}}_controller_spec.cr.ecr

@@ -28,7 +28,7 @@ describe <%= class_name %>Controller do
 end
 
 # Controller Integration Test
-class <%= class_name %>ControllerTest < GarnetSpec::Controller::Test
+class <%= class_name %>ControllerTest < GarnetSpec::SystemTest
   getter handler : Amber::Pipe::Pipeline
 
   def initialize

src/amber/cli/templates/scaffold/controller/spec/controllers/{{name}}_controller_spec.cr.ecr

@@ -25,7 +25,7 @@ def <%= create_model_method %>
   model
 end
 
-class <%= class_name %>ControllerTest < GarnetSpec::Controller::Test
+class <%= class_name %>ControllerTest < GarnetSpec::SystemTest
   getter handler : Amber::Pipe::Pipeline
 
   def initialize

src/amber/controller/helpers/responders.cr

@@ -90,7 +90,7 @@ module Amber::Controller::Helpers
       accept = context.request.headers["Accept"]?
       if accept && !accept.empty?
         accepts = accept.split(";").first?.try(&.split(Content::ACCEPT_SEPARATOR_REGEX))
-        return accepts if !accepts.nil? && accepts.any?
+        return accepts if !accepts.nil? && !accepts.empty?
       end
     end
 

src/amber/pipes/cors.cr

@@ -63,7 +63,7 @@ module Amber
       private def put_response_headers(response)
         response.headers[Headers::ALLOW_CREDENTIALS] = @credentials.to_s if @credentials
         response.headers[Headers::ALLOW_ORIGIN] = @origin.request_origin.not_nil!
-        response.headers[Headers::VARY] = vary unless @origin.any?
+        response.headers[Headers::VARY] = vary unless @origin.any_origin?
       end
 
       private def vary
@@ -122,7 +122,7 @@ module Amber
       def match?(request)
         return false if @origins.empty?
         return false unless origin_header?(request)
-        return true if any?
+        return true if any_origin?
 
         @origins.any? do |origin|
           case origin
@@ -132,7 +132,7 @@ module Amber
         end
       end
 
-      def any?
+      def any_origin?
         @origins.includes? "*"
       end
 

src/amber/router/cookies/store.cr

@@ -90,6 +90,9 @@ module Amber::Router::Cookies
             expires : Time? = nil, domain : String? = nil,
             secure : Bool = false, http_only : Bool = false,
             extension : String? = nil)
+      name = URI.encode_www_form(name)
+      value = URI.encode_www_form(value)
+
       if @cookies[name]? != value || expires
         @cookies[name] = value
         @set_cookies[name] = HTTP::Cookie.new(name, value, path, expires, domain, secure, http_only, extension)

src/amber/router/session/abstract_store.cr

@@ -3,7 +3,7 @@ module Amber::Router::Session
   abstract class AbstractStore
     abstract def id
     abstract def destroy
-    abstract def update(other_hash : Hash(String | Symbol, String))
+    abstract def update(hash : Hash(String | Symbol, String))
     abstract def set_session
     abstract def current_session
   end

src/amber/router/session/redis_store.cr

@@ -56,7 +56,7 @@ module Amber::Router::Session
     end
 
     def to_h
-      store.hgetall(session_id).each_slice(2).to_h
+      store.hgetall(session_id)
     end
 
     def update(hash : Hash(String | Symbol, String))

src/amber/version.cr

@@ -1,3 +1,3 @@
 module Amber
-  VERSION = "1.2.2"
+  VERSION = "1.3.2"
 end

src/amber/websockets/client_socket.cr

@@ -45,7 +45,7 @@ module Amber
 
       def self.get_topic_channel(topic_path)
         topic_channels = @@channels.select { |ch| WebSockets.topic_path(ch[:path]) == topic_path }
-        return topic_channels[0][:channel] if topic_channels.any?
+        return topic_channels[0][:channel] if !topic_channels.empty?
       end
 
       # Broadcast a message to all subscribers of the topic