jkito authored 8 months ago and jkito committed 8 months ago

this updates the helpers starting the obfsvpn bridge
to use the new client interface that was  introduced
in obfsvpn 1.0.0

jkito authored 8 months ago and jkito committed 8 months ago

go1.19 is out of support

Pea Nut authored 8 months ago and jkito committed 8 months ago

It does not have a value any more. Directly calling
GatewayPool.GetGatewaysByTimezone instead. Therefore making it public.

Pea Nut authored 8 months ago and jkito committed 8 months ago

It first checks if p.recommended is len=0, then iterates over it.

Pea Nut authored 8 months ago and jkito committed 8 months ago

After removing `Bitmask.ReloadFirewall()` there are no calls to `b.api.GetAllGateways()`
where the calling function actually need the gateways as return values. So I renamed the
function to FetchAllGateways. The function now only returns an error (instead of
`([]bonafide.Gateway, error)`). This simplifies the code of v5 a bit, as
we get rid of yet another function/abstraction. In v3 `gatewayPool.getAll()`
now also returns just an error.

Pea Nut authored 8 months ago and jkito committed 8 months ago

It's never called.

Pea Nut authored 9 months ago and jkito committed 8 months ago

This fixes a small logic error. Each gateway can be multiple times in
p.recommened (for each transport). If a location was selected manually,
each gateway was added twice to the returned list. The returned list is
the list of gateways we will connect to. For more information, please
check: !213

- provider is bitmask
- version is current main, as testing latest stable in the CI after
  each commit does not have a real benefit
- Added a new directory ArchLinux (there is already a for directory debian)
- There is a new target `package_arch` to create the Arch package - There is a check before if we are on a Arch System
- I did not add the `package_arch`  to the `packages` target (we can not be an Debian and Arch on the same time)
- `make clean` deletes temporary build files
- Updated .gitignore

jkito authored 9 months ago and jkito committed 8 months ago

apart from its use in snowflake/bootstrap.go its also used
to form the path to eip-service.json file on disk and motd
package both expects the provider name instead of provider
url

Pea Nut authored 9 months ago and jkito committed 9 months ago

This reverts commit 6b18a110. The file
gets deployed if installed as package or by using -i/--install-helper.

As discussed in #838. This is a security fix. Normally this file is
deployed by the Debian packge (with 755).

Pea Nut authored 9 months ago and jkito committed 9 months ago

- document environment variables
- add snap install instruction
- standardize a bit
- add PROVIDER= env to build instruction

Pea Nut authored 9 months ago and jkito committed 9 months ago

It is not needed. It's also not available on Ubuntu 22.04.

jkito authored 9 months ago and jkito committed 9 months ago

without running 'make vendor' the resulting client has bitmask
branding but provider is set to riseup

although the default provider is now set to bitmask in the vendor.conf
file

Pea Nut authored 9 months ago and jkito committed 9 months ago

os.MkdirAll returns nil if the directory exists, so we don't check if
it exists.

currently the check expects only RSA private keys but
newer menshen could serve other keys as well

this updates the check to only look for 'PRIVATE KEY'
in the preamble instead of RSA PRIVATE KEY

kwadronaut authored 9 months ago and jkito committed 9 months ago

adds the terms of service url

the test is failing with the following error:
```
--- FAIL: TestLatency (0.00s)
    integration_test.go:65:
        	Error Trace:	/builds/jkito/bitmask-vpn/pkg/vpn/menshen/integration_test.go:65
        	Error:      	Received unexpected error:
        	            	socket: permission denied
        	Test:       	TestLatency
        	Messages:   	Could not calc latency
FAIL
FAIL	0xacab.org/leap/bitmask-vpn/pkg/vpn/menshen	0.029s
```

it is beacause of missing permissons, the pro-bing library attempts
to send an "unprivileged" ping via UDP. On Linux, this must be enabled
with a sysctl command

jobs that were failing were commented to make the CI green
these have now been removed

in the CI we want to make sure that the build works with provider
branding and debian package builds correctly, which is now covered
by the existing jobs: debian, build

the whole build/qt/release directory needs to be uploaded to
be able to download and run it

this sets the artifact expiry time to 1 week to reduce storage
consumption, 1 week should be sufficient as these artifacts are
meant to be tested during development to validate a MR

this removes the go generate way of generating the
pkg/config/version.go file and uses build time  ld
flags to set the version when building from git or
uses .gitattributes to set it during git archive

jkito authored 10 months ago and jkito committed 9 months ago

osx was the old name, nowadays its called macos installation
was successful but the helper's launchd service was not setup
as post-install was not running due to the faulty check, this
was missed in 9666f8bd

ref: https://doc.qt.io/qtinstallerframework/scripting-systeminfo.html#productType-prop

after few of the qml components were modified in gui/components
the patch had to be updated or else it wasn't getting applied

after few of the qml components were updated in gui/components
the patch had to be updated or else it wasn't getting applied

this was used to create the connect/disconnect button
which now uses an icon instead of this component

this implements the buttons from the new design mockup

earlier the bottom footer toolbar would
disapper when the Drawer was open, with
the new design that is not required

this also removes the 'drawerOn' property
since it is now ununsed

this removes the custom component NavigationDrawer earlier
as during testing with Qt6.4 the default Qml Drawer  works
as expected

since we have stopped creating snap for now this should be
safe to do

this implements the new floating design for the location
selection toolbar, it is now removed from the MainView's
Page footer and added as a object of StatusBox component

Since the StatusBox is in the MainView component's  Page
Content area the background image doesn't extend to  the
Footer area of the Page if a footer is present, hence it
is removed

in the new design the riseup crow image used to
show the disconnected state takes more space