From 0e21b41138a43ca73ab28aba8501e00a4ec06183 Mon Sep 17 00:00:00 2001
From: aeberhardt <alexandre.eberhardt@savoirfairelinux.com>
Date: Mon, 27 Jan 2025 16:25:00 -0500
Subject: [PATCH] public nameserver: fix registration error
this commit fix the error when creating user while using the public nameserver
replace http by https in the nameserver address
add Publickey and Signature to the registration request
Gitlab: #150
Change-Id: I124fe47a7d6f3c263f441dc8a3fe454a23c91742
---
.../main/java/net/jami/jams/authmodule/RegisterUserFlow.java | 4 ++++
.../src/components/IdentityManagement/IdentityManagement.tsx | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/authentication-module/src/main/java/net/jami/jams/authmodule/RegisterUserFlow.java b/authentication-module/src/main/java/net/jami/jams/authmodule/RegisterUserFlow.java
index 3102d3cc..9d40d55b 100644
--- a/authentication-module/src/main/java/net/jami/jams/authmodule/RegisterUserFlow.java
+++ b/authentication-module/src/main/java/net/jami/jams/authmodule/RegisterUserFlow.java
@@ -56,8 +56,12 @@ public class RegisterUserFlow {
NameRegistrationRequest nameRegistrationRequest = new NameRegistrationRequest();
nameRegistrationRequest.setOwner(ethKeyPair[0]);
nameRegistrationRequest.setAddr(user.getJamiId());
+ nameRegistrationRequest.setPublickey("");
+ nameRegistrationRequest.setSignature("");
if (nameServer != null
&& nameServer.registerName(user.getUsername(), nameRegistrationRequest) != 200) {
+ log.error(
+ "Failed to register the user " + user.getUsername() + " with the name server.");
return false;
}
datastore.getUserDao().storeObject(user);
diff --git a/jams-react-client/src/components/IdentityManagement/IdentityManagement.tsx b/jams-react-client/src/components/IdentityManagement/IdentityManagement.tsx
index 1e6f304b..17bf83d5 100644
--- a/jams-react-client/src/components/IdentityManagement/IdentityManagement.tsx
+++ b/jams-react-client/src/components/IdentityManagement/IdentityManagement.tsx
@@ -273,7 +273,7 @@ export default function IdentityManagement(props: IdentityManagementProps) {
activeDirectorySettings: {},
};
const settings: localSettings = {
- publicNameServer: "http://ns.jami.net",
+ publicNameServer: "https://ns.jami.net",
publicNames: nameServerChecked,
};
--
GitLab