Skip to content

set to stage#1434

Open
ArnabChatterjee20k wants to merge 1 commit intomasterfrom
test-old-sdk
Open

set to stage#1434
ArnabChatterjee20k wants to merge 1 commit intomasterfrom
test-old-sdk

Conversation

@ArnabChatterjee20k
Copy link
Copy Markdown
Member

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 9, 2026

Greptile Summary

This PR replaces the production WebSocket realtime endpoint (wss://cloud.appwrite.io/v1) with a staging URL (wss://fra.stage.cloud.appwrite.io/v1) across all four language test files. This appears to be an accidental commit of local development/staging configuration that should not be merged.

  • All four test files (Android, Apple, Flutter, Web) have the same staging URL substitution that must be reverted before merging.

Confidence Score: 1/5

Not safe to merge — all changes are accidental staging configuration that must be reverted.

Every changed line in this PR replaces a production endpoint with a staging URL in test files. This will break CI for other contributors whenever the staging server is unavailable and leaks an internal hostname into the public repo.

All four files: tests/languages/android/Tests.kt, tests/languages/apple/Tests.swift, tests/languages/flutter/tests.dart, tests/languages/web/index.html

Vulnerabilities

No security vulnerabilities introduced. The staging endpoint substitution is an accidental configuration leak rather than a security concern, though it does expose an internal staging hostname in a public repository.

Important Files Changed

Filename Overview
tests/languages/android/Tests.kt Realtime endpoint changed from production to staging URL — should not be merged as-is.
tests/languages/apple/Tests.swift Realtime endpoint changed from production to staging URL — same issue as Android test.
tests/languages/flutter/tests.dart Realtime endpoint changed from production to staging URL — same issue across all test files.
tests/languages/web/index.html Realtime endpoint changed from production to staging URL — same issue across all test files.

Reviews (1): Last reviewed commit: "set to stage" | Re-trigger Greptile

// reset configs
client.setProject("console")
.setEndpointRealtime("wss://cloud.appwrite.io/v1")
.setEndpointRealtime("wss://fra.stage.cloud.appwrite.io/v1")
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Staging URL committed to test file

This replaces the production endpoint with a staging URL (wss://fra.stage.cloud.appwrite.io/v1). Test files in this repo should point to the canonical production endpoint; the staging server may be unavailable in CI, causing non-deterministic test failures for all contributors.

Suggested change
.setEndpointRealtime("wss://fra.stage.cloud.appwrite.io/v1")
.setEndpointRealtime("wss://cloud.appwrite.io/v1")

@@ -34,7 +34,7 @@ class Tests: XCTestCase {

// reset configs
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Staging URL committed to test file

Same staging endpoint replacement as in the Android test. This should remain pointed at the production URL.

Suggested change
// reset configs
client.setEndpointRealtime("wss://cloud.appwrite.io/v1")

client.setSelfSigned();
client.setProject('console');
client.setEndPointRealtime("wss://cloud.appwrite.io/v1");
client.setEndPointRealtime("wss://fra.stage.cloud.appwrite.io/v1");
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Staging URL committed to test file

Same staging endpoint replacement. Revert to the production URL.

Suggested change
client.setEndPointRealtime("wss://fra.stage.cloud.appwrite.io/v1");
client.setEndPointRealtime("wss://cloud.appwrite.io/v1");

// Realtime setup
client.setProject('console');
client.setEndpointRealtime('wss://cloud.appwrite.io/v1');
client.setEndpointRealtime('wss://fra.stage.cloud.appwrite.io/v1');
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Staging URL committed to test file

Same staging endpoint replacement. Revert to the production URL.

Suggested change
client.setEndpointRealtime('wss://fra.stage.cloud.appwrite.io/v1');
client.setEndpointRealtime('wss://cloud.appwrite.io/v1');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant