Commit graph

977 commits

Author SHA1 Message Date
4s3ti
71dc364418
Merge pull request #1367 from MichaIng/patch-3
Remove debconf-apt-progress usage
2021-10-24 18:03:51 +02:00
MichaIng
f9d86cba40
Remove debconf-apt-progress usage
debconf-apt-progress is a tool to show a whiptail based dialog with progress bar for apt package installs, but it is some downsides:
- It aborts whenever apt or debconf halt for an interactive input, hence this would need to be prevented carefully, e.g. via "DEBIAN_FRONTEND=noninteractive" and "--force-confdef/old/new/miss", while it is questionable whether PiVPN should mute such configuration inputs for users.
- It even aborts when such interactive input is not actually required in some cases, but triggered by some other debconf load internals: pivpn#1360

Most importantly, aside of the visually probably appealing progress bar, debconf-apt-progress has not any upsides but reduces transparency of what is actually done, and the installer has a fallback already.

This commit removes the debconf-apt-progress usage in favour of the fallback: direct apt-get usage.

Signed-off-by: MichaIng <micha@dietpi.com>
2021-10-24 17:53:32 +02:00
4s3ti
306ddc7369
Merge pull request #1366 from MichaIng/patch-1
Remove wget usage and dependency
2021-10-24 17:47:09 +02:00
MichaIng
7e64b27590
Remove wget usage and dependency
There is only a single wget call in the installer, at the beginning of a pipe where curl may be the more natural choice. Since curl is a dependency already, this commit replaces the only wget call with curl and hence removes wget from installer dependencies.

Additionally, all curl calls get additional flags:
-s: The "silent" flag is now consequently used to suppress all processing output. It is not necessarily required when the STDOUT is a pipe or command substitution, but it does not hurt.
-S: Even when the curl output is piped, it may be helpful to have errors visible via STDERR, for debugging and being transparent about actual connection/download errors. This flag preserves error messages but does not send them to STDOUT, hence the data processed in the pipe or command substitution is not affected.
-f: There are cases where webservers return a 40x HTML document which is then printed to STDOUT by curl, like a 20x document, and hence unintentionally processed by the end of the pipe or command substitution. Usually this just cases a syntax error, but in theory it can have any unintended or even dangerous effect. The "fail" flag assures that 40x responses are printed as shortened error messages to STDERR and nothing is output to STDOUT. It should be hence seen as mandatory flag whenever curl is called to not print something to console but the output is processed.
-L: wget follows redirects automatically, curl requires the "-L" flag for this. For the wget => curl migration it is hence added to not change the behaviour.

Signed-off-by: MichaIng <micha@dietpi.com>
2021-10-24 17:34:51 +02:00
ImgBotApp
3358aa97e5
[ImgBot] Optimize images
*Total -- 475.54kb -> 460.64kb (3.13%)

/logos.jpg -- 72.87kb -> 62.17kb (14.68%)
/pivpnbanner.png -- 75.15kb -> 70.96kb (5.57%)
/logos.png -- 327.53kb -> 327.51kb (0%)

Signed-off-by: ImgBotApp <ImgBotHelp@gmail.com>
2021-10-24 15:19:23 +00:00
4s3ti
b9230a011f typo: Typo on changelog
Month was spelled incorrectly
2021-10-23 23:09:08 +02:00
4s3ti
dc2cf12d91 Keep wireguard and openvpn logos
Adding back wireguard and openvpn logos underneath pivpn banner
2021-10-23 23:06:47 +02:00
4s3ti
24ef49712e docs: Readme and changelog
Changed LatesteUpdate.md to CHANGELOG.md
README.md
  * Added status page badge
  * Changed websites links text and position
  * Replaced banner with pivpn banner
2021-10-23 23:01:30 +02:00
4s3ti
d7f73b6342
Merge pull request #1363 from MichaIng/patch-1
Add missing leading slash
2021-10-11 16:30:21 +02:00
MichaIng
8797b60374
Add missing leading slash
Introduced with: https://github.com/pivpn/pivpn/pull/1362

Signed-off-by: MichaIng <micha@dietpi.com>
2021-10-11 16:27:07 +02:00
4s3ti
090642fd7d
Merge pull request #1362 from MichaIng/patch-1
Fix errors on logging configuration
2021-10-11 15:40:22 +02:00
MichaIng
2848a6baea
Fix errors on logging configuration
rsyslog and/or logrotate may not be installed on the system, in which case creating the configurations may fail. As rsyslog and logrotate may be installed later on, it makes sense to pre-create those directories so that then OpenVPN logs are handled as expected.

Also, if the rsyslog service is not installed or not active, it should not and does not need to be restarted. Since there is not `-e` flag, ` || true` has not effect, but also, when using that flag it makes sense to exit if for some reason the running service couldn't be restarted, which may then be caused by the added configuration.

Signed-off-by: MichaIng <micha@dietpi.com>
2021-10-11 15:29:48 +02:00
4s3ti
226722f632
Merge pull request #1358 from kaihoffman/master
Fix grammar in uninstall.sh
2021-09-20 12:32:39 +02:00
Kai Hoffman
1423f97376
Fix grammar
Update "simpling" to "simply".
2021-09-17 16:05:45 +01:00
Orazio
50798ccd42 Important change to custom MTU handling (mainly to fix issue #1357)
- When NOT providing a custom MTU, 1420 is used for both clients and server.
    Using such value for clients can break WireGuard connections on some networks,
    so iOS and Android clients have logic in place to select a proper MTU value
    when not specified. Now "pivpnMTU" with only be used to set the MTU value for
    the PiVPN installation but not for profile creation.
2021-09-15 17:52:17 +02:00
4s3ti
027f257931 Latest Changes update.
Updated LatestUpdate.md with info about last pivpn updates
2021-08-05 15:12:33 +02:00
4s3ti
78cf44156b Markdown and logo
Fixed markdown syntax
Changed logo to include pivpn logo
2021-08-05 14:54:58 +02:00
4s3ti
9005fde563 Update README
Updated readme, updated dead links, added new badges
Fix #1345
2021-08-05 13:51:28 +02:00
Orazio
c002031ce3
Merge pull request #1321 from DerDanilo/add_wireguard_persistentkeepalive
Add wireguard persistentkeepalive
2021-06-23 10:17:47 +02:00
DerDanilo
6efd6a47d7 change persistentkeepalive to official suggestion 2021-06-21 22:37:34 +02:00
DerDanilo
fd15e62b4e remove inline comment 2021-06-07 23:31:29 +02:00
DerDanilo
88a701c16a add PERSISTENTKEEPALIVE support 2021-06-06 01:10:43 +02:00
DerDanilo
96102d009c
Merge pull request #1 from pivpn/master
update to current state
2021-06-06 00:57:32 +02:00
4s3ti
5aa0de636f IRC From freenode to libera.chat
Changed info about IRC Chanel,
Chanel moved from freenode to libera.chat
2021-05-31 21:02:44 +02:00
4s3ti
fb4df6d7bc IRC From freenode to libera.chat
Changed info about IRC Chanel,
Chanel moved from freenode to libera.chat
2021-05-31 21:01:46 +02:00
4s3ti
5179168c84
Merge pull request #1313 from orazioedoardo/remove-kernel-check
Remove kernel version check on Raspbian
2021-05-24 22:41:56 +02:00
Orazio
8c030c5e15
Merge pull request #1312 from orazioedoardo/debug-update
Debug update
2021-05-24 15:43:57 +02:00
Orazio
ba4ef93297 More compact commit log
- Do not show the full commit body
  - Show the branch
2021-05-12 18:17:35 +02:00
Orazio
88f6fca36e Changed link for the FAQ to https://docs.pivpn.io/faq 2021-05-11 15:22:52 +02:00
Orazio
82677135f8 Remove kernel version check on Raspbian
- Raspbian images have been shipping with kernel 5.10.17 since March 4.
    This kernel version has the WireGuard module built in which means we
    don't need to check if the user has updated, neither to install the
    module via DKMS.
2021-05-11 13:38:55 +02:00
4s3ti
4a5804a24c Multiple fixes and Shellcheck complaiance
fixes for Issue #1306, qrcode now defaults to ansiutf8,
added flag -a256 or --ansi256 if their fonts are having trouble

Shellcheck compliance #1233 for qrcodeCONF.sh,
LIST array is now created with mapfile instead
quoted multiple variables.

fixes for Issue #1307, User creation won't allow user creation with name
starting with "-", qrcodeCONF.sh won't accept wrong options or users starting with "-"
and exit with error code 1
2021-05-08 15:51:27 +02:00
4s3ti
9de9f84ff0 Update pivpn status page link
uptimerobot is no longer supporting custom domains on their free
offering, updated link to status page to use their regular link.
2021-05-07 22:24:29 +02:00
4s3ti
ec75c44f92 Pre-merge commit
Updated latestUpdate.md with changes going into next merge to master
2021-05-07 21:09:59 +02:00
4s3ti
18c115da48
Merge pull request #1305 from GHPS/master
Fix corrupted QR code image in various fonts
2021-05-07 01:31:56 +02:00
GHPS
615c2a41f4
Fix corrupted QR code image in various fonts
(affected fonts include Ubuntu Mono, Mitra Mono, InconsolataGo Nerd Mono)
2021-05-06 16:52:25 +02:00
4s3ti
bf7c4593d0
Merge pull request #1279 from DerDanilo/support_setupvars_for_wg_net_and_allowed_ips
Support setupVars for pivpnNET, subnetClass and ALLOWED_IPS via unattended setup
2021-04-21 23:52:56 +02:00
DerDanilo
70ebefdb78
Merge branch 'test' into support_setupvars_for_wg_net_and_allowed_ips 2021-04-16 21:37:07 +02:00
4s3ti
8a17b55ff7
Merge pull request #1272 from mkoester/patch-2
ufw: add comment
2021-03-29 11:47:49 +02:00
4s3ti
dcd852e02e
Merge pull request #1278 from DerDanilo/add_mtu_support
add Wireguard MTU support via setupVARS.conf
2021-03-29 11:46:21 +02:00
4s3ti
9ba27efa33
Merge pull request #1280 from codacy-badger/codacy-badge
Add a Codacy badge to README.md
2021-03-16 13:05:09 +01:00
Orazio
ba92faa15d
Merge pull request #1276 from orazioedoardo/disable-progress-bar
Disable progress bar when running unattended
2021-03-16 12:25:39 +01:00
Mirko Köster
73eb1c501d
requested change
see comment https://github.com/pivpn/pivpn/pull/1272/files#r595064063
2021-03-16 12:06:15 +01:00
The Codacy Badger
8569ef35b5 Add Codacy badge 2021-03-16 08:43:33 +00:00
DerDanilo
a3d652755c support ALLOWED_IPS from setupVARs.conf 2021-03-15 18:33:47 +01:00
DerDanilo
d3e009d171 support pivpnNET from setupVARs.conf 2021-03-15 18:33:21 +01:00
DerDanilo
805de5f77f support subnetClass from setupVARs.conf 2021-03-15 18:32:40 +01:00
DerDanilo
3c7175a41c remove uncessary vars from wireguard example file 2021-03-15 18:26:23 +01:00
DerDanilo
94744c77a9 add Wireguard MTU support via setupVARS.conf 2021-03-15 18:03:05 +01:00
Orazio
931b742862 Disable progress bar when running unattended 2021-03-15 13:54:56 +01:00
Mirko Köster
12d0efcea6
ufw: add comment
if you have a lot of ufw rules and look at them via `sudo ufw status` it would be beneficial to have comments indicating the purpose of a rule
2021-03-08 17:26:20 +01:00