Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: golang/crypto
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.17.0
Choose a base ref
...
head repository: golang/crypto
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.22.0
Choose a head ref
  • 18 commits
  • 26 files changed
  • 10 contributors

Commits on Jan 2, 2024

  1. internal/poly1305: drop Go 1.12 compatibility

    Other packages already dropped compatibility with go1.12, so it should be safe to remove it for this package as well.
    
    Change-Id: Ieecc7cd06a0a4e69e8c1c09ef6fefe95d78ceb75
    GitHub-Last-Rev: 1971e03
    GitHub-Pull-Request: #239
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/448239
    Reviewed-by: Joedian Reid <joedian@google.com>
    Auto-Submit: Filippo Valsorda <filippo@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Filippo Valsorda <filippo@golang.org>
    Reviewed-by: Roland Shoemaker <roland@golang.org>
    Reviewed-by: Matthew Dempsky <mdempsky@google.com>
    thaJeztah authored and gopherbot committed Jan 2, 2024
    Configuration menu
    Copy the full SHA
    08396bb View commit details
    Browse the repository at this point in the history

Commits on Jan 4, 2024

  1. go.mod: update golang.org/x dependencies

    Update golang.org/x dependencies to their latest tagged versions.
    
    Change-Id: I2df91d0602cd1ef94370de4a60a935edeb441333
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/553996
    Reviewed-by: Than McIntosh <thanm@google.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
    Auto-Submit: Gopher Robot <gobot@golang.org>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    gopherbot committed Jan 4, 2024
    Configuration menu
    Copy the full SHA
    055043d View commit details
    Browse the repository at this point in the history

Commits on Jan 8, 2024

  1. ssh/test: avoid leaking a net.UnixConn in server.TryDialWithAddr

    For golang/go#64959.
    
    Change-Id: I2153166f4960058cdc2b82ae34ca250dcc6ba1c6
    Cq-Include-Trybots: luci.golang.try:x_crypto-gotip-linux-amd64-longtest,x_crypto-gotip-windows-amd64-longtest
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/554062
    Run-TryBot: Bryan Mills <bcmills@google.com>
    Auto-Submit: Bryan Mills <bcmills@google.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
    TryBot-Result: Gopher Robot <gobot@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    Bryan C. Mills authored and gopherbot committed Jan 8, 2024
    Configuration menu
    Copy the full SHA
    403f699 View commit details
    Browse the repository at this point in the history
  2. ssh/test: skip tests on darwin that fail on the darwin-amd64-longtest…

    … LUCI builder
    
    We don't yet understand why these tests fail, but the Apple sshd seems
    to have some non-trivial vendor patches, so it is plausibly a
    platform-specific bug in the test. Let's skip that failure mode on the
    whole platform until/unless someone has time to reproduce and
    investigate the failure.
    
    For golang/go#64959.
    
    Cq-Include-Trybots: luci.golang.try:x_crypto-gotip-darwin-amd64-longtest,x_crypto-gotip-linux-amd64-longtest,x_crypto-gotip-windows-amd64-longtest
    Change-Id: I9e43579469de3fe9329c093b5916bbed0edd3751
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/554077
    Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    Reviewed-by: Nicola Murino <nicola.murino@gmail.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Auto-Submit: Bryan Mills <bcmills@google.com>
    Bryan C. Mills authored and gopherbot committed Jan 8, 2024
    Configuration menu
    Copy the full SHA
    dbb6ec1 View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2024

  1. x509roots/fallback: update bundle

    This is an automated CL which updates the NSS root bundle.
    
    Change-Id: I6d9163026799e5d134f6bb6819e22448d7ebe719
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/561395
    Reviewed-by: Roland Shoemaker <roland@golang.org>
    Auto-Submit: Gopher Robot <gobot@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    gopherbot committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    913d3ae View commit details
    Browse the repository at this point in the history
  2. go.mod: update golang.org/x dependencies

    Update golang.org/x dependencies to their latest tagged versions.
    
    Change-Id: I87d766a485104662172bf983d2a9deb085eddd99
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/562417
    Reviewed-by: Than McIntosh <thanm@google.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    Auto-Submit: Gopher Robot <gobot@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: David Chase <drchase@google.com>
    gopherbot committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    405cb3b View commit details
    Browse the repository at this point in the history

Commits on Feb 8, 2024

  1. x509roots/nss: manually exclude a confusingly constrained root

    Fixes golang/go#61963
    
    Change-Id: I16920d160af74772ef5aa650d1274e07c3ca9adc
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/562475
    Reviewed-by: Filippo Valsorda <filippo@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    rolandshoemaker committed Feb 8, 2024
    Configuration menu
    Copy the full SHA
    62c9f17 View commit details
    Browse the repository at this point in the history

Commits on Feb 14, 2024

  1. ssh/test: don't use DSA keys in integrations tests, update test RSA key

    on RHEL 9 OpenSSH does not support DSA keys and RSA keys with size less
    than 2048 bits, furthermore signing with ssh-rsa (SHA-1 signatures) and
    ssh-dss is not allowed, therefore:
    
    1) replaced the 1024-bit RSA key used in the test with a new 2048-bit
       one
    2) removed DSA key from itegration tests
    3) allowed signature errors using ssh-rsa in agent integration tests, we
       also check SHA-2 variants that are not skipped
    
    Fixes golang/go#65581
    
    Change-Id: I54bf997b61ef4d91d38eb624275737ba7291bb20
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/562755
    Reviewed-by: Roland Shoemaker <roland@golang.org>
    Reviewed-by: David Chase <drchase@google.com>
    Reviewed-by: Filippo Valsorda <filippo@golang.org>
    Auto-Submit: Nicola Murino <nicola.murino@gmail.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    drakkan authored and gopherbot committed Feb 14, 2024
    Configuration menu
    Copy the full SHA
    1c981e6 View commit details
    Browse the repository at this point in the history

Commits on Feb 15, 2024

  1. x/crypto/internal/poly1305: improve sum_ppc64le.s

    This contains a few minor improvements to sum_ppc64le.s
    which result in up to 10% performance improvement for
    some of the benchmarks in this directory.
    
    - ADDZE followed by ADD can be combined into ADDE
    - PCALIGN added to the loop
    - Eliminate a few unnecessary register moves
    
    goos: linux
    goarch: ppc64le
    pkg: golang.org/x/crypto/internal/poly1305
    cpu: POWER10
                     │ poly.orig.out │              poly.out              │
                     │    sec/op     │   sec/op     vs base               │
    64                   40.34n ± 0%   38.13n ± 0%   -5.47% (p=0.002 n=6)
    1K                   482.2n ± 0%   444.6n ± 0%   -7.81% (p=0.002 n=6)
    2M                   978.4µ ± 0%   879.3µ ± 0%  -10.12% (p=0.002 n=6)
    64Unaligned          40.35n ± 0%   38.16n ± 0%   -5.42% (p=0.002 n=6)
    1KUnaligned          482.0n ± 0%   444.2n ± 0%   -7.84% (p=0.002 n=6)
    2MUnaligned          978.4µ ± 0%   879.4µ ± 0%  -10.12% (p=0.002 n=6)
    Write64              32.69n ± 0%   30.71n ± 0%   -6.04% (p=0.002 n=6)
    Write1K              472.4n ± 0%   436.5n ± 0%   -7.60% (p=0.002 n=6)
    Write2M              978.3µ ± 0%   879.4µ ± 0%  -10.11% (p=0.002 n=6)
    Write64Unaligned     32.67n ± 0%   30.71n ± 0%   -6.00% (p=0.002 n=6)
    Write1KUnaligned     472.6n ± 0%   436.4n ± 0%   -7.66% (p=0.002 n=6)
    Write2MUnaligned     978.5µ ± 0%   879.6µ ± 0%  -10.10% (p=0.002 n=6)
    geomean              2.569µ        2.367µ        -7.87%
    
    Change-Id: I63314e7252ef10fb2d157f623c4bc2e31a63ae32
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/558775
    Reviewed-by: David Chase <drchase@google.com>
    Reviewed-by: Michael Knyszek <mknyszek@google.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Paul Murphy <murp@ibm.com>
    Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com>
    TryBot-Result: Gopher Robot <gobot@golang.org>
    Reviewed-by: Than McIntosh <thanm@google.com>
    laboger committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    1a86580 View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2024

  1. ocsp: don't use iota for externally defined constants

    Style fix: iota shouldn't be used for values that come from RFCs or
    things that go over the wire or to disk.
    
    Change-Id: Ib903ec1bce7e71ff81094db3de8d0b71e16f52f3
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/564695
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    Auto-Submit: Roland Shoemaker <roland@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Roland Shoemaker <roland@golang.org>
    bradfitz authored and gopherbot committed Feb 18, 2024
    Configuration menu
    Copy the full SHA
    5bead59 View commit details
    Browse the repository at this point in the history

Commits on Feb 26, 2024

  1. all: update go.mod x/net dependency

    Not necessary for anything, but need to test something harmless.
    
    Change-Id: I4faa64ef2c8f1f14df5ccaa37836fc56beed4f50
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/566917
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Auto-Submit: Gopher Robot <gobot@golang.org>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    Reviewed-by: Roland Shoemaker <roland@golang.org>
    Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
    rolandshoemaker authored and gopherbot committed Feb 26, 2024
    Configuration menu
    Copy the full SHA
    0aab8d0 View commit details
    Browse the repository at this point in the history

Commits on Feb 28, 2024

  1. ssh/test: work around for TestCiphers failures on macOS

    TestCiphers is already skipped on macOS when testing data received
    from the server, so move the test for sending data after the
    receiving one to work around this new integration test failure.
    
    Fixes golang/go#65732
    
    Change-Id: Ie0c614c5373735ae8aefdd7ded643579b130f4b3
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/564457
    Reviewed-by: Bryan Mills <bcmills@google.com>
    Reviewed-by: Carlos Amedee <carlos@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Auto-Submit: Nicola Murino <nicola.murino@gmail.com>
    Commit-Queue: Nicola Murino <nicola.murino@gmail.com>
    drakkan authored and gopherbot committed Feb 28, 2024
    Configuration menu
    Copy the full SHA
    0d2316b View commit details
    Browse the repository at this point in the history

Commits on Mar 4, 2024

  1. go.mod: update golang.org/x dependencies

    Update golang.org/x dependencies to their latest tagged versions.
    
    Change-Id: I54256afe881714f60fc37c471cd65b060090d9d6
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/568816
    Reviewed-by: Than McIntosh <thanm@google.com>
    Reviewed-by: Michael Knyszek <mknyszek@google.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Auto-Submit: Gopher Robot <gobot@golang.org>
    gopherbot committed Mar 4, 2024
    Configuration menu
    Copy the full SHA
    7067223 View commit details
    Browse the repository at this point in the history

Commits on Mar 18, 2024

  1. all: remove redundant words in comments and fix some typos

    Change-Id: I3078492dc020770aca630e0b362df0212bd41e32
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/569156
    Auto-Submit: Ian Lance Taylor <iant@google.com>
    Reviewed-by: David Chase <drchase@google.com>
    Reviewed-by: Nicola Murino <nicola.murino@gmail.com>
    Reviewed-by: Ian Lance Taylor <iant@google.com>
    Auto-Submit: Nicola Murino <nicola.murino@gmail.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    cuishuang authored and gopherbot committed Mar 18, 2024
    Configuration menu
    Copy the full SHA
    b91329d View commit details
    Browse the repository at this point in the history

Commits on Mar 26, 2024

  1. x/crypto/chacha20: cleanup chacha_ppc64le.s

    - Adding PCALIGN before the loops
    - Changing WORD directive with corresponding Vector Merge EVEN/ODD word  instructions
    - Replacing Branch Conditional (BC) with its extended mnemonic form  BDNZ
    - VPERMXOR instruction usage in place of VXOR instructions followed by
      VRLW (rotate left) for cases of rotating in multiples of 8. This
      replacements give performace improvement both in time and space  of around 7%-8% as listed below
      using benchstat tool.
    
    goos: linux
    goarch: ppc64le
    pkg: golang.org/x/crypto/chacha20
    cpu: POWER10
                     | chacha20.prev.out |       chacha20.new.out            |
                     |      sec/op       |   sec/op     vs base              |
    ChaCha20/64              171.9n ± 0%   156.6n ± 1%  -8.90% (p=0.002 n=6)
    ChaCha20/256             165.5n ± 0%   152.4n ± 0%  -7.92% (p=0.002 n=6)
    ChaCha20/10x25           505.8n ± 0%   504.3n ± 2%  -0.32% (p=0.589 n=6)
    ChaCha20/4096            2.265µ ± 0%   2.052µ ± 0%  -9.40% (p=0.002 n=6)
    ChaCha20/100x40          5.359µ ± 3%   5.018µ ± 2%  -6.37% (p=0.002 n=6)
    ChaCha20/65536           35.71µ ± 0%   32.29µ ± 0%  -9.57% (p=0.002 n=6)
    ChaCha20/1000x65         44.63µ ± 0%   41.05µ ± 0%  -8.02% (p=0.002 n=6)
    geomean                  2.235µ        2.073µ       -7.26%
    
                     | chacha20.prev.out |          chacha20.new.out         |
                     |       B/s         |     B/s       vs base             |
    ChaCha20/64             355.1Mi ± 0%   389.8Mi ± 1%   +9.78% (p=0.002 n=6)
    ChaCha20/256            1.440Gi ± 0%   1.565Gi ± 0%   +8.62% (p=0.002 n=6)
    ChaCha20/10x25          471.3Mi ± 0%   472.8Mi ± 2%   +0.31% (p=0.589 n=6)
    ChaCha20/4096           1.684Gi ± 0%   1.859Gi ± 0%  +10.38% (p=0.002 n=6)
    ChaCha20/100x40         711.8Mi ± 3%   760.3Mi ± 2%   +6.80% (p=0.002 n=6)
    ChaCha20/65536          1.709Gi ± 0%   1.890Gi ± 0%  +10.59% (p=0.002 n=6)
    ChaCha20/1000x65        1.356Gi ± 0%   1.475Gi ± 0%   +8.72% (p=0.002 n=6)
    geomean                 957.3Mi        1.008Gi        +7.83%
    Change-Id: Ib31cb10a2a11eacdacf0272fbfd887eb5ccd8bcb
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/564797
    Reviewed-by: Lynn Boger <laboger@linux.vnet.ibm.com>
    Run-TryBot: Paul Murphy <murp@ibm.com>
    TryBot-Result: Gopher Robot <gobot@golang.org>
    Reviewed-by: David Chase <drchase@google.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com>
    Reviewed-by: Cherry Mui <cherryyz@google.com>
    jkrishmys authored and laboger committed Mar 26, 2024
    Configuration menu
    Copy the full SHA
    8d0d405 View commit details
    Browse the repository at this point in the history

Commits on Apr 3, 2024

  1. ssh: add server side multi-step authentication

    Add support for sending back partial success to the client while
    handling authentication in the server. This is implemented by a special
    error that can be returned by any of the authentication methods, which
    contains the authentication methods to offer next.
    
    This patch is based on CL 399075 with some minor changes and the
    addition of test cases.
    
    Fixes golang/go#17889
    Fixes golang/go#61447
    Fixes golang/go#64974
    
    Co-authored-by: Peter Verraedt <peter.verraedt@kuleuven.be>
    Change-Id: I05c8f913bb407d22c2e41c4cbe965e36ab4739b0
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/516355
    Reviewed-by: Andrew Lytvynov <awly@tailscale.com>
    Reviewed-by: Than McIntosh <thanm@google.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Filippo Valsorda <filippo@golang.org>
    Auto-Submit: Filippo Valsorda <filippo@golang.org>
    2 people authored and gopherbot committed Apr 3, 2024
    Configuration menu
    Copy the full SHA
    6f79b5a View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2024

  1. ssh: respect MaxAuthTries also for "none" auth attempts

    Only the first "none" auth attempt is allowed without penality
    
    Change-Id: Ibe776e968ba406445eeb94e8f1959383b88c98f7
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/575995
    Reviewed-by: Filippo Valsorda <filippo@golang.org>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    Auto-Submit: Nicola Murino <nicola.murino@gmail.com>
    Commit-Queue: Nicola Murino <nicola.murino@gmail.com>
    Reviewed-by: Than McIntosh <thanm@google.com>
    drakkan authored and gopherbot committed Apr 4, 2024
    Configuration menu
    Copy the full SHA
    b92bf94 View commit details
    Browse the repository at this point in the history
  2. go.mod: update golang.org/x dependencies

    Update golang.org/x dependencies to their latest tagged versions.
    
    Change-Id: Ib20227810c7e72942dd6fc33731fc613784aedec
    Reviewed-on: https://go-review.googlesource.com/c/crypto/+/576515
    Reviewed-by: Than McIntosh <thanm@google.com>
    Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
    LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
    Auto-Submit: Gopher Robot <gobot@golang.org>
    gopherbot committed Apr 4, 2024
    Configuration menu
    Copy the full SHA
    d042a39 View commit details
    Browse the repository at this point in the history
Loading