1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462146314641465146614671468146914701471147214731474147514761477147814791480148114821483148414851486148714881489149014911492149314941495149614971498149915001501150215031504150515061507150815091510151115121513151415151516151715181519152015211522152315241525152615271528152915301531153215331534153515361537153815391540 |
- // Copyright (C) 2016 The Syncthing Authors.
- //
- // This Source Code Form is subject to the terms of the Mozilla Public
- // License, v. 2.0. If a copy of the MPL was not distributed with this file,
- // You can obtain one at https://mozilla.org/MPL/2.0/.
- package api
- import (
- "bytes"
- "compress/gzip"
- "context"
- "encoding/json"
- "fmt"
- "io"
- "net"
- "net/http"
- "net/http/httptest"
- "os"
- "path/filepath"
- "reflect"
- "strconv"
- "strings"
- "testing"
- "time"
- "github.com/d4l3k/messagediff"
- "github.com/syncthing/syncthing/lib/assets"
- "github.com/syncthing/syncthing/lib/build"
- "github.com/syncthing/syncthing/lib/config"
- connmocks "github.com/syncthing/syncthing/lib/connections/mocks"
- discovermocks "github.com/syncthing/syncthing/lib/discover/mocks"
- "github.com/syncthing/syncthing/lib/events"
- eventmocks "github.com/syncthing/syncthing/lib/events/mocks"
- "github.com/syncthing/syncthing/lib/fs"
- "github.com/syncthing/syncthing/lib/locations"
- "github.com/syncthing/syncthing/lib/logger"
- loggermocks "github.com/syncthing/syncthing/lib/logger/mocks"
- "github.com/syncthing/syncthing/lib/model"
- modelmocks "github.com/syncthing/syncthing/lib/model/mocks"
- "github.com/syncthing/syncthing/lib/protocol"
- "github.com/syncthing/syncthing/lib/rand"
- "github.com/syncthing/syncthing/lib/svcutil"
- "github.com/syncthing/syncthing/lib/sync"
- "github.com/syncthing/syncthing/lib/tlsutil"
- "github.com/syncthing/syncthing/lib/ur"
- "github.com/thejerf/suture/v4"
- "golang.org/x/exp/slices"
- )
- var (
- confDir = filepath.Join("testdata", "config")
- token = filepath.Join(confDir, "csrftokens.txt")
- dev1 protocol.DeviceID
- apiCfg = newMockedConfig()
- testAPIKey = "foobarbaz"
- )
- func init() {
- dev1, _ = protocol.DeviceIDFromString("AIR6LPZ-7K4PTTV-UXQSMUU-CPQ5YWH-OEDFIIQ-JUG777G-2YQXXR5-YD6AWQR")
- apiCfg.GUIReturns(config.GUIConfiguration{APIKey: testAPIKey, RawAddress: "127.0.0.1:0"})
- }
- func TestMain(m *testing.M) {
- orig := locations.GetBaseDir(locations.ConfigBaseDir)
- locations.SetBaseDir(locations.ConfigBaseDir, confDir)
- exitCode := m.Run()
- locations.SetBaseDir(locations.ConfigBaseDir, orig)
- os.Exit(exitCode)
- }
- func TestCSRFToken(t *testing.T) {
- t.Parallel()
- max := 10 * maxCsrfTokens
- int := 5
- if testing.Short() {
- max = 1 + maxCsrfTokens
- int = 2
- }
- m := newCsrfManager("unique", "prefix", config.GUIConfiguration{}, nil, "")
- t1 := m.newToken()
- t2 := m.newToken()
- t3 := m.newToken()
- if !m.validToken(t3) {
- t.Fatal("t3 should be valid")
- }
- valid := make(map[string]struct{}, maxCsrfTokens)
- for _, token := range m.tokens {
- valid[token] = struct{}{}
- }
- for i := 0; i < max; i++ {
- if i%int == 0 {
- // t1 and t2 should remain valid by virtue of us checking them now
- // and then.
- if !m.validToken(t1) {
- t.Fatal("t1 should be valid at iteration", i)
- }
- if !m.validToken(t2) {
- t.Fatal("t2 should be valid at iteration", i)
- }
- }
- if len(m.tokens) == maxCsrfTokens {
- // We're about to add a token, which will remove the last token
- // from m.tokens.
- delete(valid, m.tokens[len(m.tokens)-1])
- }
- // The newly generated token is always valid
- t4 := m.newToken()
- if !m.validToken(t4) {
- t.Fatal("t4 should be valid at iteration", i)
- }
- valid[t4] = struct{}{}
- v := make(map[string]struct{}, maxCsrfTokens)
- for _, token := range m.tokens {
- v[token] = struct{}{}
- }
- if !reflect.DeepEqual(v, valid) {
- t.Fatalf("want valid tokens %v, got %v", valid, v)
- }
- }
- if m.validToken(t3) {
- t.Fatal("t3 should have expired by now")
- }
- }
- func TestStopAfterBrokenConfig(t *testing.T) {
- t.Parallel()
- cfg := config.Configuration{
- GUI: config.GUIConfiguration{
- RawAddress: "127.0.0.1:0",
- RawUseTLS: false,
- },
- }
- w := config.Wrap("/dev/null", cfg, protocol.LocalDeviceID, events.NoopLogger)
- srv := New(protocol.LocalDeviceID, w, "", "syncthing", nil, nil, nil, events.NoopLogger, nil, nil, nil, nil, nil, nil, false).(*service)
- defer os.Remove(token)
- srv.started = make(chan string)
- sup := suture.New("test", svcutil.SpecWithDebugLogger(l))
- sup.Add(srv)
- ctx, cancel := context.WithCancel(context.Background())
- sup.ServeBackground(ctx)
- <-srv.started
- // Service is now running, listening on a random port on localhost. Now we
- // request a config change to a completely invalid listen address. The
- // commit will fail and the service will be in a broken state.
- newCfg := config.Configuration{
- GUI: config.GUIConfiguration{
- RawAddress: "totally not a valid address",
- RawUseTLS: false,
- },
- }
- if err := srv.VerifyConfiguration(cfg, newCfg); err == nil {
- t.Fatal("Verify config should have failed")
- }
- cancel()
- }
- func TestAssetsDir(t *testing.T) {
- t.Parallel()
- // For any given request to $FILE, we should return the first found of
- // - assetsdir/$THEME/$FILE
- // - compiled in asset $THEME/$FILE
- // - assetsdir/default/$FILE
- // - compiled in asset default/$FILE
- // The asset map contains compressed assets, so create a couple of gzip compressed assets here.
- buf := new(bytes.Buffer)
- gw := gzip.NewWriter(buf)
- gw.Write([]byte("default"))
- gw.Close()
- def := assets.Asset{
- Content: buf.String(),
- Gzipped: true,
- }
- buf = new(bytes.Buffer)
- gw = gzip.NewWriter(buf)
- gw.Write([]byte("foo"))
- gw.Close()
- foo := assets.Asset{
- Content: buf.String(),
- Gzipped: true,
- }
- e := &staticsServer{
- theme: "foo",
- mut: sync.NewRWMutex(),
- assetDir: "testdata",
- assets: map[string]assets.Asset{
- "foo/a": foo, // overridden in foo/a
- "foo/b": foo,
- "default/a": def, // overridden in default/a (but foo/a takes precedence)
- "default/b": def, // overridden in default/b (but foo/b takes precedence)
- "default/c": def,
- },
- }
- s := httptest.NewServer(e)
- defer s.Close()
- // assetsdir/foo/a exists, overrides compiled in
- expectURLToContain(t, s.URL+"/a", "overridden-foo")
- // foo/b is compiled in, default/b is overridden, return compiled in
- expectURLToContain(t, s.URL+"/b", "foo")
- // only exists as compiled in default/c so use that
- expectURLToContain(t, s.URL+"/c", "default")
- // only exists as overridden default/d so use that
- expectURLToContain(t, s.URL+"/d", "overridden-default")
- }
- func expectURLToContain(t *testing.T, url, exp string) {
- res, err := http.Get(url)
- if err != nil {
- t.Error(err)
- return
- }
- if res.StatusCode != 200 {
- t.Errorf("Got %s instead of 200 OK", res.Status)
- return
- }
- data, err := io.ReadAll(res.Body)
- res.Body.Close()
- if err != nil {
- t.Error(err)
- return
- }
- if string(data) != exp {
- t.Errorf("Got %q instead of %q on %q", data, exp, url)
- return
- }
- }
- func TestDirNames(t *testing.T) {
- t.Parallel()
- names := dirNames("testdata")
- expected := []string{"config", "default", "foo", "testfolder"}
- if diff, equal := messagediff.PrettyDiff(expected, names); !equal {
- t.Errorf("Unexpected dirNames return: %#v\n%s", names, diff)
- }
- }
- type httpTestCase struct {
- URL string // URL to check
- Code int // Expected result code
- Type string // Expected content type
- Prefix string // Expected result prefix
- Timeout time.Duration // Defaults to a second
- }
- func TestAPIServiceRequests(t *testing.T) {
- t.Parallel()
- baseURL, cancel, err := startHTTP(apiCfg)
- if err != nil {
- t.Fatal(err)
- }
- t.Cleanup(cancel)
- cases := []httpTestCase{
- // /rest/db
- {
- URL: "/rest/db/completion?device=" + protocol.LocalDeviceID.String() + "&folder=default",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/db/file?folder=default&file=something",
- Code: 404,
- },
- {
- URL: "/rest/db/ignores?folder=default",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/db/need?folder=default",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/db/status?folder=default",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/db/browse?folder=default",
- Code: 200,
- Type: "application/json",
- Prefix: "null",
- },
- {
- URL: "/rest/db/status?folder=default",
- Code: 200,
- Type: "application/json",
- Prefix: "",
- },
- // /rest/stats
- {
- URL: "/rest/stats/device",
- Code: 200,
- Type: "application/json",
- Prefix: "null",
- },
- {
- URL: "/rest/stats/folder",
- Code: 200,
- Type: "application/json",
- Prefix: "null",
- },
- // /rest/svc
- {
- URL: "/rest/svc/deviceid?id=" + protocol.LocalDeviceID.String(),
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/svc/lang",
- Code: 200,
- Type: "application/json",
- Prefix: "[",
- },
- {
- URL: "/rest/svc/report",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- Timeout: 5 * time.Second,
- },
- // /rest/system
- {
- URL: "/rest/system/browse?current=~",
- Code: 200,
- Type: "application/json",
- Prefix: "[",
- },
- {
- URL: "/rest/system/config",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/config/insync",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/connections",
- Code: 200,
- Type: "application/json",
- Prefix: "null",
- },
- {
- URL: "/rest/system/discovery",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/error?since=0",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/ping",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/status",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/version",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/debug",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/log?since=0",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/system/log.txt?since=0",
- Code: 200,
- Type: "text/plain",
- Prefix: "",
- },
- // /rest/config
- {
- URL: "/rest/config",
- Code: 200,
- Type: "application/json",
- Prefix: "",
- },
- {
- URL: "/rest/config/folders",
- Code: 200,
- Type: "application/json",
- Prefix: "",
- },
- {
- URL: "/rest/config/folders/missing",
- Code: 404,
- Type: "text/plain",
- Prefix: "",
- },
- {
- URL: "/rest/config/devices",
- Code: 200,
- Type: "application/json",
- Prefix: "",
- },
- {
- URL: "/rest/config/devices/illegalid",
- Code: 400,
- Type: "text/plain",
- Prefix: "",
- },
- {
- URL: "/rest/config/devices/" + protocol.GlobalDeviceID.String(),
- Code: 404,
- Type: "text/plain",
- Prefix: "",
- },
- {
- URL: "/rest/config/options",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/config/gui",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- {
- URL: "/rest/config/ldap",
- Code: 200,
- Type: "application/json",
- Prefix: "{",
- },
- }
- for _, tc := range cases {
- tc := tc
- t.Run(cases[0].URL, func(t *testing.T) {
- t.Parallel()
- testHTTPRequest(t, baseURL, tc, testAPIKey)
- })
- }
- }
- // testHTTPRequest tries the given test case, comparing the result code,
- // content type, and result prefix.
- func testHTTPRequest(t *testing.T, baseURL string, tc httpTestCase, apikey string) {
- // Should not be parallelized, as that just causes timeouts eventually with more test-cases
- timeout := time.Second
- if tc.Timeout > 0 {
- timeout = tc.Timeout
- }
- cli := &http.Client{
- Timeout: timeout,
- }
- req, err := http.NewRequest("GET", baseURL+tc.URL, nil)
- if err != nil {
- t.Errorf("Unexpected error requesting %s: %v", tc.URL, err)
- return
- }
- req.Header.Set("X-API-Key", apikey)
- resp, err := cli.Do(req)
- if err != nil {
- t.Errorf("Unexpected error requesting %s: %v", tc.URL, err)
- return
- }
- defer resp.Body.Close()
- if resp.StatusCode != tc.Code {
- t.Errorf("Get on %s should have returned status code %d, not %s", tc.URL, tc.Code, resp.Status)
- return
- }
- ct := resp.Header.Get("Content-Type")
- if !strings.HasPrefix(ct, tc.Type) {
- t.Errorf("The content type on %s should be %q, not %q", tc.URL, tc.Type, ct)
- return
- }
- data, err := io.ReadAll(resp.Body)
- if err != nil {
- t.Errorf("Unexpected error reading %s: %v", tc.URL, err)
- return
- }
- if !bytes.HasPrefix(data, []byte(tc.Prefix)) {
- t.Errorf("Returned data from %s does not have prefix %q: %s", tc.URL, tc.Prefix, data)
- return
- }
- }
- func TestHTTPLogin(t *testing.T) {
- t.Parallel()
- cfg := newMockedConfig()
- cfg.GUIReturns(config.GUIConfiguration{
- User: "üser",
- Password: "$2a$10$IdIZTxTg/dCNuNEGlmLynOjqg4B1FvDKuIV5e0BB3pnWVHNb8.GSq", // bcrypt of "räksmörgås" in UTF-8
- RawAddress: "127.0.0.1:0",
- APIKey: testAPIKey,
- })
- baseURL, cancel, err := startHTTP(cfg)
- if err != nil {
- t.Fatal(err)
- }
- t.Cleanup(cancel)
- t.Run("no auth is rejected", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusUnauthorized {
- t.Errorf("Unexpected non-401 return code %d for unauthed request", resp.StatusCode)
- }
- })
- t.Run("incorrect password is rejected", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.SetBasicAuth("üser", "rksmrgs")
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusUnauthorized {
- t.Errorf("Unexpected non-401 return code %d for incorrect password", resp.StatusCode)
- }
- })
- t.Run("incorrect username is rejected", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.SetBasicAuth("user", "räksmörgås") // string literals in Go source code are in UTF-8
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusUnauthorized {
- t.Errorf("Unexpected non-401 return code %d for incorrect username", resp.StatusCode)
- }
- })
- t.Run("UTF-8 auth works", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.SetBasicAuth("üser", "räksmörgås") // string literals in Go source code are in UTF-8
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("Unexpected non-200 return code %d for authed request (UTF-8)", resp.StatusCode)
- }
- })
- t.Run("ISO-8859-1 auth work", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.SetBasicAuth("\xfcser", "r\xe4ksm\xf6rg\xe5s") // escaped ISO-8859-1
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("Unexpected non-200 return code %d for authed request (ISO-8859-1)", resp.StatusCode)
- }
- })
- t.Run("bad X-API-Key is rejected", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.Header.Set("X-API-Key", testAPIKey+"X")
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusUnauthorized {
- t.Errorf("Unexpected non-401 return code %d for bad API key", resp.StatusCode)
- }
- })
- t.Run("good X-API-Key is accepted", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.Header.Set("X-API-Key", testAPIKey)
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("Unexpected non-200 return code %d for API key", resp.StatusCode)
- }
- })
- t.Run("bad Bearer is rejected", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.Header.Set("Authorization", "Bearer "+testAPIKey+"X")
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusUnauthorized {
- t.Errorf("Unexpected non-401 return code %d for bad API key", resp.StatusCode)
- }
- })
- t.Run("good Bearer is accepted", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.Header.Set("Authorization", "Bearer "+testAPIKey)
- resp, err := http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("Unexpected non-200 return code %d for API key", resp.StatusCode)
- }
- })
- }
- func startHTTP(cfg config.Wrapper) (string, context.CancelFunc, error) {
- m := new(modelmocks.Model)
- assetDir := "../../gui"
- eventSub := new(eventmocks.BufferedSubscription)
- diskEventSub := new(eventmocks.BufferedSubscription)
- discoverer := new(discovermocks.Manager)
- connections := new(connmocks.Service)
- errorLog := new(loggermocks.Recorder)
- systemLog := new(loggermocks.Recorder)
- for _, l := range []*loggermocks.Recorder{errorLog, systemLog} {
- l.SinceReturns([]logger.Line{
- {
- When: time.Now(),
- Message: "Test message",
- },
- })
- }
- addrChan := make(chan string)
- mockedSummary := &modelmocks.FolderSummaryService{}
- mockedSummary.SummaryReturns(new(model.FolderSummary), nil)
- // Instantiate the API service
- urService := ur.New(cfg, m, connections, false)
- svc := New(protocol.LocalDeviceID, cfg, assetDir, "syncthing", m, eventSub, diskEventSub, events.NoopLogger, discoverer, connections, urService, mockedSummary, errorLog, systemLog, false).(*service)
- defer os.Remove(token)
- svc.started = addrChan
- // Actually start the API service
- supervisor := suture.New("API test", suture.Spec{
- PassThroughPanics: true,
- })
- supervisor.Add(svc)
- ctx, cancel := context.WithCancel(context.Background())
- supervisor.ServeBackground(ctx)
- // Make sure the API service is listening, and get the URL to use.
- addr := <-addrChan
- tcpAddr, err := net.ResolveTCPAddr("tcp", addr)
- if err != nil {
- cancel()
- return "", cancel, fmt.Errorf("weird address from API service: %w", err)
- }
- host, _, _ := net.SplitHostPort(cfg.GUI().RawAddress)
- if host == "" || host == "0.0.0.0" {
- host = "127.0.0.1"
- }
- baseURL := fmt.Sprintf("http://%s", net.JoinHostPort(host, strconv.Itoa(tcpAddr.Port)))
- return baseURL, cancel, nil
- }
- func TestCSRFRequired(t *testing.T) {
- t.Parallel()
- baseURL, cancel, err := startHTTP(apiCfg)
- if err != nil {
- t.Fatal("Unexpected error from getting base URL:", err)
- }
- t.Cleanup(cancel)
- cli := &http.Client{
- Timeout: time.Minute,
- }
- // Getting the base URL (i.e. "/") should succeed.
- resp, err := cli.Get(baseURL)
- if err != nil {
- t.Fatal("Unexpected error from getting base URL:", err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Fatal("Getting base URL should succeed, not", resp.Status)
- }
- // Find the returned CSRF token for future use
- var csrfTokenName, csrfTokenValue string
- for _, cookie := range resp.Cookies() {
- if strings.HasPrefix(cookie.Name, "CSRF-Token") {
- csrfTokenName = cookie.Name
- csrfTokenValue = cookie.Value
- break
- }
- }
- t.Run("/rest without a token should fail", func(t *testing.T) {
- t.Parallel()
- resp, err := cli.Get(baseURL + "/rest/system/config")
- if err != nil {
- t.Fatal("Unexpected error from getting /rest/system/config:", err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusForbidden {
- t.Fatal("Getting /rest/system/config without CSRF token should fail, not", resp.Status)
- }
- })
- t.Run("/rest with a token should succeed", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL+"/rest/system/config", nil)
- req.Header.Set("X-"+csrfTokenName, csrfTokenValue)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal("Unexpected error from getting /rest/system/config:", err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Fatal("Getting /rest/system/config with CSRF token should succeed, not", resp.Status)
- }
- })
- t.Run("/rest with an incorrect API key should fail, X-API-Key version", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL+"/rest/system/config", nil)
- req.Header.Set("X-API-Key", testAPIKey+"X")
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal("Unexpected error from getting /rest/system/config:", err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusForbidden {
- t.Fatal("Getting /rest/system/config with incorrect API token should fail, not", resp.Status)
- }
- })
- t.Run("/rest with an incorrect API key should fail, Bearer auth version", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL+"/rest/system/config", nil)
- req.Header.Set("Authorization", "Bearer "+testAPIKey+"X")
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal("Unexpected error from getting /rest/system/config:", err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusForbidden {
- t.Fatal("Getting /rest/system/config with incorrect API token should fail, not", resp.Status)
- }
- })
- t.Run("/rest with the API key should succeed", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL+"/rest/system/config", nil)
- req.Header.Set("X-API-Key", testAPIKey)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal("Unexpected error from getting /rest/system/config:", err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Fatal("Getting /rest/system/config with API key should succeed, not", resp.Status)
- }
- })
- t.Run("/rest with the API key as a bearer token should succeed", func(t *testing.T) {
- t.Parallel()
- req, _ := http.NewRequest("GET", baseURL+"/rest/system/config", nil)
- req.Header.Set("Authorization", "Bearer "+testAPIKey)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal("Unexpected error from getting /rest/system/config:", err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Fatal("Getting /rest/system/config with API key should succeed, not", resp.Status)
- }
- })
- }
- func TestRandomString(t *testing.T) {
- t.Parallel()
- baseURL, cancel, err := startHTTP(apiCfg)
- if err != nil {
- t.Fatal(err)
- }
- defer cancel()
- cli := &http.Client{
- Timeout: time.Second,
- }
- // The default should be to return a 32 character random string
- for _, url := range []string{"/rest/svc/random/string", "/rest/svc/random/string?length=-1", "/rest/svc/random/string?length=yo"} {
- req, _ := http.NewRequest("GET", baseURL+url, nil)
- req.Header.Set("X-API-Key", testAPIKey)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var res map[string]string
- if err := json.NewDecoder(resp.Body).Decode(&res); err != nil {
- t.Fatal(err)
- }
- if len(res["random"]) != 32 {
- t.Errorf("Expected 32 random characters, got %q of length %d", res["random"], len(res["random"]))
- }
- }
- // We can ask for a different length if we like
- req, _ := http.NewRequest("GET", baseURL+"/rest/svc/random/string?length=27", nil)
- req.Header.Set("X-API-Key", testAPIKey)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var res map[string]string
- if err := json.NewDecoder(resp.Body).Decode(&res); err != nil {
- t.Fatal(err)
- }
- if len(res["random"]) != 27 {
- t.Errorf("Expected 27 random characters, got %q of length %d", res["random"], len(res["random"]))
- }
- }
- func TestConfigPostOK(t *testing.T) {
- t.Parallel()
- cfg := bytes.NewBuffer([]byte(`{
- "version": 15,
- "folders": [
- {
- "id": "foo",
- "path": "TestConfigPostOK"
- }
- ]
- }`))
- resp, err := testConfigPost(cfg)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusOK {
- t.Error("Expected 200 OK, not", resp.Status)
- }
- os.RemoveAll("TestConfigPostOK")
- }
- func TestConfigPostDupFolder(t *testing.T) {
- t.Parallel()
- cfg := bytes.NewBuffer([]byte(`{
- "version": 15,
- "folders": [
- {"id": "foo"},
- {"id": "foo"}
- ]
- }`))
- resp, err := testConfigPost(cfg)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != http.StatusBadRequest {
- t.Error("Expected 400 Bad Request, not", resp.Status)
- }
- }
- func testConfigPost(data io.Reader) (*http.Response, error) {
- baseURL, cancel, err := startHTTP(apiCfg)
- if err != nil {
- return nil, err
- }
- defer cancel()
- cli := &http.Client{
- Timeout: time.Second,
- }
- req, _ := http.NewRequest("POST", baseURL+"/rest/system/config", data)
- req.Header.Set("X-API-Key", testAPIKey)
- return cli.Do(req)
- }
- func TestHostCheck(t *testing.T) {
- t.Parallel()
- // An API service bound to localhost should reject non-localhost host Headers
- cfg := newMockedConfig()
- cfg.GUIReturns(config.GUIConfiguration{RawAddress: "127.0.0.1:0"})
- baseURL, cancel, err := startHTTP(cfg)
- if err != nil {
- t.Fatal(err)
- }
- defer cancel()
- // A normal HTTP get to the localhost-bound service should succeed
- resp, err := http.Get(baseURL)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Error("Regular HTTP get: expected 200 OK, not", resp.Status)
- }
- // A request with a suspicious Host header should fail
- req, _ := http.NewRequest("GET", baseURL, nil)
- req.Host = "example.com"
- resp, err = http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusForbidden {
- t.Error("Suspicious Host header: expected 403 Forbidden, not", resp.Status)
- }
- // A request with an explicit "localhost:8384" Host header should pass
- req, _ = http.NewRequest("GET", baseURL, nil)
- req.Host = "localhost:8384"
- resp, err = http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Error("Explicit localhost:8384: expected 200 OK, not", resp.Status)
- }
- // A request with an explicit "localhost" Host header (no port) should pass
- req, _ = http.NewRequest("GET", baseURL, nil)
- req.Host = "localhost"
- resp, err = http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Error("Explicit localhost: expected 200 OK, not", resp.Status)
- }
- // A server with InsecureSkipHostCheck set behaves differently
- cfg = newMockedConfig()
- cfg.GUIReturns(config.GUIConfiguration{
- RawAddress: "127.0.0.1:0",
- InsecureSkipHostCheck: true,
- })
- baseURL, cancel, err = startHTTP(cfg)
- if err != nil {
- t.Fatal(err)
- }
- defer cancel()
- // A request with a suspicious Host header should be allowed
- req, _ = http.NewRequest("GET", baseURL, nil)
- req.Host = "example.com"
- resp, err = http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Error("Incorrect host header, check disabled: expected 200 OK, not", resp.Status)
- }
- if !testing.Short() {
- // A server bound to a wildcard address also doesn't do the check
- cfg = newMockedConfig()
- cfg.GUIReturns(config.GUIConfiguration{
- RawAddress: "0.0.0.0:0",
- })
- baseURL, cancel, err = startHTTP(cfg)
- if err != nil {
- t.Fatal(err)
- }
- defer cancel()
- // A request with a suspicious Host header should be allowed
- req, _ = http.NewRequest("GET", baseURL, nil)
- req.Host = "example.com"
- resp, err = http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Error("Incorrect host header, wildcard bound: expected 200 OK, not", resp.Status)
- }
- }
- // This should all work over IPv6 as well
- if runningInContainer() {
- // Working IPv6 in Docker can't be taken for granted.
- return
- }
- cfg = newMockedConfig()
- cfg.GUIReturns(config.GUIConfiguration{
- RawAddress: "[::1]:0",
- })
- baseURL, cancel, err = startHTTP(cfg)
- if err != nil {
- t.Fatal(err)
- }
- defer cancel()
- // A normal HTTP get to the localhost-bound service should succeed
- resp, err = http.Get(baseURL)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Error("Regular HTTP get (IPv6): expected 200 OK, not", resp.Status)
- }
- // A request with a suspicious Host header should fail
- req, _ = http.NewRequest("GET", baseURL, nil)
- req.Host = "example.com"
- resp, err = http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusForbidden {
- t.Error("Suspicious Host header (IPv6): expected 403 Forbidden, not", resp.Status)
- }
- // A request with an explicit "localhost:8384" Host header should pass
- req, _ = http.NewRequest("GET", baseURL, nil)
- req.Host = "localhost:8384"
- resp, err = http.DefaultClient.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Error("Explicit localhost:8384 (IPv6): expected 200 OK, not", resp.Status)
- }
- }
- func TestAddressIsLocalhost(t *testing.T) {
- t.Parallel()
- testcases := []struct {
- address string
- result bool
- }{
- // These are all valid localhost addresses
- {"localhost", true},
- {"LOCALHOST", true},
- {"localhost.", true},
- {"::1", true},
- {"127.0.0.1", true},
- {"127.23.45.56", true},
- {"localhost:8080", true},
- {"LOCALHOST:8000", true},
- {"localhost.:8080", true},
- {"[::1]:8080", true},
- {"127.0.0.1:8080", true},
- {"127.23.45.56:8080", true},
- {"www.localhost", true},
- {"www.localhost:8080", true},
- // These are all non-localhost addresses
- {"example.com", false},
- {"example.com:8080", false},
- {"localhost.com", false},
- {"localhost.com:8080", false},
- {"192.0.2.10", false},
- {"192.0.2.10:8080", false},
- {"0.0.0.0", false},
- {"0.0.0.0:8080", false},
- {"::", false},
- {"[::]:8080", false},
- {":8080", false},
- }
- for _, tc := range testcases {
- result := addressIsLocalhost(tc.address)
- if result != tc.result {
- t.Errorf("addressIsLocalhost(%q)=%v, expected %v", tc.address, result, tc.result)
- }
- }
- }
- func TestAccessControlAllowOriginHeader(t *testing.T) {
- t.Parallel()
- baseURL, cancel, err := startHTTP(apiCfg)
- if err != nil {
- t.Fatal(err)
- }
- defer cancel()
- cli := &http.Client{
- Timeout: time.Second,
- }
- req, _ := http.NewRequest("GET", baseURL+"/rest/system/status", nil)
- req.Header.Set("X-API-Key", testAPIKey)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusOK {
- t.Fatal("GET on /rest/system/status should succeed, not", resp.Status)
- }
- if resp.Header.Get("Access-Control-Allow-Origin") != "*" {
- t.Fatal("GET on /rest/system/status should return a 'Access-Control-Allow-Origin: *' header")
- }
- }
- func TestOptionsRequest(t *testing.T) {
- t.Parallel()
- baseURL, cancel, err := startHTTP(apiCfg)
- if err != nil {
- t.Fatal(err)
- }
- defer cancel()
- cli := &http.Client{
- Timeout: time.Second,
- }
- req, _ := http.NewRequest("OPTIONS", baseURL+"/rest/system/status", nil)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- if resp.StatusCode != http.StatusNoContent {
- t.Fatal("OPTIONS on /rest/system/status should succeed, not", resp.Status)
- }
- if resp.Header.Get("Access-Control-Allow-Origin") != "*" {
- t.Fatal("OPTIONS on /rest/system/status should return a 'Access-Control-Allow-Origin: *' header")
- }
- if resp.Header.Get("Access-Control-Allow-Methods") != "GET, POST, PUT, PATCH, DELETE, OPTIONS" {
- t.Fatal("OPTIONS on /rest/system/status should return a 'Access-Control-Allow-Methods: GET, POST, PUT, PATCH, DELETE, OPTIONS' header")
- }
- if resp.Header.Get("Access-Control-Allow-Headers") != "Content-Type, X-API-Key" {
- t.Fatal("OPTIONS on /rest/system/status should return a 'Access-Control-Allow-Headers: Content-Type, X-API-KEY' header")
- }
- }
- func TestEventMasks(t *testing.T) {
- t.Parallel()
- cfg := newMockedConfig()
- defSub := new(eventmocks.BufferedSubscription)
- diskSub := new(eventmocks.BufferedSubscription)
- svc := New(protocol.LocalDeviceID, cfg, "", "syncthing", nil, defSub, diskSub, events.NoopLogger, nil, nil, nil, nil, nil, nil, false).(*service)
- defer os.Remove(token)
- if mask := svc.getEventMask(""); mask != DefaultEventMask {
- t.Errorf("incorrect default mask %x != %x", int64(mask), int64(DefaultEventMask))
- }
- expected := events.FolderSummary | events.LocalChangeDetected
- if mask := svc.getEventMask("FolderSummary,LocalChangeDetected"); mask != expected {
- t.Errorf("incorrect parsed mask %x != %x", int64(mask), int64(expected))
- }
- expected = 0
- if mask := svc.getEventMask("WeirdEvent,something else that doesn't exist"); mask != expected {
- t.Errorf("incorrect parsed mask %x != %x", int64(mask), int64(expected))
- }
- if res := svc.getEventSub(DefaultEventMask); res != defSub {
- t.Errorf("should have returned the given default event sub")
- }
- if res := svc.getEventSub(DiskEventMask); res != diskSub {
- t.Errorf("should have returned the given disk event sub")
- }
- if res := svc.getEventSub(events.LocalIndexUpdated); res == nil || res == defSub || res == diskSub {
- t.Errorf("should have returned a valid, non-default event sub")
- }
- }
- func TestBrowse(t *testing.T) {
- t.Parallel()
- pathSep := string(os.PathSeparator)
- ffs := fs.NewFilesystem(fs.FilesystemTypeFake, rand.String(32)+"?nostfolder=true")
- _ = ffs.Mkdir("dir", 0o755)
- _ = fs.WriteFile(ffs, "file", []byte("hello"), 0o644)
- _ = ffs.Mkdir("MiXEDCase", 0o755)
- // We expect completion to return the full path to the completed
- // directory, with an ending slash.
- dirPath := "dir" + pathSep
- mixedCaseDirPath := "MiXEDCase" + pathSep
- cases := []struct {
- current string
- returns []string
- }{
- // The directory without slash is completed to one with slash.
- {"dir", []string{"dir" + pathSep}},
- // With slash it's completed to its contents.
- // Dirs are given pathSeps.
- // Files are not returned.
- {"", []string{mixedCaseDirPath, dirPath}},
- // Globbing is automatic based on prefix.
- {"d", []string{dirPath}},
- {"di", []string{dirPath}},
- {"dir", []string{dirPath}},
- {"f", nil},
- {"q", nil},
- // Globbing is case-insensitive
- {"mixed", []string{mixedCaseDirPath}},
- }
- for _, tc := range cases {
- ret := browseFiles(ffs, tc.current)
- if !slices.Equal(ret, tc.returns) {
- t.Errorf("browseFiles(%q) => %q, expected %q", tc.current, ret, tc.returns)
- }
- }
- }
- func TestPrefixMatch(t *testing.T) {
- t.Parallel()
- cases := []struct {
- s string
- prefix string
- expected int
- }{
- {"aaaA", "aaa", matchExact},
- {"AAAX", "BBB", noMatch},
- {"AAAX", "aAa", matchCaseIns},
- {"äÜX", "äü", matchCaseIns},
- }
- for _, tc := range cases {
- ret := checkPrefixMatch(tc.s, tc.prefix)
- if ret != tc.expected {
- t.Errorf("checkPrefixMatch(%q, %q) => %v, expected %v", tc.s, tc.prefix, ret, tc.expected)
- }
- }
- }
- func TestShouldRegenerateCertificate(t *testing.T) {
- // Self signed certificates expiring in less than a month are errored so we
- // can regenerate in time.
- crt, err := tlsutil.NewCertificateInMemory("foo.example.com", 29)
- if err != nil {
- t.Fatal(err)
- }
- if err := shouldRegenerateCertificate(crt); err == nil {
- t.Error("expected expiry error")
- }
- // Certificates with at least 31 days of life left are fine.
- crt, err = tlsutil.NewCertificateInMemory("foo.example.com", 31)
- if err != nil {
- t.Fatal(err)
- }
- if err := shouldRegenerateCertificate(crt); err != nil {
- t.Error("expected no error:", err)
- }
- if build.IsDarwin {
- // Certificates with too long an expiry time are not allowed on macOS
- crt, err = tlsutil.NewCertificateInMemory("foo.example.com", 1000)
- if err != nil {
- t.Fatal(err)
- }
- if err := shouldRegenerateCertificate(crt); err == nil {
- t.Error("expected expiry error")
- }
- }
- }
- func TestConfigChanges(t *testing.T) {
- t.Parallel()
- const testAPIKey = "foobarbaz"
- cfg := config.Configuration{
- GUI: config.GUIConfiguration{
- RawAddress: "127.0.0.1:0",
- RawUseTLS: false,
- APIKey: testAPIKey,
- },
- }
- tmpFile, err := os.CreateTemp("", "syncthing-testConfig-")
- if err != nil {
- panic(err)
- }
- defer os.Remove(tmpFile.Name())
- w := config.Wrap(tmpFile.Name(), cfg, protocol.LocalDeviceID, events.NoopLogger)
- tmpFile.Close()
- cfgCtx, cfgCancel := context.WithCancel(context.Background())
- go w.Serve(cfgCtx)
- defer cfgCancel()
- baseURL, cancel, err := startHTTP(w)
- if err != nil {
- t.Fatal("Unexpected error from getting base URL:", err)
- }
- defer cancel()
- cli := &http.Client{
- Timeout: time.Minute,
- }
- do := func(req *http.Request, status int) *http.Response {
- t.Helper()
- req.Header.Set("X-API-Key", testAPIKey)
- resp, err := cli.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.StatusCode != status {
- t.Errorf("Expected status %v, got %v", status, resp.StatusCode)
- }
- return resp
- }
- mod := func(method, path string, data interface{}) {
- t.Helper()
- bs, err := json.Marshal(data)
- if err != nil {
- t.Fatal(err)
- }
- req, _ := http.NewRequest(method, baseURL+path, bytes.NewReader(bs))
- do(req, http.StatusOK).Body.Close()
- }
- get := func(path string) *http.Response {
- t.Helper()
- req, _ := http.NewRequest(http.MethodGet, baseURL+path, nil)
- return do(req, http.StatusOK)
- }
- dev1Path := "/rest/config/devices/" + dev1.String()
- // Create device
- mod(http.MethodPut, "/rest/config/devices", []config.DeviceConfiguration{{DeviceID: dev1}})
- // Check its there
- get(dev1Path).Body.Close()
- // Modify just a single attribute
- mod(http.MethodPatch, dev1Path, map[string]bool{"Paused": true})
- // Check that attribute
- resp := get(dev1Path)
- var dev config.DeviceConfiguration
- if err := unmarshalTo(resp.Body, &dev); err != nil {
- t.Fatal(err)
- }
- if !dev.Paused {
- t.Error("Expected device to be paused")
- }
- folder2Path := "/rest/config/folders/folder2"
- // Create a folder and add another
- mod(http.MethodPut, "/rest/config/folders", []config.FolderConfiguration{{ID: "folder1", Path: "folder1"}})
- mod(http.MethodPut, folder2Path, config.FolderConfiguration{ID: "folder2", Path: "folder2"})
- // Check they are there
- get("/rest/config/folders/folder1").Body.Close()
- get(folder2Path).Body.Close()
- // Modify just a single attribute
- mod(http.MethodPatch, folder2Path, map[string]bool{"Paused": true})
- // Check that attribute
- resp = get(folder2Path)
- var folder config.FolderConfiguration
- if err := unmarshalTo(resp.Body, &folder); err != nil {
- t.Fatal(err)
- }
- if !dev.Paused {
- t.Error("Expected folder to be paused")
- }
- // Delete folder2
- req, _ := http.NewRequest(http.MethodDelete, baseURL+folder2Path, nil)
- do(req, http.StatusOK)
- // Check folder1 is still there and folder2 gone
- get("/rest/config/folders/folder1").Body.Close()
- req, _ = http.NewRequest(http.MethodGet, baseURL+folder2Path, nil)
- do(req, http.StatusNotFound)
- mod(http.MethodPatch, "/rest/config/options", map[string]int{"maxSendKbps": 50})
- resp = get("/rest/config/options")
- var opts config.OptionsConfiguration
- if err := unmarshalTo(resp.Body, &opts); err != nil {
- t.Fatal(err)
- }
- if opts.MaxSendKbps != 50 {
- t.Error("Expected 50 for MaxSendKbps, got", opts.MaxSendKbps)
- }
- }
- func TestSanitizedHostname(t *testing.T) {
- cases := []struct {
- in, out string
- }{
- {"foo.BAR-baz", "foo.bar-baz"},
- {"~.~-Min 1:a Räksmörgås-dator 😀😎 ~.~-", "min1araksmorgas-dator"},
- {"Vicenç-PC", "vicenc-pc"},
- {"~.~-~.~-", ""},
- {"", ""},
- }
- for _, tc := range cases {
- res, err := sanitizedHostname(tc.in)
- if tc.out == "" && err == nil {
- t.Errorf("%q should cause error", tc.in)
- } else if res != tc.out {
- t.Errorf("%q => %q, expected %q", tc.in, res, tc.out)
- }
- }
- }
- // runningInContainer returns true if we are inside Docker or LXC. It might
- // be prone to false negatives if things change in the future, but likely
- // not false positives.
- func runningInContainer() bool {
- if !build.IsLinux {
- return false
- }
- bs, err := os.ReadFile("/proc/1/cgroup")
- if err != nil {
- return false
- }
- if bytes.Contains(bs, []byte("/docker/")) {
- return true
- }
- if bytes.Contains(bs, []byte("/lxc/")) {
- return true
- }
- return false
- }
|