Merge branch 'develop'

Author: ku1ik

assets/css/_base.scss

@@ -1,4 +1,4 @@
-@import "./_source_sans_pro.css";
+@import "./fonts/source_sans_pro.css";
 
 html {
   height: 100%;
@@ -142,6 +142,18 @@ form {
   .col-form-label {
     font-weight: 600;
   }
+
+  .form-group.row {
+    margin-bottom: 1.5rem;
+  }
+
+  .form-check {
+      margin-bottom: 0.75em;
+  }
+
+  .visibility-radios {
+    padding-top: 0.375rem;
+  }
 }
 
 .special-label {

assets/css/app.scss

@@ -18,7 +18,7 @@ $primary: #06989a;
 @import "./_user_profile.scss";
 @import "./_user_login.scss";
 @import "./_user_edit.scss";
-@import "./powerline-symbols.css";
+@import "./fonts/powerline-symbols.css";
 @import "./fonts/fira-code-nerd-font.css";
 @import "./fonts/jetbrains-mono-nerd-font.css";
 @import "~asciinema-player/dist/bundle/asciinema-player.css";

assets/css/powerline-symbols.css renamed to assets/css/fonts/powerline-symbols.css

@@ -1,4 +1,4 @@
 @font-face {
   font-family: 'Powerline Symbols';
-  src: local('Powerline Symbols'), local('PowerlineSymbols'), url('../static/fonts/PowerlineSymbols.otf') format('opentype');
+  src: local('Powerline Symbols'), local('PowerlineSymbols'), url('../../static/fonts/PowerlineSymbols.otf') format('opentype');
 }

assets/css/_source_sans_pro.css renamed to assets/css/fonts/source_sans_pro.css

@@ -2,29 +2,29 @@
   font-family: 'Source Sans Pro';
   font-style: normal;
   font-weight: 300;
-  src: local('Source Sans Pro Light'), local('SourceSansPro-Light'), url('../static/fonts/SourceSansPro-Light.ttf') format('truetype');
+  src: local('Source Sans Pro Light'), local('SourceSansPro-Light'), url('../../static/fonts/SourceSansPro-Light.ttf') format('truetype');
 }
 @font-face {
   font-family: 'Source Sans Pro';
   font-style: normal;
   font-weight: 400;
-  src: local('Source Sans Pro'), local('SourceSansPro-Regular'), url('../static/fonts/SourceSansPro-Regular.ttf') format('truetype');
+  src: local('Source Sans Pro'), local('SourceSansPro-Regular'), url('../../static/fonts/SourceSansPro-Regular.ttf') format('truetype');
 }
 @font-face {
   font-family: 'Source Sans Pro';
   font-style: normal;
   font-weight: 600;
-  src: local('Source Sans Pro Semibold'), local('SourceSansPro-Semibold'), url('../static/fonts/SourceSansPro-Semibold.ttf') format('truetype');
+  src: local('Source Sans Pro Semibold'), local('SourceSansPro-Semibold'), url('../../static/fonts/SourceSansPro-Semibold.ttf') format('truetype');
 }
 @font-face {
   font-family: 'Source Sans Pro';
   font-style: normal;
   font-weight: 700;
-  src: local('Source Sans Pro Bold'), local('SourceSansPro-Bold'), url('../static/fonts/SourceSansPro-Bold.ttf') format('truetype');
+  src: local('Source Sans Pro Bold'), local('SourceSansPro-Bold'), url('../../static/fonts/SourceSansPro-Bold.ttf') format('truetype');
 }
 @font-face {
   font-family: 'Source Sans Pro';
   font-style: italic;
   font-weight: 400;
-  src: local('Source Sans Pro Italic'), local('SourceSansPro-It'), url('../static/fonts/SourceSansPro-Italic.ttf') format('truetype');
+  src: local('Source Sans Pro Italic'), local('SourceSansPro-It'), url('../../static/fonts/SourceSansPro-Italic.ttf') format('truetype');
 }

assets/css/iframe.scss

@@ -1,6 +1,8 @@
-@import "./_source_sans_pro.css";
+@import "./fonts/source_sans_pro.css";
+@import "./fonts/fira-code-nerd-font.css";
+@import "./fonts/jetbrains-mono-nerd-font.css";
 @import "~asciinema-player/dist/bundle/asciinema-player.css";
-@import "./powerline-symbols.css";
+@import "./fonts/powerline-symbols.css";
 
 body.iframe {
   background-color: transparent;

config/config.exs

@@ -7,6 +7,12 @@
 # General application configuration
 import Config
 
+config :asciinema, :session_opts,
+  store: :cookie,
+  key: "_asciinema_key",
+  signing_salt: "qJL+3s0T",
+  same_site: "Lax"
+
 config :asciinema,
   ecto_repos: [Asciinema.Repo]
 
@@ -47,7 +53,7 @@ config :logger,
 config :phoenix, :json_library, Jason
 
 config :phoenix, :template_engines, md: PhoenixMarkdown.Engine
-config :phoenix_template, :format_encoders, svg: Phoenix.HTML.Engine
+config :phoenix_template, :format_encoders, svg: Phoenix.HTML.Engine, xml: Phoenix.HTML.Engine
 
 config :sentry,
   dsn: "https://public:secret@sentry.io/1",

lib/asciinema.ex

@@ -31,6 +31,9 @@ defmodule Asciinema do
   defdelegate verify_login_token(token), to: Accounts
 
   def merge_accounts(src_user, dst_user) do
+    src_user = Accounts.find_user(src_user)
+    dst_user = Accounts.find_user(dst_user)
+
     Repo.transact(fn ->
       Recordings.reassign_asciicasts(src_user.id, dst_user.id)
       Streaming.reassign_live_streams(src_user.id, dst_user.id)

lib/asciinema/accounts.ex

@@ -6,7 +6,7 @@ defmodule Asciinema.Accounts do
   alias Asciinema.{Fonts, Repo, Themes}
   alias Ecto.Changeset
 
-  @valid_email_re ~r/^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,}$/i
+  @valid_email_re ~r/^[A-Z0-9._%+-]+@([A-Z0-9-]+\.)+[A-Z]{2,}$/i
   @valid_username_re ~r/^[a-zA-Z0-9][a-zA-Z0-9-]*[a-zA-Z0-9]$/
 
   def fetch_user(id) do
@@ -20,6 +20,16 @@ defmodule Asciinema.Accounts do
 
   def get_user(id), do: Repo.get(User, id)
 
+  def find_user(%User{} = user), do: user
+
+  def find_user(id) when is_integer(id), do: get_user(id)
+
+  def find_user(id) when is_binary(id) do
+    {_, user} = lookup_user(id)
+
+    user
+  end
+
   def find_user_by_username(username) do
     Repo.one(
       from(u in User,
@@ -38,8 +48,9 @@ defmodule Asciinema.Accounts do
     result =
       %User{}
       |> cast(attrs, [:email])
-      |> validate_format(:email, @valid_email_re)
       |> validate_required([:email])
+      |> update_change(:email, &String.downcase/1)
+      |> validate_format(:email, @valid_email_re)
       |> add_contraints()
       |> Repo.insert()
 
@@ -79,6 +90,8 @@ defmodule Asciinema.Accounts do
       :terminal_font_family,
       :asciicasts_private_by_default
     ])
+    |> validate_required([:email])
+    |> update_change(:email, &String.downcase/1)
     |> validate_format(:email, @valid_email_re)
     |> validate_format(:username, @valid_username_re)
     |> validate_length(:username, min: 2, max: 16)
@@ -100,7 +113,7 @@ defmodule Asciinema.Accounts do
 
     user
     |> change_user(params)
-    |> validate_required([:username, :email])
+    |> validate_required([:username])
     |> Repo.update()
   end
 
@@ -139,7 +152,7 @@ defmodule Asciinema.Accounts do
 
   def lookup_user(identifier) when is_binary(identifier) do
     if String.contains?(identifier, "@") do
-      {:email, Repo.get_by(User, email: identifier)}
+      {:email, Repo.get_by(User, email: String.downcase(identifier))}
     else
       {:username, find_user_by_username(identifier)}
     end

lib/asciinema/authorization.ex

@@ -4,6 +4,7 @@ defmodule Asciinema.Authorization do
   alias Asciinema.Streaming.LiveStream
 
   defmodule Policy do
+    def can?(_user, :show, %LiveStream{visibility: v}) when v in [:public, :unlisted], do: true
     def can?(nil, _action, _thing), do: false
     def can?(%User{is_admin: true}, _action, _thing), do: true
     def can?(_user, :make_featured, %Asciicast{}), do: false
@@ -23,16 +24,4 @@ defmodule Asciinema.Authorization do
 
     Policy.can?(user, action, thing)
   end
-
-  defmodule ForbiddenError do
-    defexception plug_status: 403, message: "Forbidden"
-  end
-
-  def can!(user, action, thing) do
-    if can?(user, action, thing) do
-      :ok
-    else
-      raise ForbiddenError
-    end
-  end
 end

lib/asciinema/file_store/local.ex

@@ -42,7 +42,7 @@ defmodule Asciinema.FileStore.Local do
     conn
     |> put_resp_header("content-type", MIME.from_path(path))
     |> send_file(200, full_path(path))
-    |> halt
+    |> halt()
   end
 
   @impl true