fix: resolved login issue (#844)

* fix: Improve Handling of Sensitive Data in Browser Signed-off-by: bhavanakarwade <[email protected]> * improved error handling for layout component Signed-off-by: bhavanakarwade <[email protected]> * refactor: improve variables initialization logic Signed-off-by: bhavanakarwade <[email protected]> * removed hardcoded variables Signed-off-by: bhavanakarwade <[email protected]> * added file path in yml file Signed-off-by: bhavanakarwade <[email protected]> * refactor: added variable in constants file Signed-off-by: bhavanakarwade <[email protected]> * fix: signin issue Signed-off-by: bhavanakarwade <[email protected]> --------- Signed-off-by: bhavanakarwade <[email protected]>
credebl · Dec 23, 2024 · d7b19c7 · d7b19c7
1 parent a4f8a93
commit d7b19c7
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/src/app/LayoutCommon.astro b/src/app/LayoutCommon.astro
@@ -16,7 +16,6 @@ if (allEnvKeys.length === 0) {
 const exposedEnvKeys = allEnvKeys.filter((key) => !excludeKeys.includes(key));
 
 const initData: Record<string, any> = {};
-const excludedEnvData: Record<string, any> = {};
 
 allEnvKeys.forEach((key) => {
   const value = process.env[key] || import.meta.env[key];

diff --git a/src/config/SocketConfig.ts b/src/config/SocketConfig.ts
@@ -1,7 +1,7 @@
 import { envConfig } from "./envConfig"
 import io from "socket.io-client"
 
-const SOCKET = io(`${import.meta.env.PUBLIC_BASE_URL}`, {
+const SOCKET = io(`${envConfig.PUBLIC_BASE_URL}`, {
     reconnection: true,
     reconnectionDelay: 500,
     reconnectionAttempts: Infinity,

diff --git a/src/config/ssrApiConfig.ts b/src/config/ssrApiConfig.ts
@@ -20,7 +20,7 @@ const API = async ({ token, url, method, payload }: IProps) => {
 			method,
 			body: JSON.stringify(payload),
 		};
-		const baseURL = globalThis.baseUrl || import.meta.env.PUBLIC_BASE_URL || process.env.PUBLIC_BASE_URL;
+		const baseURL = globalThis.baseUrl || envConfig.PUBLIC_BASE_URL || process.env.PUBLIC_BASE_URL;
 		const apiURL = baseURL + url;
 		const res = await fetch(apiURL, {
 			...config,

diff --git a/src/pages/user/[user].astro b/src/pages/user/[user].astro
@@ -1,10 +1,11 @@
 ---
 import LogoFile from '../../commonComponents/LogoFile';
 import CustomAvatar from '../../components/Avatar';
+import { envConfig } from '../../config/envConfig';
 import { pathRoutes } from '../../config/pathRoutes';
 
 const { user } = Astro.params;
-const baseUrl = process.env.PUBLIC_BASE_URL || import.meta.env.PUBLIC_BASE_URL
+const baseUrl = process.env.PUBLIC_BASE_URL || envConfig.PUBLIC_BASE_URL
 const response = await fetch(`${baseUrl}/users/public-profiles/${user}`);
 const data = await response.json();
 const userData = data?.data;

diff --git a/src/services/axiosIntercepter.ts b/src/services/axiosIntercepter.ts
@@ -6,7 +6,7 @@ import { getFromLocalStorage, setToLocalStorage } from '../api/Auth';
 import { apiStatusCodes, storageKeys } from '../config/CommonConstant';
 
 const instance = axios.create({
-	baseURL: import.meta.env.PUBLIC_BASE_URL,
+	baseURL: envConfig.PUBLIC_BASE_URL,
 });
 
 const EcosystemInstance = axios.create({
@@ -16,7 +16,7 @@ const EcosystemInstance = axios.create({
 const checkAuthentication = async (sessionCookie: string, request: AxiosRequestConfig) => {
 	const isAuthPage = window.location.href.includes('/authentication/sign-in') || window.location.href.includes('/authentication/sign-up')
 	try {
-		const baseURL = import.meta.env.PUBLIC_BASE_URL || process.env.PUBLIC_BASE_URL;
+		const baseURL = envConfig.PUBLIC_BASE_URL || process.env.PUBLIC_BASE_URL;
 		const config = {
 			headers: {
 				'Content-Type': 'application/json',

diff --git a/src/utils/check-session.ts b/src/utils/check-session.ts
@@ -33,7 +33,7 @@ export const checkUserSession = async ({
 
 	try {
 		const baseURL =
-			import.meta.env.PUBLIC_BASE_URL ||
+			envConfig.PUBLIC_BASE_URL ||
 			process.env.PUBLIC_BASE_URL;
 		const config = {
 			headers: {