Passing findOneOrFail(undefined) will not throw an error, it returns the current table's first entry.

[paroczigergo]

Issue type:

[x] bug report
[ ] feature request
[ ] documentation issue

Database system/driver:

[ ] cordova
[ ] mongodb
[ ] mssql
[ ] mysql / mariadb
[ ] oracle
[x] postgres
[ ] cockroachdb
[ ] sqlite
[ ] sqljs
[ ] react-native
[ ] expo

TypeORM version:

[] latest
[ ] @next
[ x] 0.2.25

Steps to reproduce or a small repository showing the problem:

Passing undefined value as id will not throw an error, it returns the current table's first entry.
repository.findOneOrFail(id)

Expectation:
It should throws an exception, and should not return db value.

[imnotjames]

This is a breaking change which is ideally gonna happen in the next minor release

[imnotjames]

Duplicate of #2990

[wanghq]

Seems this problem still exists. Can anyone confirm?

[ashrafatef]

any updates because it still exist