Skip to content

Commit 39f1ab9

Browse files
authored
Cleanup unused env variables (#137)
1 parent c33c250 commit 39f1ab9

File tree

8 files changed

+5
-116
lines changed

8 files changed

+5
-116
lines changed

.env.example

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,11 @@
1-
# WALLET [Required]
2-
# ------
3-
# Either a private key
4-
WALLET_PRIVATE_KEY=<your_admin_wallet_private_key>
5-
# Or from AWS KMS
1+
# WALLET [Optional]
2+
# If you want, you can configure AWS KMS
63
AWS_KMS_KEY_ID=<your_aws_kms_key_id>
74
AWS_ACCESS_KEY_ID=<your_aws_access_key_id>
85
AWS_SECRET_ACCESS_KEY=<your_aws_secret_access_key>
96
AWS_REGION=<your_aws_region>
107

11-
# Or from GOOGLE KMS
8+
# Or GOOGLE KMS
129
# Required for Google Auth
1310
GOOGLE_APPLICATION_CREDENTIAL_EMAIL=<client_email_from_download_service_account_json>
1411
GOOGLE_APPLICATION_CREDENTIAL_PRIVATE_KEY=<private_key_from_download_service_account_json>

Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ export POSTGRES_USER=postgres
99
export POSTGRES_PASSWORD=postgres
1010
export POSTGRES_PORT=5432
1111
export POSTGRES_USE_SSL=false
12-
export WALLET_PRIVATE_KEY=0xac0974bec39a17e36ba4a6b4d238ff944bacb478cbed5efcae784d7bf4f2ff80
1312
# export THIRDWEB_API_SECRET_KEY=klRsqmatrdlEpik_pHKgYy_q2YzGe3bTewO1VC26eY_H184Kc7xOVqKVj0mHwOOW2AOx2N-a3GqLCQ7Z9s9-sw
1413

1514
test-evm: FORCE

core/database/dbConnect.ts

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,13 @@ import pg, { Knex } from "knex";
22
import { env } from "../env";
33

44
// Defaults to postgres
5-
const dbClient = env.DATABASE_CLIENT;
65
const connectionString = env.POSTGRES_CONNECTION_URL;
76

87
export const connectToDatabase = async (
98
databaseURL?: string,
109
): Promise<Knex> => {
1110
let knexConfig: Knex.Config = {
12-
client: dbClient,
11+
client: "pg",
1312
connection: {
1413
connectionString: databaseURL || connectionString,
1514
ssl: {
@@ -19,18 +18,8 @@ export const connectToDatabase = async (
1918
acquireConnectionTimeout: 10000,
2019
};
2120

22-
// Set the appropriate databse client package
23-
let dbClientPackage: typeof pg;
24-
switch (dbClient) {
25-
case "pg":
26-
dbClientPackage = pg;
27-
break;
28-
default:
29-
throw new Error(`Unsupported database client: ${dbClient}`);
30-
}
31-
3221
// instantiate connection with new config
33-
const knex = dbClientPackage(knexConfig);
22+
const knex = pg(knexConfig);
3423

3524
return knex;
3625
};

core/database/dbPrereqs.ts

Lines changed: 0 additions & 73 deletions
This file was deleted.

core/env.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -23,25 +23,18 @@ export const env = createEnv({
2323
NODE_ENV: z
2424
.enum(["production", "development", "testing", "local"])
2525
.default("development"),
26-
WALLET_PRIVATE_KEY: z.string().min(1).optional(),
2726
AWS_KMS_KEY_ID: z.string().min(1).optional(),
2827
GOOGLE_KMS_KEY_ID: z.string().min(1).optional(),
2928
AWS_ACCESS_KEY_ID: z.string().min(1).optional(),
3029
AWS_SECRET_ACCESS_KEY: z.string().min(1).optional(),
3130
AWS_REGION: z.string().min(1).optional(),
3231
THIRDWEB_API_SECRET_KEY: z.string().min(1),
33-
THIRDWEB_API_ORIGIN: z.string().default("http://api.thirdweb.com"),
34-
DATABASE_CLIENT: z.string().default("pg"),
3532
POSTGRES_CONNECTION_URL: z
3633
.string()
3734
.default(
3835
"postgresql://postgres:postgres@localhost:5432/postgres?sslmode=disable",
3936
),
4037
OPENAPI_BASE_ORIGIN: z.string().default("http://localhost:3005"),
41-
DB_TABLES_LIST: z.string().default("wallets,transactions"),
42-
DB_TRIGGERS_LIST: z
43-
.string()
44-
.default("trigger_notification,trigger_tx_table"),
4538
PORT: z.coerce.number().default(3005),
4639
HOST: z.string().default("0.0.0.0"),
4740
MIN_TRANSACTION_TO_PROCESS: z.coerce.number().default(1),
@@ -69,18 +62,13 @@ export const env = createEnv({
6962
isServer: true,
7063
runtimeEnvStrict: {
7164
NODE_ENV: process.env.NODE_ENV,
72-
WALLET_PRIVATE_KEY: process.env.WALLET_PRIVATE_KEY,
7365
AWS_KMS_KEY_ID: process.env.AWS_KMS_KEY_ID,
7466
GOOGLE_KMS_KEY_ID: process.env.GOOGLE_KMS_KEY_ID,
7567
AWS_ACCESS_KEY_ID: process.env.AWS_ACCESS_KEY_ID,
7668
AWS_SECRET_ACCESS_KEY: process.env.AWS_SECRET_ACCESS_KEY,
7769
AWS_REGION: process.env.AWS_REGION,
7870
THIRDWEB_API_SECRET_KEY: process.env.THIRDWEB_API_SECRET_KEY,
79-
THIRDWEB_API_ORIGIN: process.env.THIRDWEB_API_ORIGIN,
80-
DATABASE_CLIENT: process.env.DATABASE_CLIENT,
8171
POSTGRES_CONNECTION_URL: process.env.POSTGRES_CONNECTION_URL,
82-
DB_TABLES_LIST: process.env.DB_TABLES_LIST,
83-
DB_TRIGGERS_LIST: process.env.DB_TRIGGERS_LIST,
8472
PORT: process.env.PORT,
8573
HOST: process.env.HOST,
8674
OPENAPI_BASE_ORIGIN: process.env.OPENAPI_BASE_ORIGIN,

core/helpers/index.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,3 @@
1-
import { env } from "../env";
2-
3-
const AWS_ACCESS_KEY_ID = env.AWS_ACCESS_KEY_ID;
4-
const AWS_SECRET_ACCESS_KEY = env.AWS_SECRET_ACCESS_KEY;
5-
const AWS_REGION = env.AWS_REGION;
6-
7-
const WALLET_PRIVATE_KEY = env.WALLET_PRIVATE_KEY;
8-
91
export const isValidHttpUrl = (urlString: string): boolean => {
102
let url;
113

server/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ const main = async () => {
2020

2121
try {
2222
// Check for the Tables Existence post startup
23-
// await implementTriggerOnStartUp(server);
2423
await startTxUpdatesNotificationListener(server);
2524
//check walletType and make sure i got all the access i need
2625
} catch (err) {

test/e2e/setup-init.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,4 @@ import createServer from "../../server/helpers/server";
22

33
const server = await createServer("Test-Suite");
44
server.log.info("Setting up required Tables, Triggers and Functions");
5-
6-
// await implementTriggerOnStartUp(server);
75
server.log.info("Server setup done");

0 commit comments

Comments
 (0)