[PR #17122] [CLOSED] Make IPSet updates atomic and validated #479

Closed
opened 2026-08-29 05:22:41 +03:00 by mirror-bot · 0 comments

📋 Pull Request Information

Original PR: https://github.com/Flowseal/zapret-discord-youtube/pull/17122
Author: @anntr1k3
Created: 8/28/2026
Status: ❌ Closed

Base: main ← Head: fix/atomic-ipset-updates


📝 Commits (1)

  • f86a632 Make IPSet updates atomic

📊 Changes

1 file changed (+98 additions, -40 deletions)

View changed files

📝 service.bat (+98 -40)

📄 Description

Summary

Download IPSet data to a temporary file, validate it, then atomically activate it.

Changes

  • use curl failure/status/time limits;
  • validate CIDR data before activation;
  • preserve the active any/none mode while updating the saved full list;
  • keep backups and avoid deleting the active list before replacement;
  • handle a missing active file explicitly.

Validated locally with valid/invalid CIDR cases and atomic helper tests.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Flowseal/zapret-discord-youtube/pull/17122 **Author:** [@anntr1k3](https://github.com/anntr1k3) **Created:** 8/28/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/atomic-ipset-updates` --- ### 📝 Commits (1) - [`f86a632`](https://github.com/Flowseal/zapret-discord-youtube/commit/f86a632219a6b6f2a39277a2d5e61cccad3e7ad2) Make IPSet updates atomic ### 📊 Changes **1 file changed** (+98 additions, -40 deletions) <details> <summary>View changed files</summary> 📝 `service.bat` (+98 -40) </details> ### 📄 Description ## Summary Download IPSet data to a temporary file, validate it, then atomically activate it. ## Changes - use curl failure/status/time limits; - validate CIDR data before activation; - preserve the active any/none mode while updating the saved full list; - keep backups and avoid deleting the active list before replacement; - handle a missing active file explicitly. Validated locally with valid/invalid CIDR cases and atomic helper tests. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror-bot added the pull-request label 2026-08-29 05:22:41 +03:00
mirror-bot changed title from [PR #17122] Make IPSet updates atomic and validated to [PR #17122] [CLOSED] Make IPSet updates atomic and validated 2026-08-29 21:22:13 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Utilities/zapret-discord-youtube#479