Merge branch 'master' into dependabot/hex/oban-2.17.12

.github/workflows/test.yml

@@ -4,8 +4,8 @@ name: Test
 on: push
 
 env:
-  OTP_VERSION_SPEC: "24.1"
-  ELIXIR_VERSION_SPEC: "1.12.3"
+  OTP_VERSION_SPEC: "26.2.4"
+  ELIXIR_VERSION_SPEC: "1.16.2"
   MIX_ENV: test
 
 jobs:

.tool-versions

@@ -1,2 +1,2 @@
-erlang 24.1.7
-elixir 1.12.3-otp-24
+erlang 26.2.4
+elixir 1.16.2-otp-26

Dockerfile

@@ -1,5 +1,5 @@
 # Set the Docker image you want to base your image off.
-FROM hexpm/elixir:1.12.3-erlang-24.1.7-debian-buster-20210902 as builder
+FROM hexpm/elixir:1.16.2-erlang-26.2.4-debian-bullseye-20240423-slim as builder
 
 # Install other stable dependencies that don't change often
 RUN apt-get update && \
@@ -27,7 +27,7 @@ RUN mix do compile, release
 # END BUILDER
 #
 
-FROM debian:buster-slim
+FROM debian:bullseye-slim
 
 RUN apt-get -qq update
 RUN apt-get -qq install -y locales locales-all openssl

lib/slax/github.ex

@@ -2,7 +2,6 @@ defmodule Slax.Github do
   @moduledoc """
   Functions for working with the Github API
   """
-
   alias Slax.Http
   alias Slax.Http.Error
   alias Slax.ProjectRepos
@@ -507,7 +506,7 @@ defmodule Slax.Github do
   """
   def load_issue(repo_and_issue) do
     with {org, repo, issue} <- parse_repo_org_issue(repo_and_issue),
-         {token, warning_message} <- retrieve_token(repo),
+         {token, warning_message} <- retrieve_token(repo, org),
          client <- Tentacat.Client.new(%{access_token: token}),
          {200, issue, _http_response} <- Issues.find(client, org, repo, issue) do
       {:ok, issue, warning_message}
@@ -531,7 +530,7 @@ defmodule Slax.Github do
 
   def load_pr(repo_and_pr) do
     with {org, repo, pr} <- parse_repo_org_issue(repo_and_pr),
-         {token, warning_message} <- retrieve_token(repo),
+         {token, warning_message} <- retrieve_token(repo, org),
          client <- Tentacat.Client.new(%{access_token: token}),
          {200, pr, _http_response} <- Prs.find(client, org, repo, pr) do
       {:ok, pr, warning_message}
@@ -566,8 +565,8 @@ defmodule Slax.Github do
     end)
   end
 
-  defp retrieve_token(repo) do
-    case ProjectRepos.get_by_repo(repo) do
+  defp retrieve_token(repo, org) do
+    case ProjectRepos.get_by_repo_and_org(repo, org) do
       %{token: token} when not is_nil(token) ->
         {token, ""}
 

lib/slax/poker.ex

@@ -72,7 +72,7 @@ defmodule Slax.Poker do
     {org, repo, issue} = Github.parse_repo_org_issue(round.issue)
 
     client =
-      case ProjectRepos.get_by_repo(repo) do
+      case ProjectRepos.get_by_repo_and_org(repo, org) do
         %{token: token} when not is_nil(token) ->
           Tentacat.Client.new(%{access_token: token})
 

lib/slax/projects/project_repos.ex

@@ -72,8 +72,19 @@ defmodule Slax.ProjectRepos do
     |> Repo.all()
   end
 
-  def get_by_repo(repo_name) do
-    Repo.get_by(ProjectRepo, repo_name: repo_name)
+  def get_by_repo_and_org(repo_name, org_name) do
+    lower_repo_name = String.downcase(repo_name)
+    lower_org_name = String.downcase(org_name)
+
+    query =
+      from(
+        p in ProjectRepo,
+        where:
+          fragment("lower(?)", p.repo_name) == ^lower_repo_name and
+            fragment("lower(?)", p.org_name) == ^lower_org_name
+      )
+
+    Repo.one(query)
   end
 
   def list_needs_reminder_message() do

lib/slax_web/websockets/issue.ex

@@ -55,7 +55,7 @@ defmodule SlaxWeb.Issue do
   defp maybe_load_from_default_repo(default_repo, issue_number) do
     case default_repo do
       nil ->
-        "No default repo set for this channel"
+        "Use the format 'org/repo#{issue_number}' or set a default repo for this channel with '/slax'."
 
       _ ->
         org_name = default_repo.org_name

lib/slax_web/websockets/poker.ex

@@ -89,7 +89,7 @@ defmodule SlaxWeb.Poker do
         estimates =
           round.estimates
           |> Enum.sort_by(& &1.value)
-          |> Enum.map(&"#{&1.user} (#{&1.value})#{if &1.reason, do: ': #{&1.reason}'}")
+          |> Enum.map(&"#{&1.user} (#{&1.value})#{if &1.reason, do: ~c": #{&1.reason}"}")
 
         message = "Estimates for round:\n---\n#{Enum.join(estimates, "\n")}"
 

mix.exs

@@ -7,7 +7,7 @@ defmodule Slax.Mixfile do
       version: "0.0.1",
       elixir: "~> 1.6",
       elixirc_paths: elixirc_paths(Mix.env()),
-      compilers: [:phoenix] ++ Mix.compilers(),
+      compilers: Mix.compilers(),
       build_embedded: Mix.env() == :prod,
       start_permanent: Mix.env() == :prod,
       aliases: aliases(),
@@ -33,10 +33,10 @@ defmodule Slax.Mixfile do
   # Type `mix help deps` for examples and options.
   defp deps do
     [
-      {:phoenix, "~> 1.6.0", override: true},
+      {:phoenix, "~> 1.7.14", override: true},
       {:phoenix_pubsub, "~> 2.0"},
       {:phoenix_ecto, "~> 4.0"},
-      {:ecto_sql, "~> 3.11.1"},
+      {:ecto_sql, "~> 3.12.1"},
       {:postgrex, "~> 0.16"},
       {:plug_cowboy, "~> 2.5"},
       {:plug, "~> 1.15.3"},
@@ -47,7 +47,7 @@ defmodule Slax.Mixfile do
       {:jason, "~> 1.1"},
       {:ex_machina, "~> 2.2", only: :test},
       {:sobelow, "~> 0.13", only: [:dev, :test], runtime: false},
-      {:stream_data, "~> 0.6.0", only: :test},
+      {:stream_data, "~> 1.1.1", only: :test},
       {:quantum, "~> 3.0"},
       {:timex, "~> 3.7"},
       {:tentacat, "~> 2.2.0"},
@@ -56,7 +56,8 @@ defmodule Slax.Mixfile do
       {:gun, "~> 2.0.1"},
       {:oban, "~> 2.13"},
       {:certifi, "~> 2.8.0"},
-      {:castore, "~> 1.0"}
+      {:castore, "~> 1.0"},
+      {:phoenix_view, "~> 2.0"}
     ]
   end