Skip to content

Update entrypoint.sh to require exactly one configuration file#51

Merged
jordanpotter merged 1 commit into
mainfrom
config_files
Jan 29, 2026
Merged

Update entrypoint.sh to require exactly one configuration file#51
jordanpotter merged 1 commit into
mainfrom
config_files

Conversation

@jordanpotter

@jordanpotter jordanpotter commented Jan 29, 2026

Copy link
Copy Markdown
Owner

Improvements from the conversation in issue #48, which identified that the image doesn't correctly handle when multiple configuration files are provided.

This change ensures the container errors out explicitly if more than one configuration file is detected.

@BobSammers BobSammers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. A quick test proves it does what it promises and, as you say, it makes config choice explicit and straightforward.

My only note is that filenames containing whitespace cause issues, but a little experimentation suggests actually trying to use such names opens a whole can of worms with wg-quick and invalid interface names, so I don't think it's worth spending any time on. Only a masochist would try this anyway.

@jordanpotter jordanpotter merged commit dab5a4b into main Jan 29, 2026
1 check passed
@jordanpotter jordanpotter deleted the config_files branch January 29, 2026 16:06
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