[go] crypto/x509: update x509-limbo tests

4 views
Skip to first unread message

Daniel McCarney (Gerrit)

unread,
Sep 3, 2026, 1:41:42 PMSep 3
to goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, golang-co...@googlegroups.com
Attention needed from Daniel McCarney

Message from Daniel McCarney

Set Ready For Review

Open in Gerrit

Related details

Attention is currently required from:
  • Daniel McCarney
Submit Requirements:
  • requirement is not satisfiedCode-Review
  • requirement satisfiedNo-Unresolved-Comments
  • requirement is not satisfiedReview-Enforcement
  • requirement is not satisfiedTryBots-Pass
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: I066f9e4da0f7c817bc71737d9a49b7b1a54337a8
Gerrit-Change-Number: 827346
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel McCarney <dan...@binaryparadox.net>
Gerrit-Reviewer: Daniel McCarney <dan...@binaryparadox.net>
Gerrit-Attention: Daniel McCarney <dan...@binaryparadox.net>
Gerrit-Comment-Date: Thu, 03 Sep 2026 17:41:35 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
unsatisfied_requirement
satisfied_requirement
open
diffy

Roland Shoemaker (Gerrit)

unread,
Sep 3, 2026, 2:25:42 PMSep 3
to Daniel McCarney, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Filippo Valsorda, golang-co...@googlegroups.com
Attention needed from Daniel McCarney and Filippo Valsorda

Roland Shoemaker voted Code-Review+2

Code-Review+2
Open in Gerrit

Related details

Attention is currently required from:
  • Daniel McCarney
  • Filippo Valsorda
Submit Requirements:
  • requirement satisfiedCode-Review
  • requirement satisfiedNo-Unresolved-Comments
  • requirement is not satisfiedReview-Enforcement
  • requirement satisfiedTryBots-Pass
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: I066f9e4da0f7c817bc71737d9a49b7b1a54337a8
Gerrit-Change-Number: 827346
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel McCarney <dan...@binaryparadox.net>
Gerrit-Reviewer: Daniel McCarney <dan...@binaryparadox.net>
Gerrit-Reviewer: Filippo Valsorda <fil...@golang.org>
Gerrit-Reviewer: Roland Shoemaker <rol...@golang.org>
Gerrit-Attention: Filippo Valsorda <fil...@golang.org>
Gerrit-Attention: Daniel McCarney <dan...@binaryparadox.net>
Gerrit-Comment-Date: Thu, 03 Sep 2026 18:25:36 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Michael Pratt (Gerrit)

unread,
Sep 4, 2026, 5:58:32 PMSep 4
to Daniel McCarney, goph...@pubsubhelper.golang.org, Michael Pratt, golang...@luci-project-accounts.iam.gserviceaccount.com, Roland Shoemaker, Filippo Valsorda, golang-co...@googlegroups.com
Attention needed from Daniel McCarney and Filippo Valsorda

Michael Pratt voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Daniel McCarney
  • Filippo Valsorda
Submit Requirements:
    • requirement satisfiedCode-Review
    • requirement satisfiedNo-Unresolved-Comments
    • requirement satisfiedReview-Enforcement
    • requirement satisfiedTryBots-Pass
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: go
    Gerrit-Branch: master
    Gerrit-Change-Id: I066f9e4da0f7c817bc71737d9a49b7b1a54337a8
    Gerrit-Change-Number: 827346
    Gerrit-PatchSet: 2
    Gerrit-Owner: Daniel McCarney <dan...@binaryparadox.net>
    Gerrit-Reviewer: Daniel McCarney <dan...@binaryparadox.net>
    Gerrit-Reviewer: Filippo Valsorda <fil...@golang.org>
    Gerrit-Reviewer: Michael Pratt <mpr...@google.com>
    Gerrit-Reviewer: Roland Shoemaker <rol...@golang.org>
    Gerrit-Attention: Filippo Valsorda <fil...@golang.org>
    Gerrit-Attention: Daniel McCarney <dan...@binaryparadox.net>
    Gerrit-Comment-Date: Fri, 04 Sep 2026 21:58:29 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Daniel McCarney (Gerrit)

    unread,
    Sep 5, 2026, 8:32:50 AMSep 5
    to goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Michael Pratt, golang...@luci-project-accounts.iam.gserviceaccount.com, Roland Shoemaker, Filippo Valsorda, golang-co...@googlegroups.com

    Daniel McCarney submitted the change

    Unreviewed changes

    1 is the latest approved patch-set.
    No files were changed between the latest approved patch-set and the submitted one.

    Change information

    Commit message:
    crypto/x509: update x509-limbo tests

    This commit updates the pinned x509-limbo dependency, regenerating the
    version metadata, and ensuring the updated testcases are run.

    The updated corpus adds coverage for bare and leading-period email and
    URI name constraints. It also adds tests for empty permittedSubtrees and
    excludedSubtrees sequences, which Go accepts when the other field is
    non-empty. Allow that existing behavior for now.

    All other new test cases pass unmodified.
    Change-Id: I066f9e4da0f7c817bc71737d9a49b7b1a54337a8
    Reviewed-by: Roland Shoemaker <rol...@golang.org>
    Reviewed-by: Michael Pratt <mpr...@google.com>
    Files:
    • M src/crypto/internal/cryptotest/x509limbo/_schema/schema_gen.go
    • M src/crypto/internal/cryptotest/x509limbo/schemaversion.go
    • M src/crypto/x509/x509limbo_test.go
    Change size: XS
    Delta: 3 files changed, 7 insertions(+), 2 deletions(-)
    Branch: refs/heads/master
    Submit Requirements:
    Open in Gerrit
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: merged
    Gerrit-Project: go
    Gerrit-Branch: master
    Gerrit-Change-Id: I066f9e4da0f7c817bc71737d9a49b7b1a54337a8
    Gerrit-Change-Number: 827346
    Gerrit-PatchSet: 3
    open
    diffy
    satisfied_requirement
    Reply all
    Reply to author
    Forward
    0 new messages