Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 20, 2025

Bumps golang.org/x/crypto from 0.40.0 to 0.45.0.

Commits
  • 4e0068c go.mod: update golang.org/x dependencies
  • e79546e ssh: curb GSSAPI DoS risk by limiting number of specified OIDs
  • f91f7a7 ssh/agent: prevent panic on malformed constraint
  • 2df4153 acme/autocert: let automatic renewal work with short lifetime certs
  • bcf6a84 acme: pass context to request
  • b4f2b62 ssh: fix error message on unsupported cipher
  • 79ec3a5 ssh: allow to bind to a hostname in remote forwarding
  • 122a78f go.mod: update golang.org/x dependencies
  • c0531f9 all: eliminate vet diagnostics
  • 0997000 all: fix some comments
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

@dependabot dependabot bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Nov 20, 2025
@sofuture sofuture force-pushed the dependabot/go_modules/golang.org/x/crypto-0.45.0 branch from b004b7d to 78dc9ce Compare December 17, 2025 15:04
if err != nil {
t.Fatal(err)
}
t.Chdir(cwd)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Calling t.Chdir inside cleanup causes wrong directory state

The call to t.Chdir(cwd) inside the t.Cleanup callback is problematic. Since t.Chdir at line 72 already registers an automatic cleanup to restore the original directory, calling t.Chdir again inside the cleanup function registers another cleanup that runs after all current cleanups finish. Due to LIFO cleanup ordering, this results in the working directory being left at "fixtures/loadEnvFiles" instead of the original directory after the test completes, potentially affecting subsequent tests.

Fix in Cursor Fix in Web

@sofuture sofuture force-pushed the dependabot/go_modules/golang.org/x/crypto-0.45.0 branch from 5a18a04 to 946e889 Compare December 17, 2025 16:40
@sofuture sofuture force-pushed the dependabot/go_modules/golang.org/x/crypto-0.45.0 branch from 946e889 to 838fe5a Compare January 8, 2026 21:58
dependabot bot and others added 2 commits January 8, 2026 16:22
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.40.0 to 0.45.0.
- [Commits](golang/crypto@v0.40.0...v0.45.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-version: 0.45.0
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@sofuture sofuture force-pushed the dependabot/go_modules/golang.org/x/crypto-0.45.0 branch from 838fe5a to a9a9d4a Compare January 8, 2026 23:22
@amacneil
Copy link
Owner

amacneil commented Jan 8, 2026

@dependabot rebase

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Jan 8, 2026

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

@amacneil amacneil merged commit 2b74bc5 into main Jan 8, 2026
11 checks passed
@amacneil amacneil deleted the dependabot/go_modules/golang.org/x/crypto-0.45.0 branch January 8, 2026 23:32
amacneil added a commit that referenced this pull request Jan 8, 2026
Remove redundant `t.Chdir(cwd)` call and add comments to clarify
directory cleanup in `TestLoadEnvFiles`.

Identified by
#709 (comment)
although that bugbot comment was incorrect.

The `t.Chdir` function automatically registers a cleanup to restore the
original directory, making the explicit `t.Chdir(cwd)` call in the
`t.Cleanup` block redundant. This change simplifies the test setup and
clarifies that `t.Chdir` handles its own directory restoration.

---
<a
href="https://cursor.com/background-agent?bcId=bc-43db5ef2-82e0-4c1a-b568-b39d4f0df70e"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/open-in-cursor-dark.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in
Cursor"
src="https://cursor.com/open-in-cursor.svg"></picture></a>&nbsp;<a
href="https://cursor.com/agents?id=bc-43db5ef2-82e0-4c1a-b568-b39d4f0df70e"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/open-in-web-dark.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web"
src="https://cursor.com/open-in-web.svg"></picture></a>

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants