Adjust & improve

Author: akegaviar

.env.example

@@ -9,6 +9,7 @@ BASE_NODE_ENDPOINT_2=https://your-node-endpoint-2.com
 
 # Simple label for resulting filename
 REGION=singapore
+
 NUMBER_OF_TRANSACTIONS=10
 
 # Use eth_sendRawTransactionSync (true) or standard async method (false)

README.md

@@ -23,16 +23,17 @@ PRIVATE_KEY=your_private_key_here
 TO_ADDRESS=0xc2F695613de0885dA3bdd18E8c317B9fAf7d4eba
 
 # If you are going with the asynchronous method and poll for the transaction receipts separately
-POLLING_INTERVAL_MS=100
+POLLING_INTERVAL_MS=50
 
 BASE_NODE_ENDPOINT_1=https://your-node-endpoint-1.com
 BASE_NODE_ENDPOINT_2=https://your-node-endpoint-2.com
 
 # This a simple label for the resulting filename to remember where you sent the test transactions from. Not used in any node routing.
 REGION=singapore
+
 NUMBER_OF_TRANSACTIONS=10
 
-# With `true`, the synchronous eth_sendRawTransactionSync method is used. With `false`, the asynchronous method is used with POLLING_INTERVAL_MS=100
+# With `true`, the synchronous eth_sendRawTransactionSync method is used. With `false`, the asynchronous method is used with POLLING_INTERVAL_MS=50
 SEND_TXN_SYNC=true
 
 RUN_ENDPOINT2_TESTING=true
@@ -57,7 +58,7 @@ RUN_ENDPOINT2_TESTING=true
 **`POLLING_INTERVAL_MS`**: 
 - Used only when `SEND_TXN_SYNC=false`
 - Defines how frequently (in milliseconds) to check for transaction receipts
-- Default: 100ms
+- Default: 50ms (recommended for accurate latency measurements)
 - Lower values = more frequent polling = faster detection but higher load
 - Not used when `SEND_TXN_SYNC=true` since confirmations are immediate
 
@@ -81,6 +82,19 @@ docker build -t transaction-latency .
 docker run -v $(pwd)/data:/data --env-file .env --rm -it transaction-latency
 ```
 
+### Note about .env file loading
+
+When running with Docker, you may see the log message:
+```
+Error loading .env file
+```
+
+This is expected and harmless. The application uses two methods to load environment variables:
+1. **Docker's `--env-file` flag** (recommended): Sets environment variables at container runtime
+2. **Direct .env file loading**: Attempts to read the `.env` file from inside the container
+
+Since we use the `--env-file` approach for security (keeping secrets out of the image), the direct file loading fails but the environment variables are still available via Docker. The application continues normally and all configuration works as expected.
+
 ## Results
 
 After completion, results are saved to the `./data/` directory:

main.go

@@ -66,7 +66,7 @@ func main() {
 sendTxnSync := os.Getenv("SEND_TXN_SYNC") == "true"
 runEndpoint2Testing := os.Getenv("RUN_ENDPOINT2_TESTING") != "false"
 
-pollingIntervalMs := 100
+pollingIntervalMs := 50
 if pollingEnv := os.Getenv("POLLING_INTERVAL_MS"); pollingEnv != "" {
 if parsed, err := strconv.Atoi(pollingEnv); err == nil {
 pollingIntervalMs = parsed
@@ -137,10 +137,10 @@ func main() {
 time.Sleep(5 * time.Second)
 
 if runEndpoint2Testing {
-log.Printf("Starting endpoint2 transactions")
+log.Printf("Starting endpoint2 transactions, syncMode=%v", sendTxnSync)
 for i := 0; i < numberOfTransactions; i++ {
-// Use async mode for endpoint2 testing
-timing, err := timeTransaction(chainId, privateKey, fromAddress, toAddress, endpoint2Client, false, pollingIntervalMs)
+// Use the same mode as endpoint1 for fair comparison
+timing, err := timeTransaction(chainId, privateKey, fromAddress, toAddress, endpoint2Client, sendTxnSync, pollingIntervalMs)
 if err != nil {
 endpoint2Errors += 1
 log.Printf("Failed to send transaction: %v", err)
@@ -155,12 +155,12 @@ func main() {
 log.Printf("Skipping endpoint2 transactions (RUN_ENDPOINT2_TESTING=false)")
 }
 
-if err := writeToFile(fmt.Sprintf("./data/endpoint1-%s.csv", region), endpoint1Timings); err != nil {
+if err := writeToFile(fmt.Sprintf("/data/endpoint1-%s.csv", region), endpoint1Timings); err != nil {
 log.Fatalf("Failed to write to file: %v", err)
 }
 
 if runEndpoint2Testing {
-if err := writeToFile(fmt.Sprintf("./data/endpoint2-%s.csv", region), endpoint2Timings); err != nil {
+if err := writeToFile(fmt.Sprintf("/data/endpoint2-%s.csv", region), endpoint2Timings); err != nil {
 log.Fatalf("Failed to write to file: %v", err)
 }
 }