A comment inside a YAML sequence is encrypted as an ENC[...,type:comment]
element and is restored as a sops.Comment node when the file is loaded
again. Since getsops#2227, Tree.Decrypt performed the MAC-inclusion test on the
input node (isComment) instead of on the decrypted result. A comment that
decrypts successfully becomes a non-Comment value, so testing the input
excluded it from the MAC. Tree.Encrypt still folds the comment in (at
encrypt time it is a non-Comment node), so the encrypt-side and
decrypt-side MACs disagreed and every file containing a comment inside a
sequence failed to decrypt with "MAC mismatch" -- including files that
3.13.2 encrypted itself.

Restore the pre-getsops#2227 behaviour of testing the decrypted value's type,
while keeping the non-string panic guard added in getsops#2227. Add a YAML store
round-trip regression test that fails before this change.

Fixes getsops#2243

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
Signed-off-by: Chris Coutinho <[email protected]>

A comment inside a YAML sequence is encrypted as an ENC[...,type:comment]
element and is restored as a sops.Comment node when the file is loaded
again. Since getsops#2227, Tree.Decrypt performed the MAC-inclusion test on the
input node (isComment) instead of on the decrypted result. A comment that
decrypts successfully becomes a non-Comment value, so testing the input
excluded it from the MAC. Tree.Encrypt still folds the comment in (at
encrypt time it is a non-Comment node), so the encrypt-side and
decrypt-side MACs disagreed and every file containing a comment inside a
sequence failed to decrypt with "MAC mismatch" -- including files that
3.13.2 encrypted itself.

Restore the pre-getsops#2227 behaviour of testing the decrypted value's type,
while keeping the non-string panic guard added in getsops#2227. Add a YAML store
round-trip regression test that fails before this change.

Fixes getsops#2243

Signed-off-by: Chris Coutinho <[email protected]>

A comment inside a YAML sequence is encrypted as an ENC[...,type:comment]
element and is restored as a sops.Comment node when the file is loaded
again. Since getsops#2227, Tree.Decrypt performed the MAC-inclusion test on the
input node (isComment) instead of on the decrypted result. A comment that
decrypts successfully becomes a non-Comment value, so testing the input
excluded it from the MAC. Tree.Encrypt still folds the comment in (at
encrypt time it is a non-Comment node), so the encrypt-side and
decrypt-side MACs disagreed and every file containing a comment inside a
sequence failed to decrypt with "MAC mismatch" -- including files that
3.13.2 encrypted itself.

Restore the pre-getsops#2227 behaviour of testing the decrypted value's type,
while keeping the non-string panic guard added in getsops#2227. Add a YAML store
round-trip regression test that fails before this change.

Fixes getsops#2243

Signed-off-by: Chris Coutinho <[email protected]>

fix: address review feedback on MAC sequence-comment fix

- Correct the explanatory comment in Tree.Decrypt: an encrypted
  non-Comment slice element whose metadata marks it a comment decrypts
  back to a Comment, so the MAC decision must use the decrypted result's
  type, not the input's.
- Move the regression test into sops_test.go, building the tree from
  SOPS objects directly and encrypting/decrypting the same tree instead
  of round-tripping through the YAML store.

This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [getsops/sops](https://github.com/getsops/sops) | patch | `v3.13.1` → `v3.13.2` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>getsops/sops (getsops/sops)</summary>

### [`v3.13.2`](https://github.com/getsops/sops/releases/tag/v3.13.2)

[Compare Source](getsops/sops@v3.13.1...v3.13.2)

#### Installation

To install `sops`, download one of the pre-built binaries provided for your platform from the artifacts attached to this release.

For instance, if you are using Linux on an AMD64 architecture:

```shell

# Download the binary
curl -LO https://github.com/getsops/sops/releases/download/v3.13.2/sops-v3.13.2.linux.amd64

# Move the binary in to your PATH
mv sops-v3.13.2.linux.amd64 /usr/local/bin/sops

# Make the binary executable
chmod +x /usr/local/bin/sops
```

##### Verify checksums file signature

The checksums file provided within the artifacts attached to this release is signed using [Cosign](https://docs.sigstore.dev/cosign/overview/) with GitHub OIDC. To validate the signature of this file, run the following commands:

```shell

# Download the checksums file, certificate and signature
curl -LO https://github.com/getsops/sops/releases/download/v3.13.2/sops-v3.13.2.checksums.txt
curl -LO https://github.com/getsops/sops/releases/download/v3.13.2/sops-v3.13.2.checksums.sigstore.json

# Verify the checksums file
cosign verify-blob sops-v3.13.2.checksums.txt \
  --bundle sops-v3.13.2.checksums.sigstore.json \
  --certificate-identity-regexp=https://github.com/getsops \
  --certificate-oidc-issuer=https://token.actions.githubusercontent.com
```

##### Verify binary integrity

To verify the integrity of the downloaded binary, you can utilize the checksums file after having validated its signature:

```shell

# Verify the binary using the checksums file
sha256sum -c sops-v3.13.2.checksums.txt --ignore-missing
```

##### Verify artifact provenance

The [SLSA provenance](https://slsa.dev/provenance/v0.2) of the binaries, packages, and SBOMs can be found within the artifacts associated with this release. It is presented through an [in-toto](https://in-toto.io/) link metadata file named `sops-v3.13.2.intoto.jsonl`. To verify the provenance of an artifact, you can utilize the [`slsa-verifier`](https://github.com/slsa-framework/slsa-verifier#artifacts) tool:

```shell

# Download the metadata file
curl -LO  https://github.com/getsops/sops/releases/download/v3.13.2/sops-v3.13.2.intoto.jsonl

# Verify the provenance of the artifact
slsa-verifier verify-artifact <artifact> \
  --provenance-path sops-v3.13.2.intoto.jsonl \
  --source-uri github.com/getsops/sops \
  --source-tag v3.13.2
```

#### Container Images

The `sops` binaries are also available as container images, based on Debian (slim) and Alpine Linux. The Debian-based container images include any dependencies which may be required to make use of certain key services, such as GnuPG, AWS KMS, Azure Key Vault, and Google Cloud KMS. The Alpine-based container images are smaller in size, but do not include these dependencies.

These container images are available for the following architectures: `linux/amd64` and `linux/arm64`.

##### GitHub Container Registry

- `ghcr.io/getsops/sops:v3.13.2`
- `ghcr.io/getsops/sops:v3.13.2-alpine`

##### Quay.io

- `quay.io/getsops/sops:v3.13.2`
- `quay.io/getsops/sops:v3.13.2-alpine`

##### Verify container image signature

The container images are signed using [Cosign](https://docs.sigstore.dev/cosign/overview/) with GitHub OIDC. To validate the signature of an image, run the following command:

```shell
cosign verify ghcr.io/getsops/sops:v3.13.2 \
  --certificate-identity-regexp=https://github.com/getsops \
  --certificate-oidc-issuer=https://token.actions.githubusercontent.com \
  -o text
```

##### Verify container image provenance

The container images include [SLSA provenance](https://slsa.dev/provenance/v0.2) attestations. For more information around the verification of this, please refer to the [`slsa-verifier` documentation](https://github.com/slsa-framework/slsa-verifier#containers).

#### Software Bill of Materials

The Software Bill of Materials (SBOM) for each binary is accessible within the artifacts enclosed with this release. It is presented as an [SPDX](https://spdx.dev/) JSON file, formatted as `<binary>.spdx.sbom.json`.

#### What's Changed

- build(deps): Bump the go group with 3 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2185](getsops/sops#2185)
- build(deps): Bump github/codeql-action from 4.35.4 to 4.35.5 in the ci group by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2184](getsops/sops#2184)
- build(deps): Bump the go group with 11 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2193](getsops/sops#2193)
- build(deps): Bump the ci group with 4 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2192](getsops/sops#2192)
- build(deps): Bump serde\_json from 1.0.149 to 1.0.150 in /functional-tests in the rust group by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2191](getsops/sops#2191)
- build(deps): Bump the go group with 11 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2197](getsops/sops#2197)
- build(deps): Bump docker/setup-qemu-action from 4.0.0 to 4.1.0 in the ci group by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2196](getsops/sops#2196)
- test: unset all age env vars in make test target by [@&#8203;arpitjain099](https://github.com/arpitjain099) in [#&#8203;2208](getsops/sops#2208)
- build(deps): Bump the go group with 10 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2212](getsops/sops#2212)
- build(deps): Bump the ci group with 2 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2211](getsops/sops#2211)
- build(deps): Bump the go group with 15 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2218](getsops/sops#2218)
- build(deps): Bump alpine from 3.23 to 3.24 in /.release in the docker group by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2217](getsops/sops#2217)
- fix: handle pointers when serializing context by [@&#8203;tlercher](https://github.com/tlercher) in [#&#8203;2219](getsops/sops#2219)
- docs: fix typo in exec-file --filename help text by [@&#8203;s3onghyun](https://github.com/s3onghyun) in [#&#8203;2221](getsops/sops#2221)
- JSON store: preserve large integers that fit into int64 by [@&#8203;s3onghyun](https://github.com/s3onghyun) in [#&#8203;2222](getsops/sops#2222)
- Fix panic when expecting an encrypted string, but a non-string is encountered by [@&#8203;felixfontein](https://github.com/felixfontein) in [#&#8203;2227](getsops/sops#2227)
- Fix INI store no longer double-encoding newlines by [@&#8203;felixfontein](https://github.com/felixfontein) in [#&#8203;2189](getsops/sops#2189)
- exec-file/exec-env: reset supplementary groups when changing user by [@&#8203;felixfontein](https://github.com/felixfontein) in [#&#8203;2194](getsops/sops#2194)
- build(deps): Bump the go group with 6 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2229](getsops/sops#2229)
- build(deps): Bump actions/checkout from 6.0.3 to 7.0.0 in the ci group by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2228](getsops/sops#2228)
- Fix issue when changing user in exec subcommands by [@&#8203;sabre1041](https://github.com/sabre1041) in [#&#8203;2230](getsops/sops#2230)
- Shorten .md lines by [@&#8203;felixfontein](https://github.com/felixfontein) in [#&#8203;2206](getsops/sops#2206)
- Update all Go dependencies with `go get -t -u ./...` by [@&#8203;felixfontein](https://github.com/felixfontein) in [#&#8203;2231](getsops/sops#2231)
- build(deps): Bump github.com/opencontainers/runc from 1.2.8 to 1.3.6 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2233](getsops/sops#2233)
- build(deps): Bump the ci group with 2 updates by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;2236](getsops/sops#2236)
- Release 3.13.2 by [@&#8203;felixfontein](https://github.com/felixfontein) in [#&#8203;2232](getsops/sops#2232)

#### New Contributors

- [@&#8203;arpitjain099](https://github.com/arpitjain099) made their first contribution in [#&#8203;2208](getsops/sops#2208)
- [@&#8203;tlercher](https://github.com/tlercher) made their first contribution in [#&#8203;2219](getsops/sops#2219)
- [@&#8203;s3onghyun](https://github.com/s3onghyun) made their first contribution in [#&#8203;2221](getsops/sops#2221)

**Full Changelog**: <getsops/sops@v3.13.1...v3.13.2>

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever MR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yNTUuMiIsInVwZGF0ZWRJblZlciI6IjQzLjI1NS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiLCJhdXRvbWF0aW9uOmJvdC1hdXRob3JlZCIsImRlcGVuZGVuY3ktdHlwZTo6cGF0Y2giXX0=-->

A comment inside a YAML sequence is encrypted as an ENC[...,type:comment]
element and is restored as a sops.Comment node when the file is loaded
again. Since getsops#2227, Tree.Decrypt performed the MAC-inclusion test on the
input node (isComment) instead of on the decrypted result. A comment that
decrypts successfully becomes a non-Comment value, so testing the input
excluded it from the MAC. Tree.Encrypt still folds the comment in (at
encrypt time it is a non-Comment node), so the encrypt-side and
decrypt-side MACs disagreed and every file containing a comment inside a
sequence failed to decrypt with "MAC mismatch" -- including files that
3.13.2 encrypted itself.

Restore the pre-getsops#2227 behaviour of testing the decrypted value's type,
while keeping the non-string panic guard added in getsops#2227. Add a YAML store
round-trip regression test that fails before this change.

Fixes getsops#2243

Signed-off-by: Chris Coutinho <[email protected]>

fix: address review feedback on MAC sequence-comment fix

- Correct the explanatory comment in Tree.Decrypt: an encrypted
  non-Comment slice element whose metadata marks it a comment decrypts
  back to a Comment, so the MAC decision must use the decrypted result's
  type, not the input's.
- Move the regression test into sops_test.go, building the tree from
  SOPS objects directly and encrypting/decrypting the same tree instead
  of round-tripping through the YAML store.

Signed-off-by: Chris Coutinho <[email protected]>