Merge pull request #1703 from Logflare/feat/display-endpoints-auth

feat: display whether endpoints auth is enabled on index page
Logflare · Oct 2, 2023 · 4991fc9 · 4991fc9
2 parents 5c5b9c6 + a55893f
commit 4991fc9
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/lib/logflare_web/live/endpoints/actions/index.html.heex b/lib/logflare_web/live/endpoints/actions/index.html.heex
@@ -33,6 +33,13 @@
         <%= endpoint.description %>
       </p>
       <div class="tw-text-sm">
+        <span :if={endpoint.enable_auth} title="Auth enabled">
+          <i class="fas fa-lock"></i>
+        </span>
+
+        <span :if={!endpoint.enable_auth} title="Auth disabled">
+          <i class="fas fa-lock-open"></i>
+        </span>
         <span>caches: <%= endpoint.metrics.cache_count %></span>
       </div>
     </li>

diff --git a/lib/logflare_web/live/endpoints/endpoints_live.ex b/lib/logflare_web/live/endpoints/endpoints_live.ex
@@ -75,6 +75,13 @@ defmodule LogflareWeb.EndpointsLive do
           # set changeset
           |> assign(:endpoint_changeset, Endpoints.change_query(endpoint, %{}))
 
+        # index page
+        %{assigns: %{live_action: :index}} = socket ->
+          socket
+          |> refresh_endpoints()
+          |> assign(:endpoint_changeset, nil)
+          |> assign(:query_result_rows, nil)
+
         other ->
           other
           # reset the changeset

diff --git a/test/logflare_web/live_views/endpoints_live_test.exs b/test/logflare_web/live_views/endpoints_live_test.exs
@@ -26,6 +26,7 @@ defmodule LogflareWeb.EndpointsLiveTest do
 
       # description
       assert has_element?(view, "ul li p", endpoint.description)
+      assert has_element?(view, "ul li *[title='Auth enabled']")
 
       # link to show
       view