Skip to content

Commit

Permalink
Ecarton/cumulus 3928 imf work (#3831) (#3842)
Browse files Browse the repository at this point in the history
* guarantee non-numeric nonNumericString

* bringing in pg8.13 snyk suggestion

* trying to kcik up the sync-granules task error

* trying to get repeatable tries to sync-granules

* focus on just the important part in syncgranule

* trying to get publish to work

* does only this need to change to import?

* move to import function import

* reverting bad code for testing reasons

* changelog

* keep string length the same, no reason to twiddle this

Co-authored-by: etcart <[email protected]>
  • Loading branch information
Jkovarik and etcart authored Oct 28, 2024
1 parent 084f17b commit e81ff58
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 4 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,11 @@ included in the future will have a corresponding CHANGELOG entry in future relea

### Changed

- **CUMULUS-3928**
- updated publish scripting to use [email protected] for user email
- updated publish scripting to use esm over common import of latest-version
- updated bigint testing to remove intermitted failure source.
- updated postgres dependency version
- **CUMULUS-3838**
- Updated python dependencies to latest:
- cumulus-process-py 1.4.0
Expand Down
2 changes: 1 addition & 1 deletion lambdas/db-migration/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,6 @@
"dependencies": {
"@cumulus/db": "18.5.1",
"knex": "2.4.1",
"pg": "~8.10"
"pg": "~8.13"
}
}
2 changes: 1 addition & 1 deletion lambdas/db-provision-user-database/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
"@cumulus/common": "18.5.1",
"@cumulus/db": "18.5.1",
"knex": "2.4.1",
"pg": "~8.10"
"pg": "~8.13"
},
"devDependencies": {
"@types/pg": "^7.14.4"
Expand Down
2 changes: 1 addition & 1 deletion packages/db/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
"is-valid-hostname": "1.0.2",
"knex": "2.4.1",
"lodash": "^4.17.21",
"pg": "~8.10",
"pg": "~8.13",
"snake-camel": "^1.0.6",
"uuid": "8.3.2"
},
Expand Down
2 changes: 1 addition & 1 deletion packages/db/tests/test-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -442,7 +442,7 @@ test('canSafelyConvertBigInt() throws exception if number exceeds safe range', (
});

test('canSafelyConvertBigInt() throws exception for non-numeric string', (t) => {
const nonNumericString = cryptoRandomString({ length: 10 });
const nonNumericString = `a${cryptoRandomString({ length: 9 })}`;
t.throws(
() => canSafelyConvertBigInt(nonNumericString),
{
Expand Down

0 comments on commit e81ff58

Please sign in to comment.