Skip to content

Conversation

m-hulbert
Copy link
Contributor

Description

This PR updates the listed endpoints from ably.io to the new formats;

  • realtime.ably.io and rest.ably.io consolidated to main.realtime.ably.net
  • mqtt.ably.io to main.mqtt.ably.net
  • pusher.ably.io and realtime-pusher.ably.io to main.pusher.ably.net
  • pubnub.ably.io to main.pubnub.ably.net

Checklist

@m-hulbert m-hulbert self-assigned this Oct 2, 2025
Copy link

coderabbitai bot commented Oct 2, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch edu-2104-update-endpoints

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@SimonWoolf SimonWoolf left a comment

Choose a reason for hiding this comment

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

no objection from me but not approving as I'll leave it to Laura to confirm the endpoints are right from an infra pov

<Code>
```shell
curl -X POST "https://sandbox-rest.ably.io/channels/my-channel/objects" \
curl -X POST "https://sandbox-main.realtime.ably.net/channels/my-channel/objects" \
Copy link

Choose a reason for hiding this comment

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

Sandbox uses a nonprod domain:

Suggested change
curl -X POST "https://sandbox-main.realtime.ably.net/channels/my-channel/objects" \
curl -X POST "https://sandbox.realtime.ably-nonprod.net/channels/my-channel/objects" \

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah good spot! I think this should actually have been pointing to main originally. eccdd53

Copy link

@surminus surminus left a comment

Choose a reason for hiding this comment

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

Just one correction related to sandbox but otherwise looks good

@m-hulbert m-hulbert requested a review from surminus October 3, 2025 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants