Skip to content

source_vars(): Add grep regex for assign by equal = - #1405

Merged
TinCanTech merged 13 commits into
OpenVPN:masterfrom
TinCanTech:source_vars-add-grep-for-assign-by-equal
Nov 20, 2025
Merged

source_vars(): Add grep regex for assign by equal =#1405
TinCanTech merged 13 commits into
OpenVPN:masterfrom
TinCanTech:source_vars-add-grep-for-assign-by-equal

Conversation

@TinCanTech

@TinCanTech TinCanTech commented Nov 10, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@TinCanTech
source_vars(): Add grep check for assignment by '='
fc36545 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech TinCanTech self-assigned this Nov 10, 2025
@TinCanTech TinCanTech added this to the v3.2.5 milestone Nov 10, 2025
@TinCanTech TinCanTech linked an issue Nov 10, 2025 that may be closed by this pull request
There seems to be an issue with the configuration priority of environment variables #1404
Closed
@TinCanTech
ChangeLog: source_vars(): Add grep check for assignment by '='
f238bf0 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
source_vars(): Focus regex targets to 'EASYRSA_' and backtick
391e815 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
Rename $alias_days to $EASYRSA_ALIAS_DAYS and protect it from vars abuse
0ee2d6e 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
Move Initialisation requirements to verify_working_env(), after sourc…
9878de7 
…ing vars file

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech TinCanTech changed the title Source vars add grep for assign by equal source_vars(): Add grep regex for assign by equal = Nov 13, 2025
@TinCanTech
source_vars(): Improve error message format, add new-lines
e5ec746 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
TinCanTech

This comment was marked as resolved.

Sign in to view

@TinCanTech
source_vars(): Add 'set -e' to dry-run, sub-shell sourcing of vars file
6598711 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
source_vars(): Correct regexes
703d524 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
ChangeLog: source_vars(): Add 'set -e' to dry-run, sub-shell sourcing…
7dfd1b7 
… vars

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
Introduce global option --force-vars, ignore known errors in 'vars' file
5560d3c 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
ChangeLog: Introduce global option --force-vars
150bd0a 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
source_vars(): Dry-run source, use correct 'set -e'
13d2532 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
source_vars(): Enforce 'No-Password' rule
cc0d2d9 
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
TinCanTech merged commit c9f3869 into OpenVPN:master Nov 20, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@TinCanTech TinCanTech

Labels

BUG-FIX vars file Version 3.2.4-Release

Projects

None yet

Milestone

v3.2.5

Development

Successfully merging this pull request may close these issues.

There seems to be an issue with the configuration priority of environment variables

1 participant

@TinCanTech