Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

createConfiguration adds a colon to the servers hostname #1980

Closed
kraenhansen opened this issue Aug 21, 2018 · 0 comments
Closed

createConfiguration adds a colon to the servers hostname #1980

kraenhansen opened this issue Aug 21, 2018 · 0 comments
Assignees
Labels

Comments

@kraenhansen
Copy link
Member

Goals

A more predictable behaviour from the createConfiguration method.

Expected Results

I would expect the URL to not contain a : if no port was provided in the URL used for authentication.

Actual Results

When calling createConfiguration on a User that was authenticated against a server without a port (such as https://whatever.cloud.realm.io in this example) a colon (:) is appended to the hostname of the sync.url:

url: 'realms://whatever.cloud.realm.io:/default',
                                     ☝️😕

Steps to Reproduce

  1. Authenticate a user against a server with the default HTTPs port not specified.
  2. Call user.createConfiguration() and log the result.

Version of Realm and Tooling

  • Realm JS SDK Version: 2.14.2
  • Node or React Native: Node
  • Client OS & Version: N/A
  • Which debugger for React Native: N/A
@kraenhansen kraenhansen changed the title createConfiguration could use the path in the sync url createConfiguration adds a colon to the servers hostname Aug 21, 2018
@kneth kneth self-assigned this Aug 23, 2018
@kneth kneth removed their assignment Sep 6, 2018
@cmelchior cmelchior self-assigned this Nov 27, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants