algo icon indicating copy to clipboard operation
algo copied to clipboard

Fix comma bug in wg config, Dockerfile dependencies, added 'update' to Makefile

Open samsipe opened this issue 4 years ago • 2 comments

Description

This is three quick fixes I figured I would share with the community.

  1. The wireguard config generator had a comma error, fixed @ roles/wireguard/defaults/main.yml
  2. Docker Image wouldn't build on any OS (added cargo dependency and it builds on MacOs and Ubuntu) @ Dockerfile
  3. Added a quick and easy Make Command for updating users in accordance with the documentation @ Makefile
    • I also changed the Makefile to work on any OS using $$(pwd) format.

Motivation and Context

Couldn't find any open issues on this but this fixes the issues above. This makes it easier for those deploying Algo from Docker, especially if they want to build their own docker image.

How Has This Been Tested?

Deployed a VPN to AWS and Linode using these changes from GitLab CI/CD, Ubuntu, and MacOS.

Types of changes

  • [X] Bug fix (non-breaking change which fixes an issue)
  • [X] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) (Not necessary)

Checklist:

  • [X] I have read the CONTRIBUTING document.
  • [X] My code follows the code style of this project.
  • [ ] My change requires a change to the documentation. (Not necessary)
  • [ ] I have updated the documentation accordingly. (Not necessary)
  • [ ] I have added tests to cover my changes. (Not necessary)
  • [X] All new and existing tests passed.

samsipe avatar Feb 17 '21 23:02 samsipe

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Feb 17 '21 23:02 CLAassistant

It looks like all of the lint check annotations were from unchanged files... not sure why Github just caught them now.

samsipe avatar Feb 18 '21 00:02 samsipe