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

upgrade dependencies to avoid a high #147

Merged
merged 56 commits into from
Dec 5, 2024
Merged

upgrade dependencies to avoid a high #147

merged 56 commits into from
Dec 5, 2024

Conversation

konnov
Copy link
Contributor

@konnov konnov commented Nov 25, 2024

npm reported a high vulnerability in our dependencies. Upgrading them.

@konnov konnov requested a review from thpani November 25, 2024 12:27
@konnov
Copy link
Contributor Author

konnov commented Nov 25, 2024

@thpani It looks like we have to upload the contracts again?

@konnov konnov mentioned this pull request Nov 25, 2024
@thpani
Copy link
Collaborator

thpani commented Nov 25, 2024

Very likely, yes.
I automated deploying and populating for the TS monitor E2E tests. But I guess a one-off running the script is easier here.

@konnov
Copy link
Contributor Author

konnov commented Nov 29, 2024

Very likely, yes. I automated deploying and populating for the TS monitor E2E tests. But I guess a one-off running the script is easier here.

I am trying to repopulate the contract via ./scripts/setter-populate.sh. It fails with an error:

❌ error: transaction submission failed: Some(
    TransactionResult {
        fee_charged: 70341,
        result: TxFailed(
            VecM(
                [
                    OpInner(
                        InvokeHostFunction(
                            ResourceLimitExceeded,
                        ),
                    ),
                ],
            ),
        ),
        ext: V0,
    },
)

I guess, this is not expected?

@konnov konnov merged commit 6d2ea45 into main Dec 5, 2024
3 checks passed
@konnov konnov deleted the igor/upgrade-deps branch December 5, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants