Port # is used when srv is true

[Daxcor69]

When SRV = true, the port designation should be disabled. If I leave port out of the config I get 27017, if I put undefined it puts the colon and no port number. If I put false then it puts :false.

Here is my options config:

{
  host: 'cluster0.sdfddqmlkssffdsdsn.mongodb.net',
  replicaSetName: 'atlas-7ksdwebssdddhrhalbt-shard-0',
  srv: true,
  username: 'blah',
  password: 'blah',
  outDir: '/backup/2023-2-8-18-23-33',
  authenticationDatabase: 'found',
  authenticationMechanism: 'SCRAM-SHA-1',
  ssl: true,
  sslAllowInvalidCertificates: true,
  all: true
}

Here is the error:

 uri: 'mongodb+srv://blah:blah@cluster0.kysfkiehfkdikdkdkdkfnkhzn.mongodb.net:27017/?authSource=found&authMechanism=SCRAM-SHA-1',
    options: {
      replicaSet: 'atlas-7kadfkjfsjfsueyuhgmbvkfhgidydoswlgulbt-shard-0',
      ssl: true,
      tlsAllowInvalidCertificates: true
    },
    triggerError: MongoParseError: mongodb+srv URI cannot have port number

[Daxcor69]

This example doesn't make sense

This will add srv (mongodb+srv//myhost:8080):
const { mongoExport } = require('mongoback');
const options = { 
    host: 'myhost',
    port: 8080,
    srv: true,
    all: true 
};

[euberdeveloper]

I am taking a look into that, thanks for the issue

[Daxcor69]

I don't know typescript, but I would think a conditional here:

function getMongoConnectionUri(options: ConnectionOptions): string {
    let uri = options.uri;

    if (!uri) {
        const protocol = options.srv ? 'mongodb+srv' : 'mongodb';
        const host = Array.isArray(options.host)
            ? `${options.host.map(({ host, port }) => `${host}:${port}`, '').join(',')}`
            : `${options.host ?? ''}:${options.port ?? ''}`;
        const auth = options.username
            ? options.password
                ? `${options.username}:${options.password}@`
                : `${options.username}@`
            : '';
        const opt = getOptionsString(options);
        uri = `${protocol}://${auth}${host}${opt}`;
    }

    return uri;
}

to create a different protocol and host based on srv

[euberdeveloper]

Exactly, I did some commits referring this issue, this is the last one. Now I am fixing the github action that runs the tests, after that I can publish the new version and close the issue

[Daxcor69]

wow thank you for doing this right away, that is really cool. thank you for the project it is really helpful.

[euberdeveloper]

Thanks! I was chilling right now, so I saw the notificaiton email, so no problem. If you like the project feel free to add a star

[Daxcor69]

Hello,

I think there is still an issue with SRV and ports. Here is the output of the backup. You will see that it does indeed connect to Mongo Atlas service. Gets the first collection to export and the spinner goes and then after a timeout it errors out with an dns/tcp lookup. If you look it is still showing a port number.

foundry
  ⣯ products2023-02-09T20:39:24.806+0000        could not connect to server: server selection error: server selection timeout, current topology: { Type: Single, Servers: [{ Addr: cluster0.<redacted>.mongodb.net:27017, Type: Unknown, Last error: connection() error occurred during connection handshake: dial tcp: lookup cluster0.<redacted>.mongodb.net: No address associated with hostname }, ] }
  ✖ products

[euberdeveloper]

Do you have the Uri from the log? (Of course without pwd)

[Daxcor69]

not sure how to get that, but here is the options object

{
  host: 'cluster0.<redacted>.mongodb.net',
  replicaSetName: 'atlas-<redacted>-shard-0',
  srv: true,
  username: 'backup',
  password: '<redacted>',
  outDir: '/backup/2023-2-10-0-15-50',
  authenticationDatabase: 'found',
  authenticationMechanism: 'SCRAM-SHA-1',
  ssl: true,
  tlsInsecure: true,
  all: true,
  outType: 'flat',
  readPreference: 'primary',
  realtimeLog: true
}

Because it does connect on the initial part, it does NOT throw an error with the uri. Only after it times out on the export of the collection does it throw the error. Is there a way to print the fully constructed uri ?

[euberdeveloper]

You are right, the part regarding the first connection is different from the one in which the collections are exported (this)