From d2c96da5f2eb01405b857f728596778525f5574c Mon Sep 17 00:00:00 2001
From: "Jens L." <jens@goauthentik.io>
Date: Wed, 31 Jul 2024 18:00:24 +0200
Subject: [PATCH] providers/radius: fix custom attribute (#10704)

* providers/radius: fix error when adding custom attribute

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* fix ldap source property mapping form

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

---------

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
---
 authentik/providers/radius/api/providers.py                | 5 +++--
 web/src/admin/property-mappings/PropertyMappingTestForm.ts | 4 +++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/authentik/providers/radius/api/providers.py b/authentik/providers/radius/api/providers.py
index 9ded0acbc3..68e219dba6 100644
--- a/authentik/providers/radius/api/providers.py
+++ b/authentik/providers/radius/api/providers.py
@@ -117,8 +117,9 @@ class RadiusOutpostConfigViewSet(ListModelMixin, GenericViewSet):
             # Ensure the vendor exists
             if vendor_code not in dict.vendors.backward or vendor_name not in dict.vendors.forward:
                 dict.vendors.Add(vendor_name, vendor_code)
-            if attribute_name not in dict.attributes:
-                dict.attributes[f"{vendor_name}-{attribute_name}"] = Attribute(
+            full_attribute_name = f"{vendor_name}-{attribute_name}"
+            if full_attribute_name not in dict.attributes:
+                dict.attributes[full_attribute_name] = Attribute(
                     attribute_name, attribute_code, attribute_type, vendor=vendor_name
                 )
 
diff --git a/web/src/admin/property-mappings/PropertyMappingTestForm.ts b/web/src/admin/property-mappings/PropertyMappingTestForm.ts
index 8aab23c4a8..0a0372132d 100644
--- a/web/src/admin/property-mappings/PropertyMappingTestForm.ts
+++ b/web/src/admin/property-mappings/PropertyMappingTestForm.ts
@@ -21,6 +21,7 @@ import {
     PropertyMappingTestRequest,
     PropertyMappingTestResult,
     PropertymappingsApi,
+    RbacPermissionsAssignedByUsersListModelEnum,
     User,
 } from "@goauthentik/api";
 
@@ -67,7 +68,8 @@ export class PolicyTestForm extends Form<PropertyMappingTestRequest> {
     }
 
     renderExampleButtons() {
-        return this.mapping?.metaModelName === "authentik_sources_ldap.ldappropertymapping"
+        return this.mapping?.metaModelName ===
+            RbacPermissionsAssignedByUsersListModelEnum.SourcesLdapLdapsourcepropertymapping
             ? html`<p>${msg("Example context data")}</p>
                   ${this.renderExampleLDAP()}`
             : nothing;