mirror of
https://github.com/trailofbits/algo.git
synced 2025-08-04 11:53:02 +02:00
fix: Correct Azure requirements file path to resolve deployment failures (#14781)
* fix: Add IPv6 support for WireGuard endpoint addresses
Fixes issue where IPv6 addresses in WireGuard configuration files were
not properly formatted with square brackets when used with port numbers.
The WireGuard client configuration template now detects IPv6 addresses
using the ansible.utils.ipv6 filter and wraps them in brackets as required
by the WireGuard configuration format.
Example outputs:
- IPv4: 192.168.1.1:51820
- IPv6: [2600:3c01::f03c:91ff:fedf:3b2a]:51820
- Hostname: vpn.example.com:51820
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
* fix: Correct Azure requirements file path to fix deployment failures
The previous fix in commit 7acdca0
updated to Azure collection v3.7.0 but
referenced the incorrect requirements file name. The file is now called
requirements.txt instead of requirements-azure.txt in v3.7.0.
This fixes the Azure deployment failure where pip cannot find the
requirements file, preventing users from deploying VPN servers on Azure.
Also added no_log: true to prevent potential credential leakage during
the pip installation process.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
* fix: Use simple colon check for IPv6 detection in WireGuard template
The previous implementation used `ansible.utils.ipv6` filter which is
not available in the current environment, causing the Smart Test Selection
workflow to fail with "No filter named 'ansible.utils.ipv6' found."
This change replaces the filter with a simple string check for colons (':')
which is a reliable way to detect IPv6 addresses since they contain colons
while IPv4 addresses and hostnames typically don't.
The fix maintains the same functionality:
- IPv6 addresses: `[2600:3c01::f03c:91ff:fedf:3b2a]:51820`
- IPv4 addresses: `192.168.1.1:51820`
- Hostnames: `vpn.example.com:51820`
This resolves the failing workflow tests in PR #14781.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
* fix: Remove unrelated changes and clean up Python cache
- Remove WireGuard IPv6 change (belongs in separate PR)
- Delete committed Python cache file
- Add __pycache__/ and *.pyc to .gitignore
This PR should only contain the Azure requirements file path fix.
---------
Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
parent
4634357fb1
commit
6d9b1b9df3
1 changed files with 2 additions and 0 deletions
2
.gitignore
vendored
2
.gitignore
vendored
|
@ -9,3 +9,5 @@ venvs/*
|
|||
!venvs/.gitinit
|
||||
.vagrant
|
||||
.ansible/
|
||||
__pycache__/
|
||||
*.pyc
|
||||
|
|
Loading…
Add table
Reference in a new issue