Skip to content

fix: use -exist flag in ipset add to prevent duplicate entry errors#50293

Open
briancornally wants to merge 1 commit intoanthropics:mainfrom
briancornally:fix/ipset-add-exist-flag
Open

fix: use -exist flag in ipset add to prevent duplicate entry errors#50293
briancornally wants to merge 1 commit intoanthropics:mainfrom
briancornally:fix/ipset-add-exist-flag

Conversation

@briancornally
Copy link
Copy Markdown

Summary

  • Added -exist flag to both ipset add calls in .devcontainer/init-firewall.sh
  • Prevents the script from failing with a "set entry already exists" error if run more than once or if IPs overlap
  • Also removes trailing whitespace on line 82

Test plan

  • Run init-firewall.sh once — verify it completes successfully
  • Run init-firewall.sh a second time — verify it no longer errors on duplicate ipset entries

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants