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

MongoDB options never parse in connectionUrl and after my fix was parse incorrect #7437

Closed
2 of 21 tasks
gogotaro opened this issue Feb 28, 2021 · 0 comments
Closed
2 of 21 tasks

Comments

@gogotaro
Copy link
Contributor

gogotaro commented Feb 28, 2021

Issue Description

My understanding original of "parseConnectionUrl" method never parse options from connectionUrl as below.

return obeject at parseConnectionUrl method (original)

return {
            type: type,
            host: host,
            username: decodeURIComponent(username),
            password: decodeURIComponent(password),
            port: port ? parseInt(port) : undefined,
            database: afterBase || undefined
        };

But I try to use replicaSet option in connectionUrl and I ever fix issue #7401 with PR #7402 by separate method "parseMongoDBConnectionUrl" in class "DriverUtils" to parse connection mongodb only as below.

return obeject at parseMongoDBConnectionUrl method (my fixed #7402)

return {
            type: type,
            host: host,
            hostReplicaSet: hostReplicaSet,
            username: decodeURIComponent(username),
            password: decodeURIComponent(password),
            port: port ? parseInt(port) : undefined,
            database: afterBase || undefined,
            replicaSet: replicaSet || undefined
        };

It's working with option replicaSet only but another options wasn't working and parse incorrect.

PS. I know we can specified option key in connection object like { url: "mongodb://test-primary.example.com:27017/testdb", retryWrites: true, w: "majority", useUnifiedTopology: true} but some use case is inconvenience to do it.

Expected Behavior

My example connection string

mongodb://test-primary.example.com:27017/testdb?retryWrites=true&w=majority&useUnifiedTopology=true

I test by console.log(connectionString) at method "buildConnectionUrl" and console.log(mongoOptions) at method "buildConnectionOptions" in class "MongoDriver" and when connect was process result must be.

mongodb://test-primary.example.com:27017/testdb
{ retryWrites: 'true', w: 'majority', retryWrites: 'true' }

Actual Behavior

mongodb://test-primary.example.com:27017/testdb
{ replicaSet: 'true&w' }

As above my fix #7402 was make It parse incorrect. (Because I focus to parse replicaSet only, Sorry I'll fix It as fast as possible)

Steps to Reproduce

  1. Create connection typescript source code
createConnection({
    type: "mongodb",
    url: "mongodb://test-primary.example.com:27017/testdb?retryWrites=true&w=majority&useUnifiedTopology=true"
}).then(connection => { 

   console.log("TypeORM with MongoDB"); 
}).catch(error => console.log(error));
  1. console.log(connectionString) at method "buildConnectionUrl" and console.log(mongoOptions) at method "buildConnectionOptions" in class "MongoDriver"
  2. run these connection code.

My Environment

Dependency Version
Operating System Ubuntu 20.04
Node.js version v14.16.0
Typescript version ^v4.2.2
TypeORM version v0.2.31

Relevant Database Driver(s)

  • aurora-data-api
  • aurora-data-api-pg
  • better-sqlite3
  • cockroachdb
  • cordova
  • expo
  • mongodb
  • mysql
  • nativescript
  • oracle
  • postgres
  • react-native
  • sap
  • sqlite
  • sqlite-abstract
  • sqljs
  • sqlserver

Are you willing to resolve this issue by submitting a Pull Request?

  • Yes, I have the time, and I know how to start.
  • Yes, I have the time, but I don't know how to start. I would need guidance.
  • No, I don't have the time, although I believe I could do it if I had the time...
  • No, I don't have the time and I wouldn't even know how to start.
@gogotaro gogotaro changed the title MongoDB options never parse in connectionUrl and after my fix was parse false MongoDB options never parse in connectionUrl and after my fix was parse incorrect Feb 28, 2021
gogotaro added a commit to flowaccount/typeorm that referenced this issue Mar 17, 2021
* docs: fix small typo on package.json script example (typeorm#7408)

Add missing colon in JSON property at `package.json` `"script"` example

* feat: output Javascript Migrations instead of TypeScript (typeorm#7294)

* docs / test: Added tests and documentation for Feature 7253 - Migrations Javascript output

* Change in the test

* test: Re-arranged the tests to move them to the core tests directory

* tests: Adjusted Tests a bit

* tests - renamed tests to follow the other functional tests naming

* tests - renamed tests to follow the other functional tests naming

* tests - Fixed issues with the test connections setup

* tests - Removed unnecesary restore

* fix: improve EntityManager.save() return type (typeorm#7391)

This brings it in line with the equivalent method in Repository.

* fix: resolve issue building tree entities with embeded primary column (typeorm#7416)

Closes: typeorm#7415

* Adjust mongodb driver options & connect driver to support replica set (typeorm#7402)

- Dupplicate buildDriverOptions for mongodb especially
- Add hostReplicaSet to MongoConnectionOptions properties for collect host replica list
- Adjust buildConnectionUrl to build replica set connection url

* fix: performance issues of `RelationId`. (typeorm#7318)

* test: relationId is too slow

* perf: RelationId is too slow

When we join a lot of relations we can get 1k+ records from db even it is only 10 entities. Then when relationId are loaded the query contains too many duplciates for the same ids. The solution is to check that the relationId query fetches only unique values.

Closes: typeorm#5691

* fix: Array type default value should not generate SQL commands without change (typeorm#7409)

* fix(1532) Array type default value should not generate SQL commands without change

* Update PostgresDriver.ts

* removed `arrayCast` from `normalizeDefault` since casting for default value is already removed in `PostgresQueryRunner.loadTables()` method;
* removed support for function definition in `default` because function syntax suppose to support raw sql, we don't have to confuse things by applying custom modifications.

* Update User.ts

removed incorrect `default` definition with functions

Co-authored-by: AlexMesser <[email protected]>

* feat: add check and dryrun to migration generate (typeorm#7275)

Adds support for “check” and “drynrun” modes to the migration generate command.

Fixes typeorm#3037
Refs typeorm#6978

* chore: typescript version upgrade (typeorm#7422)

* chore: dependencies update (typeorm#7424)

* typescript version upgrade

* fixing linting

* fixing mongo query runner issues

* fixing linting

* updated all dependencies

* fixes typeorm#7418

* fixes typeorm#7418

* adding missing ILike operator docs (took from next branch)

* fix: mongodb connectionurl parse options (#1)

* fix: mongodb connectionurl parse options

- Loop every options in mongodb connection url and turn it as object to merge with connection url object before return of method "parseMongoDBConnectionUrl"
- unit test of mongodb replicaset parse connectionurl of typeorm#7401
- unit test of mongodb options parse connectionurl of typeorm#7437

* fix: add semicolon by lint suggestion

/home/circleci/typeorm/src/driver/DriverUtils.ts
  192:39  error  Missing semicolon  @typescript-eslint/semi

* chore: @beamdev package scope (#2)

* chore: update master (typeorm#3)

* fix: fixed all known enum issues (typeorm#7419)

* fix typeorm#5371

* fix typeorm#6471;
fix: `enumName` changes not handled;
fix: `enumName` does not handle table schema;

* fixed falling test;

* added test for typeorm#7217

* fix typeorm#6047, typeorm#7283;

* fix typeorm#5871

* added support for `enumName` in `joinColumns` (typeorm#5729)

* fix typeorm#5478

* fixed falling test;
updated `postgres-enum` test;

* added column `array` property change detection (typeorm#5882);
updated `postgres-enum` test;

* fix typeorm#5275

* added validation for `enum` property (typeorm#2233)

* fix typeorm#5648

* improved missing "enum" or "enumName" properties validation;

* fix typeorm#4897, typeorm#6376

* lint fix;

* fixed falling tests;

* fixed falling tests;

* removed .only

* fix typeorm#6115

* refactor: improve README.md and DEVLOPER.md code examples formatting (typeorm#7436)

* fix: correctly get referenceColumn value in `getEntityValueMap` (typeorm#7005)

* test: add test case (typeorm#7002)

* fix: correctly get referenceColumn value in `getEntityValueMap`

* test: reproduction for issue typeorm#3246 (typeorm#3247)

* Add reproduction for issue 3246

* Update test/github-issues/3246/issue-3246.ts

Co-authored-by: Json Choi <[email protected]>

Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: AlexMesser <[email protected]>
Co-authored-by: Json Choi <[email protected]>

* code refactoring in test;

* added test for typeorm#2758

* feat: allow to pass the given table name as string in RelationDecorators (typeorm#7448)

* feat(RelationDecorators): allow to pass the given table name as string

* Update EntityMetadataBuilder.ts

added parentheses;

Co-authored-by: Emily Marigold Klassen <[email protected]>

* feat: add option for installing package using CLI (typeorm#6889)

* init cli: add options for installing package

* yarg choice, add await, revert formatter changes

* init flag - set default to npm

Co-authored-by: AlexMesser <[email protected]>
Co-authored-by: Henry Boisdequin <[email protected]>
Co-authored-by: Json Choi <[email protected]>
Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: Emily Marigold Klassen <[email protected]>
Co-authored-by: Emily Marigold Klassen <[email protected]>
Co-authored-by: Gaurav Sharma <[email protected]>

* chore: update master typeorm/typeorm (typeorm#5)

* fix: fixed all known enum issues (typeorm#7419)

* fix typeorm#5371

* fix typeorm#6471;
fix: `enumName` changes not handled;
fix: `enumName` does not handle table schema;

* fixed falling test;

* added test for typeorm#7217

* fix typeorm#6047, typeorm#7283;

* fix typeorm#5871

* added support for `enumName` in `joinColumns` (typeorm#5729)

* fix typeorm#5478

* fixed falling test;
updated `postgres-enum` test;

* added column `array` property change detection (typeorm#5882);
updated `postgres-enum` test;

* fix typeorm#5275

* added validation for `enum` property (typeorm#2233)

* fix typeorm#5648

* improved missing "enum" or "enumName" properties validation;

* fix typeorm#4897, typeorm#6376

* lint fix;

* fixed falling tests;

* fixed falling tests;

* removed .only

* fix typeorm#6115

* refactor: improve README.md and DEVLOPER.md code examples formatting (typeorm#7436)

* fix: correctly get referenceColumn value in `getEntityValueMap` (typeorm#7005)

* test: add test case (typeorm#7002)

* fix: correctly get referenceColumn value in `getEntityValueMap`

* test: reproduction for issue typeorm#3246 (typeorm#3247)

* Add reproduction for issue 3246

* Update test/github-issues/3246/issue-3246.ts

Co-authored-by: Json Choi <[email protected]>

Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: AlexMesser <[email protected]>
Co-authored-by: Json Choi <[email protected]>

* code refactoring in test;

* added test for typeorm#2758

* feat: allow to pass the given table name as string in RelationDecorators (typeorm#7448)

* feat(RelationDecorators): allow to pass the given table name as string

* Update EntityMetadataBuilder.ts

added parentheses;

Co-authored-by: Emily Marigold Klassen <[email protected]>

* feat: add option for installing package using CLI (typeorm#6889)

* init cli: add options for installing package

* yarg choice, add await, revert formatter changes

* init flag - set default to npm

Co-authored-by: AlexMesser <[email protected]>
Co-authored-by: Henry Boisdequin <[email protected]>
Co-authored-by: Json Choi <[email protected]>
Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: Emily Marigold Klassen <[email protected]>
Co-authored-by: Emily Marigold Klassen <[email protected]>
Co-authored-by: Gaurav Sharma <[email protected]>

Co-authored-by: rccursach <[email protected]>
Co-authored-by: Jorge Luis Vargas <[email protected]>
Co-authored-by: Anthony Rosequist <[email protected]>
Co-authored-by: Tomas Zaluckij <[email protected]>
Co-authored-by: MG <[email protected]>
Co-authored-by: Ed Mitchell <[email protected]>
Co-authored-by: AlexMesser <[email protected]>
Co-authored-by: Christian Holm <[email protected]>
Co-authored-by: Umed Khudoiberdiev <[email protected]>
Co-authored-by: Henry Boisdequin <[email protected]>
Co-authored-by: Json Choi <[email protected]>
Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: Dan Imbrogno <[email protected]>
Co-authored-by: Emily Marigold Klassen <[email protected]>
Co-authored-by: Emily Marigold Klassen <[email protected]>
Co-authored-by: Gaurav Sharma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant