Merge branch 'main' into 4630-review-the-200-and-400-http-code

Author: jasuwienas

.github/workflows/postman.yml

@@ -1,11 +1,14 @@
 name: Postman Endpoint Tests
 
+# Disabled newman temporarily
 on:
-  pull_request:
-    branches: [ main, release/** ]
-  push:
-    branches: [ main, release/** ]
-    tags: [ v* ]
+  workflow_dispatch: # Manual trigger only
+# Disabled triggers:
+#  pull_request:
+#    branches: [ main, release/** ]
+#  push:
+#    branches: [ main, release/** ]
+#    tags: [ v* ]
 
 concurrency:
   group: pr-checks-${{ github.workflow }}-${{ github.head_ref || github.run_id }}
@@ -59,8 +62,14 @@ jobs:
       - name: Start the local relay
         run: npm run start &
 
-      - name: Install newman
-        run: npm install -g newman
+      # - name: Install newman
+      #   run: npm install -g newman
 
-      - name: Run the newman script
-        run: newman run packages/server/tests/postman.json
+      # - name: Run the newman script
+      #   run: newman run packages/server/tests/postman.json
+
+      - name: Security Warning
+        run: |
+          echo "CRITICAL: This workflow is disabled due to Newman security compromise"
+          echo "DO NOT re-enable until Newman is replaced with a secure alternative"
+          exit 1

README.md

@@ -50,24 +50,6 @@ run configuration. You should be able to just run that configuration, and it sho
 
 - It is highly recommended to read the [Testing Guide](docs/testing-guide.md) for detailed testing strategies and best practices.
 
-### Postman
-
-First ensure newman is installed locally using `npm`, then execute `newman`.
-
-```shell
-npm install -g newman
-newman run packages/server/tests/postman.json --env-var baseUrl=http://localhost:7546
-```
-
-To enable Postman test to run via helm deployment add
-
-```
-test:
-  enabled: true
-  schedule: '@daily' #How often to run the Postman test
-  baseUrl: "http://127.0.0.1:7546" # Relay URL to run the test against
-```
-
 ### Acceptance Tests
 
 The relay has a suite of acceptance tests that may be run to confirm E2E operation of the relay in either a `hedera-local-node` or deployed env.

charts/hedera-json-rpc-relay/templates/cronjob.yaml

@@ -27,7 +27,8 @@ spec:
 {{- end }}
 
 ---
-{{- if .Values.cronjob.enabled }}
+# Disabled newman temporarily
+{{- if false }} # Force disabled - was: .Values.cronjob.enabled
 apiVersion: batch/v1
 kind: CronJob
 metadata:

charts/hedera-json-rpc-relay/templates/tests/pod.yaml

@@ -1,4 +1,5 @@
-{{- if .Values.test.enabled -}}
+# Disabled newman temporarily
+{{- if false }} # Force disabled - was: .Values.test.enabled
 apiVersion: v1
 kind: Pod
 metadata:

charts/hedera-json-rpc-relay/values.yaml

@@ -138,6 +138,7 @@ redis:
   # REDIS_URL=redis://<release-name>-redis-master:6379
   autoconfig: true
 
+# Disabled newman temporarily
 cronjob:
   enabled: false
   image:
@@ -245,8 +246,9 @@ readinessProbe:
   initialDelaySeconds: 20
   timeoutSeconds: 5
 
+# Disabled newman temporarily
 test:
-  enabled: true
+  enabled: false
   image:
     pullPolicy: IfNotPresent
     repository: postman/newman