summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--Makefile2
-rw-r--r--client/client.go16
-rw-r--r--client/client_test.go4
-rw-r--r--cmd/eduvpn-cli/main.go4
-rw-r--r--exports/exports_test_wrapper.go10
-rw-r--r--i18n/err/i18nerr.go4
-rw-r--r--internal/commonver/commonver.go (renamed from internal/version/version.go)4
-rw-r--r--internal/discovery/discovery.go18
-rw-r--r--internal/eduvpnapi/cache.go (renamed from internal/api/cache.go)4
-rw-r--r--internal/eduvpnapi/eduvpnapi.go (renamed from internal/api/api.go)32
-rw-r--r--internal/eduvpnapi/eduvpnapi_test.go (renamed from internal/api/api_test.go)8
-rw-r--r--internal/eduvpnapi/endpoints/endpoints.go (renamed from internal/api/endpoints/endpoints.go)0
-rw-r--r--internal/eduvpnapi/profiles/profiles.go (renamed from internal/api/profiles/profiles.go)0
-rw-r--r--internal/eduvpnapi/redirect.go (renamed from internal/api/redirect.go)2
-rw-r--r--internal/httpwrap/httpwrap.go (renamed from internal/http/http.go)8
-rw-r--r--internal/httpwrap/httpwrap_test.go (renamed from internal/http/http_test.go)2
-rw-r--r--internal/loglevel/loglevel.go (renamed from internal/log/log.go)4
-rw-r--r--internal/loglevel/rotate.go (renamed from internal/log/rotate.go)2
-rw-r--r--internal/loglevel/rotate_test.go (renamed from internal/log/rotate_test.go)2
-rw-r--r--internal/server/custom.go10
-rw-r--r--internal/server/institute.go10
-rw-r--r--internal/server/secureinternet.go10
-rw-r--r--internal/server/server.go10
-rw-r--r--internal/server/servers.go4
-rw-r--r--internal/test/server.go6
-rwxr-xr-xmake_release.sh4
-rwxr-xr-xprepare_release.sh4
-rw-r--r--proxy/proxy.go4
-rwxr-xr-xupload_release.sh2
-rw-r--r--wrappers/python/Makefile2
30 files changed, 96 insertions, 96 deletions
diff --git a/Makefile b/Makefile
index 7b94913..494ebd9 100644
--- a/Makefile
+++ b/Makefile
@@ -2,7 +2,7 @@
.PHONY: build docs fmt lint cli test clean coverage sloc
-VERSION := $(shell grep -o 'const Version = "[^"]*' internal/version/version.go | cut -d '"' -f 2)
+VERSION := $(shell grep -o 'const Version = "[^"]*' internal/commonver/commonver.go | cut -d '"' -f 2)
build:
diff --git a/client/client.go b/client/client.go
index ae468ed..ed1f673 100644
--- a/client/client.go
+++ b/client/client.go
@@ -12,13 +12,13 @@ import (
"time"
"codeberg.org/eduVPN/eduvpn-common/i18n/err"
- "codeberg.org/eduVPN/eduvpn-common/internal/api"
"codeberg.org/eduVPN/eduvpn-common/internal/config"
"codeberg.org/eduVPN/eduvpn-common/internal/discovery"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi"
"codeberg.org/eduVPN/eduvpn-common/internal/failover"
"codeberg.org/eduVPN/eduvpn-common/internal/fsm"
- "codeberg.org/eduVPN/eduvpn-common/internal/http"
- "codeberg.org/eduVPN/eduvpn-common/internal/log"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
+ "codeberg.org/eduVPN/eduvpn-common/internal/loglevel"
"codeberg.org/eduVPN/eduvpn-common/internal/server"
"codeberg.org/eduVPN/eduvpn-common/types/cookie"
srvtypes "codeberg.org/eduVPN/eduvpn-common/types/server"
@@ -165,7 +165,7 @@ func New(name string, version string, directory string, stateCallback func(FSMSt
// Initialize provided logger or use default
if logger == nil {
- logger = &log.Logger{}
+ logger = &loglevel.Logger{}
}
c.logr = logger
slogger, err := c.logr.Init(directory)
@@ -178,7 +178,7 @@ func New(name string, version string, directory string, stateCallback func(FSMSt
c.Name = name
// register HTTP agent
- http.RegisterAgent(userAgentName(name), version)
+ httpwrap.RegisterAgent(userAgentName(name), version)
// Initialize the FSM
c.FSM = newFSM(stateCallback)
@@ -398,7 +398,7 @@ func (c *Client) AddServer(ck *cookie.Cookie, identifier string, _type srvtypes.
}
if _type != srvtypes.TypeSecureInternet {
// Convert to an identifier
- identifier, err = http.EnsureValidURL(identifier, true)
+ identifier, err = httpwrap.EnsureValidURL(identifier, true)
if err != nil {
return i18nerr.WrapInternalf(err, "failed to convert identifier: %v", identifier)
}
@@ -432,7 +432,7 @@ func (c *Client) convertIdentifier(identifier string, t srvtypes.Type) (string,
return identifier, nil
}
// Convert to an identifier, this also converts the scheme to HTTPS
- identifier, err := http.EnsureValidURL(identifier, true)
+ identifier, err := httpwrap.EnsureValidURL(identifier, true)
if err != nil {
return "", i18nerr.Wrapf(err, "The input: '%s' is not a valid URL", identifier)
}
@@ -511,7 +511,7 @@ func (c *Client) GetConfig(ck *cookie.Cookie, identifier string, _type srvtypes.
}
if err != nil {
if startup {
- if errors.Is(err, api.ErrAuthorizeDisabled) {
+ if errors.Is(err, eduvpnapi.ErrAuthorizeDisabled) {
return nil, i18nerr.Newf("The client tried to autoconnect to the VPN server: '%s', but you need to authorize again. Please manually connect again.", identifier)
}
return nil, i18nerr.Wrapf(err, "The client tried to autoconnect to the VPN server: '%s', but the operation failed to complete", identifier)
diff --git a/client/client_test.go b/client/client_test.go
index dd69688..6d34da3 100644
--- a/client/client_test.go
+++ b/client/client_test.go
@@ -10,7 +10,7 @@ import (
"testing"
"time"
- httpw "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
"codeberg.org/eduVPN/eduvpn-common/internal/test"
"codeberg.org/eduVPN/eduvpn-common/types/cookie"
"codeberg.org/eduVPN/eduvpn-common/types/protocol"
@@ -98,7 +98,7 @@ func getServerURI(t *testing.T) string {
if serverURI == "" {
t.Skip("Skipping server test as no SERVER_URI env var has been passed")
}
- serverURI, parseErr := httpw.EnsureValidURL(serverURI, true)
+ serverURI, parseErr := httpwrap.EnsureValidURL(serverURI, true)
if parseErr != nil {
t.Skip("Skipping server test as the server uri is not valid")
}
diff --git a/cmd/eduvpn-cli/main.go b/cmd/eduvpn-cli/main.go
index bb90e40..db255cd 100644
--- a/cmd/eduvpn-cli/main.go
+++ b/cmd/eduvpn-cli/main.go
@@ -11,7 +11,7 @@ import (
"codeberg.org/eduVPN/eduvpn-common/client"
"codeberg.org/eduVPN/eduvpn-common/i18n"
- "codeberg.org/eduVPN/eduvpn-common/internal/version"
+ "codeberg.org/eduVPN/eduvpn-common/internal/commonver"
"codeberg.org/eduVPN/eduvpn-common/types/cookie"
srvtypes "codeberg.org/eduVPN/eduvpn-common/types/server"
@@ -148,7 +148,7 @@ func printConfig(url string, cc string, srvType srvtypes.Type, prof string) erro
defer os.RemoveAll(dir) //nolint:errcheck
c, err = client.New(
"org.eduvpn.app.linux",
- fmt.Sprintf("%s-cli", version.Version),
+ fmt.Sprintf("%s-cli", commonver.Version),
dir,
func(oldState client.FSMStateID, newState client.FSMStateID, data any) bool {
stateCallback(oldState, newState, data, prof, dir)
diff --git a/exports/exports_test_wrapper.go b/exports/exports_test_wrapper.go
index 013a7a3..eec5a12 100644
--- a/exports/exports_test_wrapper.go
+++ b/exports/exports_test_wrapper.go
@@ -25,7 +25,7 @@ import (
"codeberg.org/eduVPN/eduvpn-common/internal/test"
"codeberg.org/eduVPN/eduvpn-common/types/error"
- httpw "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
)
func getString(in *C.char) string {
@@ -148,7 +148,7 @@ func fakeBrowserAuth(str string) (string, error) {
}
func testServer(t *testing.T) *test.Server {
- // TODO: duplicate code between this and internal/api/api_test.go
+ // TODO: duplicate code between this and internal/eduvpnapi/eduvpnapi_test.go
listen, err := net.Listen("tcp", "127.0.0.1:0")
if err != nil {
t.Fatalf("failed to setup listener for test server: %v", err)
@@ -266,7 +266,7 @@ func testServerList(t *testing.T) {
t.Fatalf("failed to obtain server client: %v", err)
}
- httpw.DefaultTransport = sclient.Client.Transport.(*http.Transport)
+ httpwrap.DefaultTransport = sclient.Client.Transport.(*http.Transport)
gerr := getError(t, AddServer(ck, 3, listS, nil))
if gerr != "" {
@@ -418,7 +418,7 @@ func testGetConfig(t *testing.T) {
t.Fatalf("failed to obtain server client: %v", err)
}
- httpw.DefaultTransport = sclient.Client.Transport.(*http.Transport)
+ httpwrap.DefaultTransport = sclient.Client.Transport.(*http.Transport)
_, cfgErr := GetConfig(ck, 3, listS, 0, 0)
cfgErrS := getError(t, cfgErr)
@@ -501,7 +501,7 @@ func testLetsConnectDiscovery(t *testing.T) {
t.Fatalf("failed to obtain server client: %v", err)
}
- httpw.DefaultTransport = sclient.Client.Transport.(*http.Transport)
+ httpwrap.DefaultTransport = sclient.Client.Transport.(*http.Transport)
// try to add an institute access server
exptErr := fmt.Sprintf("An internal error occurred. The cause of the error is: Adding a non-custom server when the client does not use discovery is not supported, identifier: %s, type: 1.", list)
diff --git a/i18n/err/i18nerr.go b/i18n/err/i18nerr.go
index 8254dd4..1c97462 100644
--- a/i18n/err/i18nerr.go
+++ b/i18n/err/i18nerr.go
@@ -8,7 +8,7 @@ import (
"log/slog"
"sync"
- "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
"golang.org/x/text/language"
"golang.org/x/text/message"
@@ -26,7 +26,7 @@ func TranslatedInner(inner error) (string, bool) {
unwrapped = errors.Unwrap(unwrapped)
}
- var tErr *http.TimeoutError
+ var tErr *httpwrap.TimeoutError
switch {
case errors.As(inner, &tErr):
return printerOrNew(language.English).Sprintf("Timeout reached contacting URL: '%s'", tErr.URL), false
diff --git a/internal/version/version.go b/internal/commonver/commonver.go
index 015a57b..71ac16e 100644
--- a/internal/version/version.go
+++ b/internal/commonver/commonver.go
@@ -1,8 +1,8 @@
-// Package version defines a version string that is used for:
+// Package commonver defines a version string for eduvpn-common that is used for:
// - building
// - the user agent
// - tagging
-package version
+package commonver
// Version is the latest version
// Update this when releasing
diff --git a/internal/discovery/discovery.go b/internal/discovery/discovery.go
index 512756b..cde846c 100644
--- a/internal/discovery/discovery.go
+++ b/internal/discovery/discovery.go
@@ -11,7 +11,7 @@ import (
"sync"
"time"
- httpw "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
"codeberg.org/eduVPN/eduvpn-common/internal/levenshtein"
"codeberg.org/eduVPN/eduvpn-common/internal/verify"
discotypes "codeberg.org/eduVPN/eduvpn-common/types/discovery"
@@ -87,7 +87,7 @@ type Discovery struct {
// mu is the read write mutex that protects the struct from concurrent access
mu sync.RWMutex
// The httpClient for sending HTTP requests
- httpClient *httpw.Client
+ httpClient *httpwrap.Client
// Organizations represents the organizations that are returned by the discovery server
OrganizationList Organizations `json:"organizations"`
@@ -105,21 +105,21 @@ func (discovery *Discovery) file(ctx context.Context, jsonFile string, previousV
var newUpdate time.Time
// No HTTP client present, create one
if discovery.httpClient == nil {
- discovery.httpClient = httpw.NewClient(nil)
+ discovery.httpClient = httpwrap.NewClient(nil)
}
// Get json data
- jsonURL, err := httpw.JoinURLPath(DiscoURL, jsonFile)
+ jsonURL, err := httpwrap.JoinURLPath(DiscoURL, jsonFile)
if err != nil {
return newUpdate, err
}
- var opts *httpw.OptionalParams
+ var opts *httpwrap.OptionalParams
if !last.IsZero() {
header := http.Header{
"If-Modified-Since": []string{last.Format(http.TimeFormat)},
}
- opts = &httpw.OptionalParams{
+ opts = &httpwrap.OptionalParams{
Headers: header,
}
}
@@ -143,7 +143,7 @@ func (discovery *Discovery) file(ctx context.Context, jsonFile string, previousV
// Get signature
sigFile := jsonFile + ".minisig"
- sigURL, err := httpw.JoinURLPath(DiscoURL, sigFile)
+ sigURL, err := httpwrap.JoinURLPath(DiscoURL, sigFile)
if err != nil {
return newUpdate, err
}
@@ -334,7 +334,7 @@ func (discovery *Discovery) Organizations(ctx context.Context, cache bool) (*Org
var jsonDecode Organizations
update, err := discovery.file(ctx, file, discovery.OrganizationList.Version, discovery.OrganizationList.UpdateHeader, &jsonDecode)
if err != nil {
- statErr := &httpw.StatusError{}
+ statErr := &httpwrap.StatusError{}
if errors.As(err, &statErr) {
if statErr.Status != 304 {
slog.Warn("failed to get fresh organization", "error", err)
@@ -383,7 +383,7 @@ func (discovery *Discovery) Servers(ctx context.Context, cache bool) (*Servers,
var jsonDecode Servers
update, err := discovery.file(ctx, file, discovery.ServerList.Version, discovery.ServerList.UpdateHeader, &jsonDecode)
if err != nil {
- statErr := &httpw.StatusError{}
+ statErr := &httpwrap.StatusError{}
if errors.As(err, &statErr) {
if statErr.Status != 304 {
slog.Warn("failed to get fresh servers", "error", err)
diff --git a/internal/api/cache.go b/internal/eduvpnapi/cache.go
index 5c682f4..d3e2b77 100644
--- a/internal/api/cache.go
+++ b/internal/eduvpnapi/cache.go
@@ -1,4 +1,4 @@
-package api
+package eduvpnapi
import (
"context"
@@ -6,7 +6,7 @@ import (
"sync"
"time"
- "codeberg.org/eduVPN/eduvpn-common/internal/api/endpoints"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi/endpoints"
)
// EndpointCache is a struct that caches well-known API endpoints
diff --git a/internal/api/api.go b/internal/eduvpnapi/eduvpnapi.go
index 0d8e03c..62fe0d1 100644
--- a/internal/api/api.go
+++ b/internal/eduvpnapi/eduvpnapi.go
@@ -1,5 +1,5 @@
-// Package api implements version 3 of the eduVPN api: https://docs.eduvpn.org/server/v3/api.html
-package api
+// Package eduvpnapi implements version 3 of the eduVPN api: https://docs.eduvpn.org/server/v3/api.html
+package eduvpnapi
import (
"context"
@@ -14,9 +14,9 @@ import (
"codeberg.org/jwijenbergh/eduoauth-go/v2"
"golang.zx2c4.com/wireguard/wgctrl/wgtypes"
- "codeberg.org/eduVPN/eduvpn-common/internal/api/endpoints"
- "codeberg.org/eduVPN/eduvpn-common/internal/api/profiles"
- httpw "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi/endpoints"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi/profiles"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
"codeberg.org/eduVPN/eduvpn-common/internal/wireguard"
"codeberg.org/eduVPN/eduvpn-common/types/protocol"
"codeberg.org/eduVPN/eduvpn-common/types/server"
@@ -52,10 +52,10 @@ type ServerData struct {
}
// Transport returns the transport to be used for the server
-// By default it uses the transport from internal/http DefaultTransport
+// By default it uses the transport from internal/httpwrap DefaultTransport
func (s *ServerData) Transport() http.RoundTripper {
if s.transport == nil {
- return httpw.DefaultTransport
+ return httpwrap.DefaultTransport
}
return s.transport
}
@@ -99,7 +99,7 @@ func NewAPI(ctx context.Context, clientID string, sd ServerData, cb Callbacks, t
cb.TokensUpdated(sd.ID, sd.Type, tok)
},
Transport: transp,
- UserAgent: httpw.UserAgent,
+ UserAgent: httpwrap.UserAgent,
}
if tokens != nil {
@@ -170,7 +170,7 @@ func (a *API) authorize(ctx context.Context) (err error) {
return nil
}
-func (a *API) authorized(ctx context.Context, method string, endpoint string, opts *httpw.OptionalParams) (http.Header, []byte, error) {
+func (a *API) authorized(ctx context.Context, method string, endpoint string, opts *httpwrap.OptionalParams) (http.Header, []byte, error) {
ep, err := GetEndpointCache().Get(ctx, a.Data.BaseWK, a.Data.Transport())
if err != nil {
return nil, nil, err
@@ -178,17 +178,17 @@ func (a *API) authorized(ctx context.Context, method string, endpoint string, op
u := ep.API.V3.API + endpoint
// TODO: Cache HTTP client?
- httpC := httpw.NewClient(a.oauth.NewHTTPClient())
+ httpC := httpwrap.NewClient(a.oauth.NewHTTPClient())
return httpC.Do(ctx, method, u, opts)
}
-func (a *API) authorizedRetry(ctx context.Context, method string, endpoint string, opts *httpw.OptionalParams) (http.Header, []byte, error) {
+func (a *API) authorizedRetry(ctx context.Context, method string, endpoint string, opts *httpwrap.OptionalParams) (http.Header, []byte, error) {
h, body, err := a.authorized(ctx, method, endpoint, opts)
if err == nil {
return h, body, nil
}
- statErr := &httpw.StatusError{}
+ statErr := &httpwrap.StatusError{}
// Only retry authorized if we get an HTTP 401
// TODO: Can the OAuth client handle this instead?
if errors.As(err, &statErr) && statErr.Status == 401 {
@@ -214,7 +214,7 @@ func (a *API) authorizedRetry(ctx context.Context, method string, endpoint strin
// Disconnect disconnects a client from the server by sending a /disconnect API call
// This cleans up resources such as WireGuard IP allocation
func (a *API) Disconnect(ctx context.Context) error {
- _, _, err := a.authorized(ctx, http.MethodPost, "/disconnect", &httpw.OptionalParams{Timeout: 5 * time.Second})
+ _, _, err := a.authorized(ctx, http.MethodPost, "/disconnect", &httpwrap.OptionalParams{Timeout: 5 * time.Second})
return err
}
@@ -308,7 +308,7 @@ func (a *API) Connect(ctx context.Context, prof profiles.Profile, protos []proto
uv.Set("prefer_tcp", boolToYesNo(pTCP))
// Construct the parameters
- params := &httpw.OptionalParams{Headers: hdrs, Body: uv}
+ params := &httpwrap.OptionalParams{Headers: hdrs, Body: uv}
h, body, err := a.authorizedRetry(ctx, http.MethodPost, "/connect", params)
if err != nil {
return nil, fmt.Errorf("failed API /connect call: %w", err)
@@ -354,11 +354,11 @@ func (a *API) Connect(ctx context.Context, prof profiles.Profile, protos []proto
}
func getEndpoints(ctx context.Context, url string, tp http.RoundTripper) (*endpoints.Endpoints, error) {
- uStr, err := httpw.JoinURLPath(url, "/.well-known/vpn-user-portal")
+ uStr, err := httpwrap.JoinURLPath(url, "/.well-known/vpn-user-portal")
if err != nil {
return nil, err
}
- httpC := httpw.NewClient(nil)
+ httpC := httpwrap.NewClient(nil)
httpC.Client.Transport = tp
_, body, err := httpC.Get(ctx, uStr)
if err != nil {
diff --git a/internal/api/api_test.go b/internal/eduvpnapi/eduvpnapi_test.go
index e88e816..23f895b 100644
--- a/internal/api/api_test.go
+++ b/internal/eduvpnapi/eduvpnapi_test.go
@@ -1,4 +1,4 @@
-package api
+package eduvpnapi
import (
"context"
@@ -15,8 +15,8 @@ import (
"testing"
"time"
- "codeberg.org/eduVPN/eduvpn-common/internal/api/profiles"
- httpw "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi/profiles"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
"codeberg.org/eduVPN/eduvpn-common/internal/test"
"codeberg.org/eduVPN/eduvpn-common/types/protocol"
"codeberg.org/eduVPN/eduvpn-common/types/server"
@@ -254,7 +254,7 @@ func TestAPIInfo(t *testing.T) {
Refresh: "validrefresh",
ExpiredTimestamp: time.Now().Add(1 * time.Hour),
}
- statErr := &httpw.StatusError{}
+ statErr := &httpwrap.StatusError{}
cases := []struct {
hp test.HandlerPath
info *profiles.Info
diff --git a/internal/api/endpoints/endpoints.go b/internal/eduvpnapi/endpoints/endpoints.go
index c98d2c7..c98d2c7 100644
--- a/internal/api/endpoints/endpoints.go
+++ b/internal/eduvpnapi/endpoints/endpoints.go
diff --git a/internal/api/profiles/profiles.go b/internal/eduvpnapi/profiles/profiles.go
index 77109f1..77109f1 100644
--- a/internal/api/profiles/profiles.go
+++ b/internal/eduvpnapi/profiles/profiles.go
diff --git a/internal/api/redirect.go b/internal/eduvpnapi/redirect.go
index 417edf5..7af31fb 100644
--- a/internal/api/redirect.go
+++ b/internal/eduvpnapi/redirect.go
@@ -1,4 +1,4 @@
-package api
+package eduvpnapi
// customRedirects supplies redirect values that should be handled by the app itself
// here we hardcode the redirect values that we should use in the OAuth requests
diff --git a/internal/http/http.go b/internal/httpwrap/httpwrap.go
index 7b9b70d..5fd42c8 100644
--- a/internal/http/http.go
+++ b/internal/httpwrap/httpwrap.go
@@ -1,5 +1,5 @@
-// Package http defines higher level helpers for the net/http package
-package http
+// Package httpwrap defines higher level helpers for the net/http package
+package httpwrap
import (
"context"
@@ -14,7 +14,7 @@ import (
"strings"
"time"
- "codeberg.org/eduVPN/eduvpn-common/internal/version"
+ "codeberg.org/eduVPN/eduvpn-common/internal/commonver"
)
// UserAgent is the user agent that is used for requests
@@ -284,5 +284,5 @@ func (e *StatusError) Error() string {
// RegisterAgent registers the user agent for client and version
func RegisterAgent(client string, verApp string) {
- UserAgent = fmt.Sprintf("%s/%s eduvpn-common/%s", client, verApp, version.Version)
+ UserAgent = fmt.Sprintf("%s/%s eduvpn-common/%s", client, verApp, commonver.Version)
}
diff --git a/internal/http/http_test.go b/internal/httpwrap/httpwrap_test.go
index 8c2ae0f..422ee3f 100644
--- a/internal/http/http_test.go
+++ b/internal/httpwrap/httpwrap_test.go
@@ -1,4 +1,4 @@
-package http
+package httpwrap
import (
"testing"
diff --git a/internal/log/log.go b/internal/loglevel/loglevel.go
index 91eaed8..74dd49f 100644
--- a/internal/log/log.go
+++ b/internal/loglevel/loglevel.go
@@ -1,5 +1,5 @@
-// Package log implements a basic level based logger
-package log
+// Package loglevel implements a basic level based logger
+package loglevel
import (
"fmt"
diff --git a/internal/log/rotate.go b/internal/loglevel/rotate.go
index 2971f70..bfd8351 100644
--- a/internal/log/rotate.go
+++ b/internal/loglevel/rotate.go
@@ -1,4 +1,4 @@
-package log
+package loglevel
import (
"io"
diff --git a/internal/log/rotate_test.go b/internal/loglevel/rotate_test.go
index 4fa77fd..d836330 100644
--- a/internal/log/rotate_test.go
+++ b/internal/loglevel/rotate_test.go
@@ -1,4 +1,4 @@
-package log
+package loglevel
import (
"io"
diff --git a/internal/server/custom.go b/internal/server/custom.go
index a9a26b9..0837c86 100644
--- a/internal/server/custom.go
+++ b/internal/server/custom.go
@@ -5,8 +5,8 @@ import (
"log/slog"
"time"
- "codeberg.org/eduVPN/eduvpn-common/internal/api"
"codeberg.org/eduVPN/eduvpn-common/internal/config/v2"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi"
"codeberg.org/eduVPN/eduvpn-common/types/server"
"codeberg.org/jwijenbergh/eduoauth-go/v2"
)
@@ -16,7 +16,7 @@ import (
// `id` is the identifier of the server, the base URL
// `ot` specifies specifies the start time OAuth was already triggered
func (s *Servers) AddCustom(ctx context.Context, id string, ot *int64) error {
- sd := api.ServerData{
+ sd := eduvpnapi.ServerData{
ID: id,
Type: server.TypeCustom,
BaseWK: id,
@@ -40,7 +40,7 @@ func (s *Servers) AddCustom(ctx context.Context, id string, ot *int64) error {
}
// Authorize by creating the API object
- _, err = api.NewAPI(ctx, s.clientID, sd, s.cb, nil)
+ _, err = eduvpnapi.NewAPI(ctx, s.clientID, sd, s.cb, nil)
if err != nil {
// authorization has failed, remove the server again
rerr := s.config.RemoveServer(id, server.TypeCustom)
@@ -58,7 +58,7 @@ func (s *Servers) AddCustom(ctx context.Context, id string, ot *int64) error {
// `tok` are the tokens such that we can initialize the API
// `disableAuth` is set to True when authorization should not be triggered
func (s *Servers) GetCustom(ctx context.Context, id string, tok *eduoauth.Token, disableAuth bool) (*Server, error) {
- sd := api.ServerData{
+ sd := eduvpnapi.ServerData{
ID: id,
Type: server.TypeCustom,
BaseWK: id,
@@ -71,7 +71,7 @@ func (s *Servers) GetCustom(ctx context.Context, id string, tok *eduoauth.Token,
if err != nil {
return nil, err
}
- a, err := api.NewAPI(ctx, s.clientID, sd, s.cb, tok)
+ a, err := eduvpnapi.NewAPI(ctx, s.clientID, sd, s.cb, tok)
if err != nil {
return nil, err
}
diff --git a/internal/server/institute.go b/internal/server/institute.go
index c357a4d..9280d1e 100644
--- a/internal/server/institute.go
+++ b/internal/server/institute.go
@@ -5,9 +5,9 @@ import (
"log/slog"
"time"
- "codeberg.org/eduVPN/eduvpn-common/internal/api"
"codeberg.org/eduVPN/eduvpn-common/internal/config/v2"
"codeberg.org/eduVPN/eduvpn-common/internal/discovery"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi"
"codeberg.org/eduVPN/eduvpn-common/types/server"
"codeberg.org/jwijenbergh/eduoauth-go/v2"
)
@@ -24,7 +24,7 @@ func (s *Servers) AddInstitute(ctx context.Context, disco *discovery.Discovery,
return err
}
- sd := api.ServerData{
+ sd := eduvpnapi.ServerData{
ID: dsrv.BaseURL,
Type: server.TypeInstituteAccess,
BaseWK: dsrv.BaseURL,
@@ -49,7 +49,7 @@ func (s *Servers) AddInstitute(ctx context.Context, disco *discovery.Discovery,
}
// Authorize by creating the API object
- _, err = api.NewAPI(ctx, s.clientID, sd, s.cb, nil)
+ _, err = eduvpnapi.NewAPI(ctx, s.clientID, sd, s.cb, nil)
if err != nil {
// authorization has failed, remove the server again
rerr := s.config.RemoveServer(dsrv.BaseURL, server.TypeInstituteAccess)
@@ -79,7 +79,7 @@ func (s *Servers) GetInstitute(ctx context.Context, id string, disco *discovery.
if err != nil {
return nil, err
}
- sd := api.ServerData{
+ sd := eduvpnapi.ServerData{
ID: dsrv.BaseURL,
Type: server.TypeInstituteAccess,
BaseWK: dsrv.BaseURL,
@@ -87,7 +87,7 @@ func (s *Servers) GetInstitute(ctx context.Context, id string, disco *discovery.
DisableAuthorize: disableAuth,
}
// Authorize by creating the API object
- a, err := api.NewAPI(ctx, s.clientID, sd, s.cb, tok)
+ a, err := eduvpnapi.NewAPI(ctx, s.clientID, sd, s.cb, tok)
if err != nil {
return nil, err
}
diff --git a/internal/server/secureinternet.go b/internal/server/secureinternet.go
index d25750f..e97efbd 100644
--- a/internal/server/secureinternet.go
+++ b/internal/server/secureinternet.go
@@ -8,9 +8,9 @@ import (
"strings"
"time"
- "codeberg.org/eduVPN/eduvpn-common/internal/api"
"codeberg.org/eduVPN/eduvpn-common/internal/config/v2"
"codeberg.org/eduVPN/eduvpn-common/internal/discovery"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi"
"codeberg.org/eduVPN/eduvpn-common/types/server"
"codeberg.org/jwijenbergh/eduoauth-go/v2"
)
@@ -54,7 +54,7 @@ func (s *Servers) AddSecure(ctx context.Context, disco *discovery.Discovery, org
return err
}
- sd := api.ServerData{
+ sd := eduvpnapi.ServerData{
ID: dorg.OrgID,
Type: server.TypeSecureInternet,
BaseWK: dsrv.BaseURL,
@@ -92,7 +92,7 @@ func (s *Servers) AddSecure(ctx context.Context, disco *discovery.Discovery, org
}
// Authorize by creating the API object
- _, err = api.NewAPI(ctx, s.clientID, sd, s.cb, nil)
+ _, err = eduvpnapi.NewAPI(ctx, s.clientID, sd, s.cb, nil)
if err != nil {
// authorization has failed, remove the server again
rerr := s.config.RemoveServer(orgID, server.TypeSecureInternet)
@@ -126,7 +126,7 @@ func (s *Servers) GetSecure(ctx context.Context, orgID string, disco *discovery.
return nil, err
}
- sd := api.ServerData{
+ sd := eduvpnapi.ServerData{
ID: dorg.OrgID,
Type: server.TypeSecureInternet,
BaseWK: dloc.BaseURL,
@@ -148,7 +148,7 @@ func (s *Servers) GetSecure(ctx context.Context, orgID string, disco *discovery.
DisableAuthorize: disableAuth,
}
- a, err := api.NewAPI(ctx, s.clientID, sd, s.cb, tok)
+ a, err := eduvpnapi.NewAPI(ctx, s.clientID, sd, s.cb, tok)
if err != nil {
return nil, err
}
diff --git a/internal/server/server.go b/internal/server/server.go
index 2372494..4623224 100644
--- a/internal/server/server.go
+++ b/internal/server/server.go
@@ -7,9 +7,9 @@ import (
"os"
"time"
- "codeberg.org/eduVPN/eduvpn-common/internal/api"
- "codeberg.org/eduVPN/eduvpn-common/internal/api/profiles"
v2 "codeberg.org/eduVPN/eduvpn-common/internal/config/v2"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi/profiles"
"codeberg.org/eduVPN/eduvpn-common/types/protocol"
srvtypes "codeberg.org/eduVPN/eduvpn-common/types/server"
)
@@ -18,7 +18,7 @@ import (
type Server struct {
identifier string
t srvtypes.Type
- apiw *api.API
+ apiw *eduvpnapi.API
storage *v2.V2
}
@@ -26,7 +26,7 @@ type Server struct {
var ErrInvalidProfile = errors.New("invalid profile")
// NewServer creates a new server
-func (s *Servers) NewServer(identifier string, t srvtypes.Type, api *api.API) Server {
+func (s *Servers) NewServer(identifier string, t srvtypes.Type, api *eduvpnapi.API) Server {
return Server{
identifier: identifier,
t: t,
@@ -65,7 +65,7 @@ func (s *Server) FreshProfiles(ctx context.Context) (*profiles.Info, error) {
return prfs, nil
}
-func (s *Server) api() (*api.API, error) {
+func (s *Server) api() (*eduvpnapi.API, error) {
if s.apiw == nil {
return nil, errors.New("no API object found")
}
diff --git a/internal/server/servers.go b/internal/server/servers.go
index bd22ffd..43716a4 100644
--- a/internal/server/servers.go
+++ b/internal/server/servers.go
@@ -5,9 +5,9 @@ import (
"errors"
"fmt"
- "codeberg.org/eduVPN/eduvpn-common/internal/api"
"codeberg.org/eduVPN/eduvpn-common/internal/config/v2"
"codeberg.org/eduVPN/eduvpn-common/internal/discovery"
+ "codeberg.org/eduVPN/eduvpn-common/internal/eduvpnapi"
srvtypes "codeberg.org/eduVPN/eduvpn-common/types/server"
"codeberg.org/jwijenbergh/eduoauth-go/v2"
)
@@ -15,7 +15,7 @@ import (
// Callbacks defines the interface for doing certain callback operations
type Callbacks interface {
// api.Callbacks is the API callback interface
- api.Callbacks
+ eduvpnapi.Callbacks
// GettingConfig is called when the config is obtained
GettingConfig() error
// InvalidProfile is called when an invalid profile is found
diff --git a/internal/test/server.go b/internal/test/server.go
index 2596f29..82dd2f7 100644
--- a/internal/test/server.go
+++ b/internal/test/server.go
@@ -8,7 +8,7 @@ import (
"net/http"
"net/http/httptest"
- httpw "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
)
// Server wraps a HTTP test server
@@ -71,7 +71,7 @@ func NewServerWithHandles(hps []HandlerPath, listener net.Listener) *Server {
}
// Client returns a test client that trusts the HTTPS certificates
-func (srv *Server) Client() (*httpw.Client, error) {
+func (srv *Server) Client() (*httpwrap.Client, error) {
// Get the certs from the test server
certs := x509.NewCertPool()
for _, c := range srv.TLS.Certificates {
@@ -91,6 +91,6 @@ func (srv *Server) Client() (*httpw.Client, error) {
},
}
// Override the client such that it only trusts the test server cert
- httpC := httpw.NewClient(client)
+ httpC := httpwrap.NewClient(client)
return httpC, nil
}
diff --git a/make_release.sh b/make_release.sh
index 02affb0..bbb633d 100755
--- a/make_release.sh
+++ b/make_release.sh
@@ -2,7 +2,7 @@
# This script was adapted from fkooman: https://git.sr.ht/~fkooman/vpn-daemon/tree/main/item/make_release.sh. Thanks!
#
-# Make a release of the version specified in internal/version/version.go and automatically release the artifacts
+# Make a release of the version specified in internal/commonver/commonver.go and automatically release the artifacts
#
# Fail if error
@@ -11,7 +11,7 @@ set -e
echo "building $(git log -n 1 | head -n 1)"
BRANCH="main"
PROJECT_NAME=$(basename "${PWD}")
-PROJECT_VERSION=$(grep -o 'const Version = "[^"]*' internal/version/version.go | cut -d '"' -f 2)
+PROJECT_VERSION=$(grep -o 'const Version = "[^"]*' internal/commonver/commonver.go | cut -d '"' -f 2)
PRERELEASE=false
while [[ "$#" -gt 0 ]]; do
diff --git a/prepare_release.sh b/prepare_release.sh
index 61d0cfc..91ab23a 100755
--- a/prepare_release.sh
+++ b/prepare_release.sh
@@ -45,8 +45,8 @@ if [[ $(git diff) ]]; then
fi
# Replace version number
-# replace in internal/version
-sed -i "s/const Version = \".*\"/const Version = \"${PROJECT_VERSION}\"/" internal/version/version.go
+# replace in internal/commonver
+sed -i "s/const Version = \".*\"/const Version = \"${PROJECT_VERSION}\"/" internal/commonver/commonver.go
sed -i "s/version = .*/version = ${PROJECT_VERSION}/" wrappers/python/setup.cfg
sed -i "s/COMMON_VERSION = \".*\"/COMMON_VERSION = \"${PROJECT_VERSION}\"/" wrappers/python/setup.py
sed -i "s/__version__ = \".*\"/__version__ = \"${PROJECT_VERSION}\"/" wrappers/python/eduvpn_common/__init__.py
diff --git a/proxy/proxy.go b/proxy/proxy.go
index af0cd89..b8a9ad9 100644
--- a/proxy/proxy.go
+++ b/proxy/proxy.go
@@ -12,7 +12,7 @@ import (
"codeberg.org/eduVPN/proxyguard"
"codeberg.org/eduVPN/eduvpn-common/i18n/err"
- httpw "codeberg.org/eduVPN/eduvpn-common/internal/http"
+ "codeberg.org/eduVPN/eduvpn-common/internal/httpwrap"
)
// Logger is defined here such that we can update the proxyguard logger
@@ -43,7 +43,7 @@ func NewProxyguard(ctx context.Context, lp int, tcpsp int, peer string, setupSoc
ListenPort: lp,
TCPSourcePort: tcpsp,
SetupSocket: setupSocket,
- UserAgent: httpw.UserAgent,
+ UserAgent: httpwrap.UserAgent,
},
resChan: make(chan struct{}),
}
diff --git a/upload_release.sh b/upload_release.sh
index a16d78d..b2537b9 100755
--- a/upload_release.sh
+++ b/upload_release.sh
@@ -14,7 +14,7 @@ fi
ORG=eduVPN
API_KEY=$(cat "$API_KEY_FILE")
PROJECT_NAME=$(basename "$(pwd)")
-PROJECT_VERSION=$(grep -o 'const Version = "[^"]*' internal/version/version.go | cut -d '"' -f 2)
+PROJECT_VERSION=$(grep -o 'const Version = "[^"]*' internal/commonver/commonver.go | cut -d '"' -f 2)
PRERELEASE=false
while [[ "$#" -gt 0 ]]; do
diff --git a/wrappers/python/Makefile b/wrappers/python/Makefile
index c55faf8..4e7f8d7 100644
--- a/wrappers/python/Makefile
+++ b/wrappers/python/Makefile
@@ -1,7 +1,7 @@
.DEFAULT_GOAL := pack
.PHONY: install-lib pack test clean lint fmt
-VERSION := $(shell grep -o 'const Version = "[^"]*' ../../internal/version/version.go | cut -d '"' -f 2)
+VERSION := $(shell grep -o 'const Version = "[^"]*' ../../internal/commonver/commonver.go | cut -d '"' -f 2)
install-lib:
rm -rf eduvpn_common/lib/*