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

fix audit #4014

Merged
merged 1 commit into from
Dec 9, 2024
Merged

fix audit #4014

merged 1 commit into from
Dec 9, 2024

Conversation

yihau
Copy link
Member

@yihau yihau commented Dec 9, 2024

Problem

the audit failed. we need to upgrade idna to >= 1.0.0. however, the dep chain looks like:

jsonrpc-core-client v18.0.0 -> jsonrpc-client-transports v18.0.0 -> url v1.7.2 -> idna v0.1.5

and jsonrpc-core-client has not released any new version in the past 3y ...

Summary of Changes

ignore it as a stopgap

@yihau yihau requested review from t-nelson and bw-solana December 9, 2024 14:24
@yihau yihau added v2.0 Backport to v2.0 branch v2.1 Backport to v2.1 branch labels Dec 9, 2024
Copy link

mergify bot commented Dec 9, 2024

Backports to the stable branch are to be avoided unless absolutely necessary for fixing bugs, security issues, and perf regressions. Changes intended for backport should be structured such that a minimum effective diff can be committed separately from any refactoring, plumbing, cleanup, etc that are not strictly necessary to achieve the goal. Any of the latter should go only into master and ride the normal stabilization schedule.

Copy link

mergify bot commented Dec 9, 2024

Backports to the beta branch are to be avoided unless absolutely necessary for fixing bugs, security issues, and perf regressions. Changes intended for backport should be structured such that a minimum effective diff can be committed separately from any refactoring, plumbing, cleanup, etc that are not strictly necessary to achieve the goal. Any of the latter should go only into master and ride the normal stabilization schedule. Exceptions include CI/metrics changes, CLI improvements and documentation updates on a case by case basis.

joncinque added a commit to joncinque/solana-program-library that referenced this pull request Dec 9, 2024
#### Problem

Audit is failing due to an issue on idna.

#### Summary of changes

Since the dependency is pretty buried, do the same thing as
anza-xyz/agave#4014 and ignore it.
joncinque added a commit to solana-labs/solana-program-library that referenced this pull request Dec 9, 2024
#### Problem

Audit is failing due to an issue on idna.

#### Summary of changes

Since the dependency is pretty buried, do the same thing as
anza-xyz/agave#4014 and ignore it.
@pgarg66
Copy link

pgarg66 commented Dec 9, 2024

CI is broken due to this. Can we approve/merge this PR?

@joncinque joncinque merged commit 6c86ce5 into anza-xyz:master Dec 9, 2024
22 checks passed
mergify bot pushed a commit that referenced this pull request Dec 9, 2024
(cherry picked from commit 6c86ce5)

# Conflicts:
#	ci/do-audit.sh
mergify bot pushed a commit that referenced this pull request Dec 9, 2024
(cherry picked from commit 6c86ce5)
yihau added a commit that referenced this pull request Dec 10, 2024
(cherry picked from commit 6c86ce5)

# Conflicts:
#	ci/do-audit.sh
yihau added a commit that referenced this pull request Dec 10, 2024
(cherry picked from commit 6c86ce5)
behzadnouri pushed a commit that referenced this pull request Dec 11, 2024
(cherry picked from commit 6c86ce5)

# Conflicts:
#	ci/do-audit.sh
t-nelson pushed a commit that referenced this pull request Dec 11, 2024
* fix audit (#4014)

(cherry picked from commit 6c86ce5)

# Conflicts:
#	ci/do-audit.sh

* Fix merge conflict

---------

Co-authored-by: Yihau Chen <yihau.chen@icloud.com>
Co-authored-by: Jon C <me@jonc.dev>
t-nelson pushed a commit that referenced this pull request Dec 11, 2024
fix audit (#4014)

(cherry picked from commit 6c86ce5)

Co-authored-by: Yihau Chen <yihau.chen@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2.0 Backport to v2.0 branch v2.1 Backport to v2.1 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants