From 218138afeec7fbd82f84334ab049c43736c8d489 Mon Sep 17 00:00:00 2001 From: Kristoffer Dalby Date: Wed, 2 Oct 2024 14:50:17 +0200 Subject: [PATCH] Redo OIDC configuration (#2020) expand user, add claims to user This commit expands the user table with additional fields that can be retrieved from OIDC providers (and other places) and uses this data in various tailscale response objects if it is available. This is the beginning of implementing https://docs.google.com/document/d/1X85PMxIaVWDF6T_UPji3OeeUqVBcGj_uHRM5CI-AwlY/edit trying to make OIDC more coherant and maintainable in addition to giving the user a better experience and integration with a provider. remove usernames in magic dns, normalisation of emails this commit removes the option to have usernames as part of MagicDNS domains and headscale will now align with Tailscale, where there is a root domain, and the machine name. In addition, the various normalisation functions for dns names has been made lighter not caring about username and special character that wont occur. Email are no longer normalised as part of the policy processing. untagle oidc and regcache, use typed cache This commits stops reusing the registration cache for oidc purposes and switches the cache to be types and not use any allowing the removal of a bunch of casting. try to make reauth/register branches clearer in oidc Currently there was a function that did a bunch of stuff, finding the machine key, trying to find the node, reauthing the node, returning some status, and it was called validate which was very confusing. This commit tries to split this into what to do if the node exists, if it needs to register etc. Signed-off-by: Kristoffer Dalby --- CHANGELOG.md | 14 + flake.nix | 2 +- go.mod | 3 +- go.sum | 12 +- hscontrol/app.go | 38 +- hscontrol/auth.go | 41 +- hscontrol/db/db.go | 28 +- hscontrol/db/db_test.go | 8 +- hscontrol/db/node.go | 68 ++-- hscontrol/db/preauth_keys.go | 6 +- hscontrol/db/routes.go | 2 +- hscontrol/db/routes_test.go | 1 + hscontrol/db/suite_test.go | 1 + hscontrol/db/users.go | 52 ++- hscontrol/db/users_test.go | 6 +- hscontrol/grpcv1.go | 30 +- hscontrol/handlers.go | 22 +- hscontrol/mapper/mapper.go | 33 +- hscontrol/mapper/mapper_test.go | 14 +- hscontrol/mapper/tail.go | 2 +- hscontrol/oidc.go | 643 ++++++++++++-------------------- hscontrol/policy/acls.go | 14 +- hscontrol/policy/acls_test.go | 21 +- hscontrol/suite_test.go | 4 +- hscontrol/types/config.go | 120 ++++-- hscontrol/types/config_test.go | 18 +- hscontrol/types/node.go | 15 +- hscontrol/types/node_test.go | 116 +----- hscontrol/types/users.go | 81 +++- hscontrol/util/dns.go | 33 -- hscontrol/util/dns_test.go | 94 ----- integration/auth_oidc_test.go | 3 - 32 files changed, 628 insertions(+), 917 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 2bf0a6d3..d09e1d22 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,11 +2,25 @@ ## Next +### BREAKING + +- Remove `dns.use_username_in_magic_dns` configuration option [#2020](https://github.com/juanfont/headscale/pull/2020) + - Having usernames in magic DNS is no longer possible. +- Redo OpenID Connect configuration [#2020](https://github.com/juanfont/headscale/pull/2020) + - `strip_email_domain` has been removed, domain is _always_ part of the username for OIDC. + - Users are now identified by `sub` claim in the ID token instead of username, allowing the username, name and email to be updated. + - User has been extended to store username, display name, profile picture url and email. + - These fields are forwarded to the client, and shows up nicely in the user switcher. + - These fields can be made available via the API/CLI for non-OIDC users in the future. - Remove versions older than 1.56 [#2149](https://github.com/juanfont/headscale/pull/2149) - Clean up old code required by old versions + +### Changes + - Improved compatibilty of built-in DERP server with clients connecting over WebSocket. - Allow nodes to use SSH agent forwarding [#2145](https://github.com/juanfont/headscale/pull/2145) + ## 0.23.0 (2024-09-18) This release was intended to be mainly a code reorganisation and refactoring, significantly improving the maintainability of the codebase. This should allow us to improve further and make it easier for the maintainers to keep on top of the project. diff --git a/flake.nix b/flake.nix index a4b87584..858dabff 100644 --- a/flake.nix +++ b/flake.nix @@ -32,7 +32,7 @@ # When updating go.mod or go.sum, a new sha will need to be calculated, # update this if you have a mismatch after doing a change to thos files. - vendorHash = "sha256-/CPUkLLCwNKK3z3UZyF+AY0ArMnLaDmH0HV3/RYHo4c="; + vendorHash = "sha256-SDJSFji6498WI9bJLmY62VGt21TtD2GxrxRAWyYyr0c="; subPackages = ["cmd/headscale"]; diff --git a/go.mod b/go.mod index 2b4a27f4..2bd17cfd 100644 --- a/go.mod +++ b/go.mod @@ -7,7 +7,6 @@ require ( github.com/coder/websocket v1.8.12 github.com/coreos/go-oidc/v3 v3.11.0 github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc - github.com/deckarep/golang-set/v2 v2.6.0 github.com/glebarez/sqlite v1.11.0 github.com/go-gormigrate/gormigrate/v2 v2.1.2 github.com/gofrs/uuid/v5 v5.3.0 @@ -19,7 +18,6 @@ require ( github.com/klauspost/compress v1.17.9 github.com/oauth2-proxy/mockoidc v0.0.0-20240214162133-caebfff84d25 github.com/ory/dockertest/v3 v3.11.0 - github.com/patrickmn/go-cache v2.1.0+incompatible github.com/philip-bui/grpc-zerolog v1.0.1 github.com/pkg/profile v1.7.0 github.com/prometheus/client_golang v1.20.2 @@ -49,6 +47,7 @@ require ( gorm.io/driver/postgres v1.5.9 gorm.io/gorm v1.25.11 tailscale.com v1.75.0-pre.0.20240926101731-7d1160ddaab7 + zgo.at/zcache/v2 v2.1.0 ) require ( diff --git a/go.sum b/go.sum index 6536e1d3..e2489aa2 100644 --- a/go.sum +++ b/go.sum @@ -128,8 +128,6 @@ github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc h1:U9qPSI2PIWSS1 github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= github.com/dblohm7/wingoes v0.0.0-20240123200102-b75a8a7d7eb0 h1:vrC07UZcgPzu/OjWsmQKMGg3LoPSz9jh/pQXIrHjUj4= github.com/dblohm7/wingoes v0.0.0-20240123200102-b75a8a7d7eb0/go.mod h1:Nx87SkVqTKd8UtT+xu7sM/l+LgXs6c0aHrlKusR+2EQ= -github.com/deckarep/golang-set/v2 v2.6.0 h1:XfcQbWM1LlMB8BsJ8N9vW5ehnnPVIw0je80NsVHagjM= -github.com/deckarep/golang-set/v2 v2.6.0/go.mod h1:VAky9rY/yGXJOLEDv3OMci+7wtDpOF4IN+y82NBOac4= github.com/digitalocean/go-smbios v0.0.0-20180907143718-390a4f403a8e h1:vUmf0yezR0y7jJ5pceLHthLaYf4bA5T14B6q39S4q2Q= github.com/digitalocean/go-smbios v0.0.0-20180907143718-390a4f403a8e/go.mod h1:YTIHhz/QFSYnu/EhlF2SpU2Uk+32abacUYA5ZPljz1A= github.com/djherbis/times v1.6.0 h1:w2ctJ92J8fBvWPxugmXIv7Nz7Q3iDMKNx9v5ocVH20c= @@ -364,8 +362,6 @@ github.com/opentracing/opentracing-go v1.1.0/go.mod h1:UkNAQd3GIcIGf0SeVgPpRdFSt github.com/orisano/pixelmatch v0.0.0-20220722002657-fb0b55479cde/go.mod h1:nZgzbfBr3hhjoZnS66nKrHmduYNpc34ny7RK4z5/HM0= github.com/ory/dockertest/v3 v3.11.0 h1:OiHcxKAvSDUwsEVh2BjxQQc/5EHz9n0va9awCtNGuyA= github.com/ory/dockertest/v3 v3.11.0/go.mod h1:VIPxS1gwT9NpPOrfD3rACs8Y9Z7yhzO4SB194iUDnUI= -github.com/patrickmn/go-cache v2.1.0+incompatible h1:HRMgzkcYKYpi3C8ajMPV8OFXaaRUnok+kx1WdO15EQc= -github.com/patrickmn/go-cache v2.1.0+incompatible/go.mod h1:3Qf8kWWT7OJRJbdiICTKqZju1ZixQ/KpMGzzAfe6+WQ= github.com/pelletier/go-toml/v2 v2.2.3 h1:YmeHyLY8mFWbdkNWwpr+qIL2bEqT0o95WSdkNHvL12M= github.com/pelletier/go-toml/v2 v2.2.3/go.mod h1:MfCQTFTvCcUyyvvwm1+G6H/jORL20Xlb6rzQu9GuUkc= github.com/petermattis/goid v0.0.0-20240813172612-4fcff4a6cae7 h1:Dx7Ovyv/SFnMFw3fD4oEoeorXc6saIiQ23LrGLth0Gw= @@ -729,11 +725,7 @@ modernc.org/token v1.1.0 h1:Xl7Ap9dKaEs5kLoOQeQmPWevfnk/DM5qcLcYlA8ys6Y= modernc.org/token v1.1.0/go.mod h1:UGzOrNV1mAFSEB63lOFHIpNRUVMvYTc6yu1SMY/XTDM= software.sslmate.com/src/go-pkcs12 v0.4.0 h1:H2g08FrTvSFKUj+D309j1DPfk5APnIdAQAB8aEykJ5k= software.sslmate.com/src/go-pkcs12 v0.4.0/go.mod h1:Qiz0EyvDRJjjxGyUQa2cCNZn/wMyzrRJ/qcDXOQazLI= -tailscale.com v1.75.0-pre.0.20240925091311-031f291c98fe h1:3+E/vlEsZa2FpWBz2Ly6/L4zh4utVO8z54Ms75HitrQ= -tailscale.com v1.75.0-pre.0.20240925091311-031f291c98fe/go.mod h1:G4R9objdXe2zAcLaLkDOcHfqN9XnspBifyBHGNwTzKg= -tailscale.com v1.75.0-pre.0.20240925102642-c17c476c0d59 h1:GSuB+bmPiVfBLRqVyLOFSU+9V00lXBz9HakAewevYZA= -tailscale.com v1.75.0-pre.0.20240925102642-c17c476c0d59/go.mod h1:G4R9objdXe2zAcLaLkDOcHfqN9XnspBifyBHGNwTzKg= -tailscale.com v1.75.0-pre.0.20240926030905-c90c9938c8a2 h1:ivZ1GEXMzCNI1VRp2TjUWmLuOtno7TqW26lZf7MlF4k= -tailscale.com v1.75.0-pre.0.20240926030905-c90c9938c8a2/go.mod h1:xKxYf3B3PuezFlRaMT+VhuVu8XTFUTLy+VCzLPMJVmg= tailscale.com v1.75.0-pre.0.20240926101731-7d1160ddaab7 h1:nfRWV6ECxwNvvXKtbqSVstjlEi1BWktzv3FuxWpyyx0= tailscale.com v1.75.0-pre.0.20240926101731-7d1160ddaab7/go.mod h1:xKxYf3B3PuezFlRaMT+VhuVu8XTFUTLy+VCzLPMJVmg= +zgo.at/zcache/v2 v2.1.0 h1:USo+ubK+R4vtjw4viGzTe/zjXyPw6R7SK/RL3epBBxs= +zgo.at/zcache/v2 v2.1.0/go.mod h1:gyCeoLVo01QjDZynjime8xUGHHMbsLiPyUTBpDGd4Gk= diff --git a/hscontrol/app.go b/hscontrol/app.go index 1d3cb629..5c85b064 100644 --- a/hscontrol/app.go +++ b/hscontrol/app.go @@ -18,7 +18,6 @@ import ( "syscall" "time" - "github.com/coreos/go-oidc/v3/oidc" "github.com/davecgh/go-spew/spew" "github.com/gorilla/mux" grpcMiddleware "github.com/grpc-ecosystem/go-grpc-middleware" @@ -33,7 +32,6 @@ import ( "github.com/juanfont/headscale/hscontrol/policy" "github.com/juanfont/headscale/hscontrol/types" "github.com/juanfont/headscale/hscontrol/util" - "github.com/patrickmn/go-cache" zerolog "github.com/philip-bui/grpc-zerolog" "github.com/pkg/profile" "github.com/prometheus/client_golang/prometheus/promhttp" @@ -41,7 +39,6 @@ import ( "github.com/rs/zerolog/log" "golang.org/x/crypto/acme" "golang.org/x/crypto/acme/autocert" - "golang.org/x/oauth2" "golang.org/x/sync/errgroup" "google.golang.org/grpc" "google.golang.org/grpc/codes" @@ -57,6 +54,7 @@ import ( "tailscale.com/types/dnstype" "tailscale.com/types/key" "tailscale.com/util/dnsname" + zcache "zgo.at/zcache/v2" ) var ( @@ -95,10 +93,9 @@ type Headscale struct { mapper *mapper.Mapper nodeNotifier *notifier.Notifier - oidcProvider *oidc.Provider - oauth2Config *oauth2.Config + registrationCache *zcache.Cache[string, types.Node] - registrationCache *cache.Cache + authProvider AuthProvider pollNetMapStreamWG sync.WaitGroup } @@ -123,7 +120,7 @@ func NewHeadscale(cfg *types.Config) (*Headscale, error) { return nil, fmt.Errorf("failed to read or create Noise protocol private key: %w", err) } - registrationCache := cache.New( + registrationCache := zcache.New[string, types.Node]( registerCacheExpiration, registerCacheCleanup, ) @@ -138,7 +135,9 @@ func NewHeadscale(cfg *types.Config) (*Headscale, error) { app.db, err = db.NewHeadscaleDatabase( cfg.Database, - cfg.BaseDomain) + cfg.BaseDomain, + registrationCache, + ) if err != nil { return nil, err } @@ -154,16 +153,30 @@ func NewHeadscale(cfg *types.Config) (*Headscale, error) { } }) + var authProvider AuthProvider + authProvider = NewAuthProviderWeb(cfg.ServerURL) if cfg.OIDC.Issuer != "" { - err = app.initOIDC() + ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second) + defer cancel() + oidcProvider, err := NewAuthProviderOIDC( + ctx, + cfg.ServerURL, + &cfg.OIDC, + app.db, + app.nodeNotifier, + app.ipAlloc, + ) if err != nil { if cfg.OIDC.OnlyStartIfOIDCIsAvailable { return nil, err } else { log.Warn().Err(err).Msg("failed to set up OIDC provider, falling back to CLI based authentication") } + } else { + authProvider = oidcProvider } } + app.authProvider = authProvider if app.cfg.DNSConfig != nil && app.cfg.DNSConfig.Proxied { // if MagicDNS // TODO(kradalby): revisit why this takes a list. @@ -429,10 +442,11 @@ func (h *Headscale) createRouter(grpcMux *grpcRuntime.ServeMux) *mux.Router { router.HandleFunc("/health", h.HealthHandler).Methods(http.MethodGet) router.HandleFunc("/key", h.KeyHandler).Methods(http.MethodGet) - router.HandleFunc("/register/{mkey}", h.RegisterWebAPI).Methods(http.MethodGet) + router.HandleFunc("/register/{mkey}", h.authProvider.RegisterHandler).Methods(http.MethodGet) - router.HandleFunc("/oidc/register/{mkey}", h.RegisterOIDC).Methods(http.MethodGet) - router.HandleFunc("/oidc/callback", h.OIDCCallback).Methods(http.MethodGet) + if provider, ok := h.authProvider.(*AuthProviderOIDC); ok { + router.HandleFunc("/oidc/callback", provider.OIDCCallbackHandler).Methods(http.MethodGet) + } router.HandleFunc("/apple", h.AppleConfigMessage).Methods(http.MethodGet) router.HandleFunc("/apple/{platform}", h.ApplePlatformConfig). Methods(http.MethodGet) diff --git a/hscontrol/auth.go b/hscontrol/auth.go index 8b8557ba..67545031 100644 --- a/hscontrol/auth.go +++ b/hscontrol/auth.go @@ -6,7 +6,6 @@ import ( "errors" "fmt" "net/http" - "strings" "time" "github.com/juanfont/headscale/hscontrol/db" @@ -19,6 +18,11 @@ import ( "tailscale.com/types/ptr" ) +type AuthProvider interface { + RegisterHandler(http.ResponseWriter, *http.Request) + AuthURL(key.MachinePublic) string +} + func logAuthFunc( registerRequest tailcfg.RegisterRequest, machineKey key.MachinePublic, @@ -125,7 +129,6 @@ func (h *Headscale) handleRegister( h.registrationCache.Set( machineKey.String(), newNode, - registerCacheExpiration, ) h.handleNewNode(writer, regReq, machineKey) @@ -164,7 +167,7 @@ func (h *Headscale) handleRegister( // https://github.com/tailscale/tailscale/blob/main/tailcfg/tailcfg.go#L648 if !regReq.Expiry.IsZero() && regReq.Expiry.UTC().Before(now) { - h.handleNodeLogOut(writer, *node, machineKey) + h.handleNodeLogOut(writer, *node) return } @@ -172,7 +175,7 @@ func (h *Headscale) handleRegister( // If node is not expired, and it is register, we have a already accepted this node, // let it proceed with a valid registration if !node.IsExpired() { - h.handleNodeWithValidRegistration(writer, *node, machineKey) + h.handleNodeWithValidRegistration(writer, *node) return } @@ -185,7 +188,6 @@ func (h *Headscale) handleRegister( writer, regReq, *node, - machineKey, ) return @@ -198,7 +200,6 @@ func (h *Headscale) handleRegister( writer, regReq, *node, - machineKey, ) return @@ -226,7 +227,6 @@ func (h *Headscale) handleRegister( h.registrationCache.Set( machineKey.String(), *node, - registerCacheExpiration, ) return @@ -386,7 +386,7 @@ func (h *Headscale) handleAuthKey( } } - h.db.Write(func(tx *gorm.DB) error { + err = h.db.Write(func(tx *gorm.DB) error { return db.UsePreAuthKey(tx, pak) }) if err != nil { @@ -447,17 +447,7 @@ func (h *Headscale) handleNewNode( // The node registration is new, redirect the client to the registration URL logTrace("The node seems to be new, sending auth url") - if h.oauth2Config != nil { - resp.AuthURL = fmt.Sprintf( - "%s/oidc/register/%s", - strings.TrimSuffix(h.cfg.ServerURL, "/"), - machineKey.String(), - ) - } else { - resp.AuthURL = fmt.Sprintf("%s/register/%s", - strings.TrimSuffix(h.cfg.ServerURL, "/"), - machineKey.String()) - } + resp.AuthURL = h.authProvider.AuthURL(machineKey) respBody, err := json.Marshal(resp) if err != nil { @@ -480,7 +470,6 @@ func (h *Headscale) handleNewNode( func (h *Headscale) handleNodeLogOut( writer http.ResponseWriter, node types.Node, - machineKey key.MachinePublic, ) { resp := tailcfg.RegisterResponse{} @@ -563,7 +552,6 @@ func (h *Headscale) handleNodeLogOut( func (h *Headscale) handleNodeWithValidRegistration( writer http.ResponseWriter, node types.Node, - machineKey key.MachinePublic, ) { resp := tailcfg.RegisterResponse{} @@ -609,7 +597,6 @@ func (h *Headscale) handleNodeKeyRefresh( writer http.ResponseWriter, registerRequest tailcfg.RegisterRequest, node types.Node, - machineKey key.MachinePublic, ) { resp := tailcfg.RegisterResponse{} @@ -685,15 +672,7 @@ func (h *Headscale) handleNodeExpiredOrLoggedOut( Str("node_key_old", regReq.OldNodeKey.ShortString()). Msg("Node registration has expired or logged out. Sending a auth url to register") - if h.oauth2Config != nil { - resp.AuthURL = fmt.Sprintf("%s/oidc/register/%s", - strings.TrimSuffix(h.cfg.ServerURL, "/"), - machineKey.String()) - } else { - resp.AuthURL = fmt.Sprintf("%s/register/%s", - strings.TrimSuffix(h.cfg.ServerURL, "/"), - machineKey.String()) - } + resp.AuthURL = h.authProvider.AuthURL(machineKey) respBody, err := json.Marshal(resp) if err != nil { diff --git a/hscontrol/db/db.go b/hscontrol/db/db.go index 44faeb91..b7661ab2 100644 --- a/hscontrol/db/db.go +++ b/hscontrol/db/db.go @@ -22,6 +22,7 @@ import ( "gorm.io/gorm/logger" "gorm.io/gorm/schema" "tailscale.com/util/set" + "zgo.at/zcache/v2" ) func init() { @@ -38,8 +39,9 @@ type KV struct { } type HSDatabase struct { - DB *gorm.DB - cfg *types.DatabaseConfig + DB *gorm.DB + cfg *types.DatabaseConfig + regCache *zcache.Cache[string, types.Node] baseDomain string } @@ -49,6 +51,7 @@ type HSDatabase struct { func NewHeadscaleDatabase( cfg types.DatabaseConfig, baseDomain string, + regCache *zcache.Cache[string, types.Node], ) (*HSDatabase, error) { dbConn, err := openDB(cfg) if err != nil { @@ -264,9 +267,6 @@ func NewHeadscaleDatabase( for item, node := range nodes { if node.GivenName == "" { - normalizedHostname, err := util.NormalizeToFQDNRulesConfigFromViper( - node.Hostname, - ) if err != nil { log.Error(). Caller(). @@ -276,7 +276,7 @@ func NewHeadscaleDatabase( } err = tx.Model(nodes[item]).Updates(types.Node{ - GivenName: normalizedHostname, + GivenName: node.Hostname, }).Error if err != nil { log.Error(). @@ -469,6 +469,17 @@ func NewHeadscaleDatabase( // Drop the old table. _ = tx.Migrator().DropTable(&preAuthKeyACLTag{}) + return nil + }, + Rollback: func(db *gorm.DB) error { return nil }, + }, + { + ID: "202407191627", + Migrate: func(tx *gorm.DB) error { + err := tx.AutoMigrate(&types.User{}) + if err != nil { + return err + } return nil }, @@ -482,8 +493,9 @@ func NewHeadscaleDatabase( } db := HSDatabase{ - DB: dbConn, - cfg: &cfg, + DB: dbConn, + cfg: &cfg, + regCache: regCache, baseDomain: baseDomain, } diff --git a/hscontrol/db/db_test.go b/hscontrol/db/db_test.go index d92a73e5..68ea2ac1 100644 --- a/hscontrol/db/db_test.go +++ b/hscontrol/db/db_test.go @@ -9,6 +9,7 @@ import ( "slices" "sort" "testing" + "time" "github.com/google/go-cmp/cmp" "github.com/google/go-cmp/cmp/cmpopts" @@ -16,6 +17,7 @@ import ( "github.com/juanfont/headscale/hscontrol/util" "github.com/stretchr/testify/assert" "gorm.io/gorm" + "zgo.at/zcache/v2" ) func TestMigrations(t *testing.T) { @@ -206,7 +208,7 @@ func TestMigrations(t *testing.T) { Sqlite: types.SqliteConfig{ Path: dbPath, }, - }, "") + }, "", emptyCache()) if err != nil && tt.wantErr != err.Error() { t.Errorf("TestMigrations() unexpected error = %v, wantErr %v", err, tt.wantErr) } @@ -250,3 +252,7 @@ func testCopyOfDatabase(src string) (string, error) { _, err = io.Copy(destination, source) return dst, err } + +func emptyCache() *zcache.Cache[string, types.Node] { + return zcache.New[string, types.Node](time.Minute, time.Hour) +} diff --git a/hscontrol/db/node.go b/hscontrol/db/node.go index a4cd9e0b..12eeeff8 100644 --- a/hscontrol/db/node.go +++ b/hscontrol/db/node.go @@ -12,7 +12,6 @@ import ( "github.com/juanfont/headscale/hscontrol/types" "github.com/juanfont/headscale/hscontrol/util" - "github.com/patrickmn/go-cache" "github.com/puzpuzpuz/xsync/v3" "github.com/rs/zerolog/log" "gorm.io/gorm" @@ -320,26 +319,17 @@ func SetLastSeen(tx *gorm.DB, nodeID types.NodeID, lastSeen time.Time) error { return tx.Model(&types.Node{}).Where("id = ?", nodeID).Update("last_seen", lastSeen).Error } -func RegisterNodeFromAuthCallback( - tx *gorm.DB, - cache *cache.Cache, +func (hsdb *HSDatabase) RegisterNodeFromAuthCallback( mkey key.MachinePublic, - userName string, + userID types.UserID, nodeExpiry *time.Time, registrationMethod string, ipv4 *netip.Addr, ipv6 *netip.Addr, ) (*types.Node, error) { - log.Debug(). - Str("machine_key", mkey.ShortString()). - Str("userName", userName). - Str("registrationMethod", registrationMethod). - Str("expiresAt", fmt.Sprintf("%v", nodeExpiry)). - Msg("Registering node from API/CLI or auth callback") - - if nodeInterface, ok := cache.Get(mkey.String()); ok { - if registrationNode, ok := nodeInterface.(types.Node); ok { - user, err := GetUser(tx, userName) + return Write(hsdb.DB, func(tx *gorm.DB) (*types.Node, error) { + if node, ok := hsdb.regCache.Get(mkey.String()); ok { + user, err := GetUserByID(tx, userID) if err != nil { return nil, fmt.Errorf( "failed to find user in register node from auth callback, %w", @@ -347,37 +337,42 @@ func RegisterNodeFromAuthCallback( ) } + log.Debug(). + Str("machine_key", mkey.ShortString()). + Str("username", user.Username()). + Str("registrationMethod", registrationMethod). + Str("expiresAt", fmt.Sprintf("%v", nodeExpiry)). + Msg("Registering node from API/CLI or auth callback") + // Registration of expired node with different user - if registrationNode.ID != 0 && - registrationNode.UserID != user.ID { + if node.ID != 0 && + node.UserID != user.ID { return nil, ErrDifferentRegisteredUser } - registrationNode.UserID = user.ID - registrationNode.User = *user - registrationNode.RegisterMethod = registrationMethod + node.UserID = user.ID + node.User = *user + node.RegisterMethod = registrationMethod if nodeExpiry != nil { - registrationNode.Expiry = nodeExpiry + node.Expiry = nodeExpiry } node, err := RegisterNode( tx, - registrationNode, + node, ipv4, ipv6, ) if err == nil { - cache.Delete(mkey.String()) + hsdb.regCache.Delete(mkey.String()) } return node, err - } else { - return nil, ErrCouldNotConvertNodeInterface } - } - return nil, ErrNodeNotFoundRegistrationCache + return nil, ErrNodeNotFoundRegistrationCache + }) } func (hsdb *HSDatabase) RegisterNode(node types.Node, ipv4 *netip.Addr, ipv6 *netip.Addr) (*types.Node, error) { @@ -392,7 +387,7 @@ func RegisterNode(tx *gorm.DB, node types.Node, ipv4 *netip.Addr, ipv6 *netip.Ad Str("node", node.Hostname). Str("machine_key", node.MachineKey.ShortString()). Str("node_key", node.NodeKey.ShortString()). - Str("user", node.User.Name). + Str("user", node.User.Username()). Msg("Registering node") // If the node exists and it already has IP(s), we just save it @@ -408,7 +403,7 @@ func RegisterNode(tx *gorm.DB, node types.Node, ipv4 *netip.Addr, ipv6 *netip.Ad Str("node", node.Hostname). Str("machine_key", node.MachineKey.ShortString()). Str("node_key", node.NodeKey.ShortString()). - Str("user", node.User.Name). + Str("user", node.User.Username()). Msg("Node authorized again") return &node, nil @@ -612,18 +607,15 @@ func enableRoutes(tx *gorm.DB, } func generateGivenName(suppliedName string, randomSuffix bool) (string, error) { - normalizedHostname, err := util.NormalizeToFQDNRulesConfigFromViper( - suppliedName, - ) - if err != nil { - return "", err + if len(suppliedName) > util.LabelHostnameLength { + return "", types.ErrHostnameTooLong } if randomSuffix { // Trim if a hostname will be longer than 63 chars after adding the hash. trimmedHostnameLength := util.LabelHostnameLength - NodeGivenNameHashLength - NodeGivenNameTrimSize - if len(normalizedHostname) > trimmedHostnameLength { - normalizedHostname = normalizedHostname[:trimmedHostnameLength] + if len(suppliedName) > trimmedHostnameLength { + suppliedName = suppliedName[:trimmedHostnameLength] } suffix, err := util.GenerateRandomStringDNSSafe(NodeGivenNameHashLength) @@ -631,10 +623,10 @@ func generateGivenName(suppliedName string, randomSuffix bool) (string, error) { return "", err } - normalizedHostname += "-" + suffix + suppliedName += "-" + suffix } - return normalizedHostname, nil + return suppliedName, nil } func isUnqiueName(tx *gorm.DB, name string) (bool, error) { diff --git a/hscontrol/db/preauth_keys.go b/hscontrol/db/preauth_keys.go index feacde61..59bbdf98 100644 --- a/hscontrol/db/preauth_keys.go +++ b/hscontrol/db/preauth_keys.go @@ -23,6 +23,7 @@ var ( ) func (hsdb *HSDatabase) CreatePreAuthKey( + // TODO(kradalby): Should be ID, not name userName string, reusable bool, ephemeral bool, @@ -37,13 +38,14 @@ func (hsdb *HSDatabase) CreatePreAuthKey( // CreatePreAuthKey creates a new PreAuthKey in a user, and returns it. func CreatePreAuthKey( tx *gorm.DB, + // TODO(kradalby): Should be ID, not name userName string, reusable bool, ephemeral bool, expiration *time.Time, aclTags []string, ) (*types.PreAuthKey, error) { - user, err := GetUser(tx, userName) + user, err := GetUserByUsername(tx, userName) if err != nil { return nil, err } @@ -95,7 +97,7 @@ func (hsdb *HSDatabase) ListPreAuthKeys(userName string) ([]types.PreAuthKey, er // ListPreAuthKeys returns the list of PreAuthKeys for a user. func ListPreAuthKeys(tx *gorm.DB, userName string) ([]types.PreAuthKey, error) { - user, err := GetUser(tx, userName) + user, err := GetUserByUsername(tx, userName) if err != nil { return nil, err } diff --git a/hscontrol/db/routes.go b/hscontrol/db/routes.go index fa27ea7c..086261aa 100644 --- a/hscontrol/db/routes.go +++ b/hscontrol/db/routes.go @@ -645,7 +645,7 @@ func EnableAutoApprovedRoutes( Msg("looking up route for autoapproving") for _, approvedAlias := range routeApprovers { - if approvedAlias == node.User.Name { + if approvedAlias == node.User.Username() { approvedRoutes = append(approvedRoutes, advertisedRoute) } else { // TODO(kradalby): figure out how to get this to depend on less stuff diff --git a/hscontrol/db/routes_test.go b/hscontrol/db/routes_test.go index 0e6535f9..5071077c 100644 --- a/hscontrol/db/routes_test.go +++ b/hscontrol/db/routes_test.go @@ -336,6 +336,7 @@ func dbForTest(t *testing.T, testName string) *HSDatabase { }, }, "", + emptyCache(), ) if err != nil { t.Fatalf("setting up database: %s", err) diff --git a/hscontrol/db/suite_test.go b/hscontrol/db/suite_test.go index d546b33d..6cc46d3d 100644 --- a/hscontrol/db/suite_test.go +++ b/hscontrol/db/suite_test.go @@ -59,6 +59,7 @@ func newTestDB() (*HSDatabase, error) { }, }, "", + emptyCache(), ) if err != nil { return nil, err diff --git a/hscontrol/db/users.go b/hscontrol/db/users.go index 1cf8e92f..135276c7 100644 --- a/hscontrol/db/users.go +++ b/hscontrol/db/users.go @@ -49,7 +49,7 @@ func (hsdb *HSDatabase) DestroyUser(name string) error { // DestroyUser destroys a User. Returns error if the User does // not exist or if there are nodes associated with it. func DestroyUser(tx *gorm.DB, name string) error { - user, err := GetUser(tx, name) + user, err := GetUserByUsername(tx, name) if err != nil { return ErrUserNotFound } @@ -90,7 +90,7 @@ func (hsdb *HSDatabase) RenameUser(oldName, newName string) error { // not exist or if another User exists with the new name. func RenameUser(tx *gorm.DB, oldName, newName string) error { var err error - oldUser, err := GetUser(tx, oldName) + oldUser, err := GetUserByUsername(tx, oldName) if err != nil { return err } @@ -98,7 +98,7 @@ func RenameUser(tx *gorm.DB, oldName, newName string) error { if err != nil { return err } - _, err = GetUser(tx, newName) + _, err = GetUserByUsername(tx, newName) if err == nil { return ErrUserExists } @@ -115,13 +115,13 @@ func RenameUser(tx *gorm.DB, oldName, newName string) error { return nil } -func (hsdb *HSDatabase) GetUser(name string) (*types.User, error) { +func (hsdb *HSDatabase) GetUserByName(name string) (*types.User, error) { return Read(hsdb.DB, func(rx *gorm.DB) (*types.User, error) { - return GetUser(rx, name) + return GetUserByUsername(rx, name) }) } -func GetUser(tx *gorm.DB, name string) (*types.User, error) { +func GetUserByUsername(tx *gorm.DB, name string) (*types.User, error) { user := types.User{} if result := tx.First(&user, "name = ?", name); errors.Is( result.Error, @@ -133,6 +133,42 @@ func GetUser(tx *gorm.DB, name string) (*types.User, error) { return &user, nil } +func (hsdb *HSDatabase) GetUserByID(id types.UserID) (*types.User, error) { + return Read(hsdb.DB, func(rx *gorm.DB) (*types.User, error) { + return GetUserByID(rx, id) + }) +} + +func GetUserByID(tx *gorm.DB, id types.UserID) (*types.User, error) { + user := types.User{} + if result := tx.First(&user, "id = ?", id); errors.Is( + result.Error, + gorm.ErrRecordNotFound, + ) { + return nil, ErrUserNotFound + } + + return &user, nil +} + +func (hsdb *HSDatabase) GetUserByOIDCIdentifier(id string) (*types.User, error) { + return Read(hsdb.DB, func(rx *gorm.DB) (*types.User, error) { + return GetUserByOIDCIdentifier(rx, id) + }) +} + +func GetUserByOIDCIdentifier(tx *gorm.DB, id string) (*types.User, error) { + user := types.User{} + if result := tx.First(&user, "provider_identifier = ?", id); errors.Is( + result.Error, + gorm.ErrRecordNotFound, + ) { + return nil, ErrUserNotFound + } + + return &user, nil +} + func (hsdb *HSDatabase) ListUsers() ([]types.User, error) { return Read(hsdb.DB, func(rx *gorm.DB) ([]types.User, error) { return ListUsers(rx) @@ -155,7 +191,7 @@ func ListNodesByUser(tx *gorm.DB, name string) (types.Nodes, error) { if err != nil { return nil, err } - user, err := GetUser(tx, name) + user, err := GetUserByUsername(tx, name) if err != nil { return nil, err } @@ -180,7 +216,7 @@ func AssignNodeToUser(tx *gorm.DB, node *types.Node, username string) error { if err != nil { return err } - user, err := GetUser(tx, username) + user, err := GetUserByUsername(tx, username) if err != nil { return err } diff --git a/hscontrol/db/users_test.go b/hscontrol/db/users_test.go index 0629480c..54399664 100644 --- a/hscontrol/db/users_test.go +++ b/hscontrol/db/users_test.go @@ -20,7 +20,7 @@ func (s *Suite) TestCreateAndDestroyUser(c *check.C) { err = db.DestroyUser("test") c.Assert(err, check.IsNil) - _, err = db.GetUser("test") + _, err = db.GetUserByName("test") c.Assert(err, check.NotNil) } @@ -73,10 +73,10 @@ func (s *Suite) TestRenameUser(c *check.C) { err = db.RenameUser("test", "test-renamed") c.Assert(err, check.IsNil) - _, err = db.GetUser("test") + _, err = db.GetUserByName("test") c.Assert(err, check.Equals, ErrUserNotFound) - _, err = db.GetUser("test-renamed") + _, err = db.GetUserByName("test-renamed") c.Assert(err, check.IsNil) err = db.RenameUser("test-does-not-exit", "test") diff --git a/hscontrol/grpcv1.go b/hscontrol/grpcv1.go index 596748f2..68793716 100644 --- a/hscontrol/grpcv1.go +++ b/hscontrol/grpcv1.go @@ -41,7 +41,7 @@ func (api headscaleV1APIServer) GetUser( ctx context.Context, request *v1.GetUserRequest, ) (*v1.GetUserResponse, error) { - user, err := api.h.db.GetUser(request.GetName()) + user, err := api.h.db.GetUserByName(request.GetName()) if err != nil { return nil, err } @@ -70,7 +70,7 @@ func (api headscaleV1APIServer) RenameUser( return nil, err } - user, err := api.h.db.GetUser(request.GetNewName()) + user, err := api.h.db.GetUserByName(request.GetNewName()) if err != nil { return nil, err } @@ -205,17 +205,18 @@ func (api headscaleV1APIServer) RegisterNode( return nil, err } - node, err := db.Write(api.h.db.DB, func(tx *gorm.DB) (*types.Node, error) { - return db.RegisterNodeFromAuthCallback( - tx, - api.h.registrationCache, - mkey, - request.GetUser(), - nil, - util.RegisterMethodCLI, - ipv4, ipv6, - ) - }) + user, err := api.h.db.GetUserByName(request.GetUser()) + if err != nil { + return nil, fmt.Errorf("looking up user: %w", err) + } + + node, err := api.h.db.RegisterNodeFromAuthCallback( + mkey, + types.UserID(user.ID), + nil, + util.RegisterMethodCLI, + ipv4, ipv6, + ) if err != nil { return nil, err } @@ -774,7 +775,7 @@ func (api headscaleV1APIServer) DebugCreateNode( ctx context.Context, request *v1.DebugCreateNodeRequest, ) (*v1.DebugCreateNodeResponse, error) { - user, err := api.h.db.GetUser(request.GetUser()) + user, err := api.h.db.GetUserByName(request.GetUser()) if err != nil { return nil, err } @@ -823,7 +824,6 @@ func (api headscaleV1APIServer) DebugCreateNode( api.h.registrationCache.Set( mkey.String(), newNode, - registerCacheExpiration, ) return &v1.DebugCreateNodeResponse{Node: newNode.Proto()}, nil diff --git a/hscontrol/handlers.go b/hscontrol/handlers.go index 6efe1984..9287eeff 100644 --- a/hscontrol/handlers.go +++ b/hscontrol/handlers.go @@ -8,6 +8,7 @@ import ( "html/template" "net/http" "strconv" + "strings" "time" "github.com/gorilla/mux" @@ -167,12 +168,29 @@ var registerWebAPITemplate = template.Must( `)) +type AuthProviderWeb struct { + serverURL string +} + +func NewAuthProviderWeb(serverURL string) *AuthProviderWeb { + return &AuthProviderWeb{ + serverURL: serverURL, + } +} + +func (a *AuthProviderWeb) AuthURL(mKey key.MachinePublic) string { + return fmt.Sprintf( + "%s/register/%s", + strings.TrimSuffix(a.serverURL, "/"), + mKey.String()) +} + // RegisterWebAPI shows a simple message in the browser to point to the CLI // Listens in /register/:nkey. // // This is not part of the Tailscale control API, as we could send whatever URL // in the RegisterResponse.AuthURL field. -func (h *Headscale) RegisterWebAPI( +func (a *AuthProviderWeb) RegisterHandler( writer http.ResponseWriter, req *http.Request, ) { @@ -187,7 +205,7 @@ func (h *Headscale) RegisterWebAPI( []byte(machineKeyStr), ) if err != nil { - log.Warn().Err(err).Msg("Failed to parse incoming nodekey") + log.Warn().Err(err).Msg("Failed to parse incoming machinekey") writer.Header().Set("Content-Type", "text/plain; charset=utf-8") writer.WriteHeader(http.StatusBadRequest) diff --git a/hscontrol/mapper/mapper.go b/hscontrol/mapper/mapper.go index 8593e167..20aa674d 100644 --- a/hscontrol/mapper/mapper.go +++ b/hscontrol/mapper/mapper.go @@ -15,7 +15,6 @@ import ( "sync/atomic" "time" - mapset "github.com/deckarep/golang-set/v2" "github.com/juanfont/headscale/hscontrol/db" "github.com/juanfont/headscale/hscontrol/notifier" "github.com/juanfont/headscale/hscontrol/policy" @@ -95,10 +94,10 @@ func generateUserProfiles( node *types.Node, peers types.Nodes, ) []tailcfg.UserProfile { - userMap := make(map[string]types.User) - userMap[node.User.Name] = node.User + userMap := make(map[uint]types.User) + userMap[node.User.ID] = node.User for _, peer := range peers { - userMap[peer.User.Name] = peer.User // not worth checking if already is there + userMap[peer.User.ID] = peer.User // not worth checking if already is there } var profiles []tailcfg.UserProfile @@ -122,32 +121,6 @@ func generateDNSConfig( dnsConfig := cfg.DNSConfig.Clone() - // if MagicDNS is enabled - if dnsConfig.Proxied { - if cfg.DNSUserNameInMagicDNS { - // Only inject the Search Domain of the current user - // shared nodes should use their full FQDN - dnsConfig.Domains = append( - dnsConfig.Domains, - fmt.Sprintf( - "%s.%s", - node.User.Name, - baseDomain, - ), - ) - - userSet := mapset.NewSet[types.User]() - userSet.Add(node.User) - for _, p := range peers { - userSet.Add(p.User) - } - for _, user := range userSet.ToSlice() { - dnsRoute := fmt.Sprintf("%v.%v", user.Name, baseDomain) - dnsConfig.Routes[dnsRoute] = nil - } - } - } - addNextDNSMetadata(dnsConfig.Resolvers, node) return dnsConfig diff --git a/hscontrol/mapper/mapper_test.go b/hscontrol/mapper/mapper_test.go index 24355993..32ea5352 100644 --- a/hscontrol/mapper/mapper_test.go +++ b/hscontrol/mapper/mapper_test.go @@ -12,6 +12,7 @@ import ( "github.com/juanfont/headscale/hscontrol/policy" "github.com/juanfont/headscale/hscontrol/types" "gopkg.in/check.v1" + "gorm.io/gorm" "tailscale.com/net/tsaddr" "tailscale.com/tailcfg" "tailscale.com/types/dnstype" @@ -29,6 +30,9 @@ func (s *Suite) TestGetMapResponseUserProfiles(c *check.C) { Hostname: hostname, UserID: userid, User: types.User{ + Model: gorm.Model{ + ID: userid, + }, Name: username, }, } @@ -73,14 +77,9 @@ func TestDNSConfigMapResponse(t *testing.T) { { magicDNS: true, want: &tailcfg.DNSConfig{ - Routes: map[string][]*dnstype.Resolver{ - "shared1.foobar.headscale.net": {}, - "shared2.foobar.headscale.net": {}, - "shared3.foobar.headscale.net": {}, - }, + Routes: map[string][]*dnstype.Resolver{}, Domains: []string{ "foobar.headscale.net", - "shared1.foobar.headscale.net", }, Proxied: true, }, @@ -128,8 +127,7 @@ func TestDNSConfigMapResponse(t *testing.T) { got := generateDNSConfig( &types.Config{ - DNSConfig: &dnsConfigOrig, - DNSUserNameInMagicDNS: true, + DNSConfig: &dnsConfigOrig, }, baseDomain, nodeInShared1, diff --git a/hscontrol/mapper/tail.go b/hscontrol/mapper/tail.go index a8ccf978..24c521dc 100644 --- a/hscontrol/mapper/tail.go +++ b/hscontrol/mapper/tail.go @@ -76,7 +76,7 @@ func tailNode( keyExpiry = time.Time{} } - hostname, err := node.GetFQDN(cfg, cfg.BaseDomain) + hostname, err := node.GetFQDN(cfg.BaseDomain) if err != nil { return nil, fmt.Errorf("tailNode, failed to create FQDN: %s", err) } diff --git a/hscontrol/oidc.go b/hscontrol/oidc.go index 72fefac3..84267b41 100644 --- a/hscontrol/oidc.go +++ b/hscontrol/oidc.go @@ -17,12 +17,13 @@ import ( "github.com/coreos/go-oidc/v3/oidc" "github.com/gorilla/mux" "github.com/juanfont/headscale/hscontrol/db" + "github.com/juanfont/headscale/hscontrol/notifier" "github.com/juanfont/headscale/hscontrol/types" "github.com/juanfont/headscale/hscontrol/util" "github.com/rs/zerolog/log" "golang.org/x/oauth2" - "gorm.io/gorm" "tailscale.com/types/key" + "zgo.at/zcache/v2" ) const ( @@ -45,49 +46,81 @@ var ( errOIDCNodeKeyMissing = errors.New("could not get node key from cache") ) -type IDTokenClaims struct { - Name string `json:"name,omitempty"` - Groups []string `json:"groups,omitempty"` - Email string `json:"email"` - Username string `json:"preferred_username,omitempty"` +type AuthProviderOIDC struct { + serverURL string + cfg *types.OIDCConfig + db *db.HSDatabase + registrationCache *zcache.Cache[string, key.MachinePublic] + notifier *notifier.Notifier + ipAlloc *db.IPAllocator + + oidcProvider *oidc.Provider + oauth2Config *oauth2.Config } -func (h *Headscale) initOIDC() error { +func NewAuthProviderOIDC( + ctx context.Context, + serverURL string, + cfg *types.OIDCConfig, + db *db.HSDatabase, + notif *notifier.Notifier, + ipAlloc *db.IPAllocator, +) (*AuthProviderOIDC, error) { var err error // grab oidc config if it hasn't been already - if h.oauth2Config == nil { - h.oidcProvider, err = oidc.NewProvider(context.Background(), h.cfg.OIDC.Issuer) - if err != nil { - return fmt.Errorf("creating OIDC provider from issuer config: %w", err) - } - - h.oauth2Config = &oauth2.Config{ - ClientID: h.cfg.OIDC.ClientID, - ClientSecret: h.cfg.OIDC.ClientSecret, - Endpoint: h.oidcProvider.Endpoint(), - RedirectURL: fmt.Sprintf( - "%s/oidc/callback", - strings.TrimSuffix(h.cfg.ServerURL, "/"), - ), - Scopes: h.cfg.OIDC.Scope, - } + oidcProvider, err := oidc.NewProvider(context.Background(), cfg.Issuer) + if err != nil { + return nil, fmt.Errorf("creating OIDC provider from issuer config: %w", err) } - return nil + oauth2Config := &oauth2.Config{ + ClientID: cfg.ClientID, + ClientSecret: cfg.ClientSecret, + Endpoint: oidcProvider.Endpoint(), + RedirectURL: fmt.Sprintf( + "%s/oidc/callback", + strings.TrimSuffix(serverURL, "/"), + ), + Scopes: cfg.Scope, + } + + registrationCache := zcache.New[string, key.MachinePublic]( + registerCacheExpiration, + registerCacheCleanup, + ) + + return &AuthProviderOIDC{ + serverURL: serverURL, + cfg: cfg, + db: db, + registrationCache: registrationCache, + notifier: notif, + ipAlloc: ipAlloc, + + oidcProvider: oidcProvider, + oauth2Config: oauth2Config, + }, nil } -func (h *Headscale) determineTokenExpiration(idTokenExpiration time.Time) time.Time { - if h.cfg.OIDC.UseExpiryFromToken { +func (a *AuthProviderOIDC) AuthURL(mKey key.MachinePublic) string { + return fmt.Sprintf( + "%s/register/%s", + strings.TrimSuffix(a.serverURL, "/"), + mKey.String()) +} + +func (a *AuthProviderOIDC) determineNodeExpiry(idTokenExpiration time.Time) time.Time { + if a.cfg.UseExpiryFromToken { return idTokenExpiration } - return time.Now().Add(h.cfg.OIDC.Expiry) + return time.Now().Add(a.cfg.Expiry) } // RegisterOIDC redirects to the OIDC provider for authentication // Puts NodeKey in cache so the callback can retrieve it using the oidc state param -// Listens in /oidc/register/:mKey. -func (h *Headscale) RegisterOIDC( +// Listens in /register/:mKey. +func (a *AuthProviderOIDC) RegisterHandler( writer http.ResponseWriter, req *http.Request, ) { @@ -108,46 +141,32 @@ func (h *Headscale) RegisterOIDC( []byte(machineKeyStr), ) if err != nil { - log.Warn(). - Err(err). - Msg("Failed to parse incoming nodekey in OIDC registration") - - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("Wrong params")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - + http.Error(writer, err.Error(), http.StatusBadRequest) return } randomBlob := make([]byte, randomByteSize) if _, err := rand.Read(randomBlob); err != nil { - util.LogErr(err, "could not read 16 bytes from rand") - http.Error(writer, "Internal server error", http.StatusInternalServerError) - return } stateStr := hex.EncodeToString(randomBlob)[:32] // place the node key into the state cache, so it can be retrieved later - h.registrationCache.Set( + a.registrationCache.Set( stateStr, machineKey, - registerCacheExpiration, ) // Add any extra parameter provided in the configuration to the Authorize Endpoint request - extras := make([]oauth2.AuthCodeOption, 0, len(h.cfg.OIDC.ExtraParams)) + extras := make([]oauth2.AuthCodeOption, 0, len(a.cfg.ExtraParams)) - for k, v := range h.cfg.OIDC.ExtraParams { + for k, v := range a.cfg.ExtraParams { extras = append(extras, oauth2.SetAuthURLParam(k, v)) } - authURL := h.oauth2Config.AuthCodeURL(stateStr, extras...) + authURL := a.oauth2Config.AuthCodeURL(stateStr, extras...) log.Debug().Msgf("Redirecting to %s for authentication", authURL) http.Redirect(writer, req, authURL, http.StatusFound) @@ -165,216 +184,165 @@ var oidcCallbackTemplate = template.Must( template.New("oidccallback").Parse(oidcCallbackTemplateContent), ) -// OIDCCallback handles the callback from the OIDC endpoint +// OIDCCallbackHandler handles the callback from the OIDC endpoint // Retrieves the nkey from the state cache and adds the node to the users email user // TODO: A confirmation page for new nodes should be added to avoid phishing vulnerabilities // TODO: Add groups information from OIDC tokens into node HostInfo // Listens in /oidc/callback. -func (h *Headscale) OIDCCallback( +func (a *AuthProviderOIDC) OIDCCallbackHandler( writer http.ResponseWriter, req *http.Request, ) { - code, state, err := validateOIDCCallbackParams(writer, req) + code, state, err := extractCodeAndStateParamFromRequest(req) if err != nil { + http.Error(writer, err.Error(), http.StatusBadRequest) return } - rawIDToken, err := h.getIDTokenForOIDCCallback(req.Context(), writer, code, state) + idToken, err := a.extractIDToken(req.Context(), code) if err != nil { + http.Error(writer, err.Error(), http.StatusBadRequest) + return + } + nodeExpiry := a.determineNodeExpiry(idToken.Expiry) + + var claims types.OIDCClaims + if err := idToken.Claims(&claims); err != nil { + http.Error(writer, fmt.Errorf("failed to decode ID token claims: %w", err).Error(), http.StatusInternalServerError) return } - idToken, err := h.verifyIDTokenForOIDCCallback(req.Context(), writer, rawIDToken) - if err != nil { - return - } - idTokenExpiry := h.determineTokenExpiration(idToken.Expiry) - - // TODO: we can use userinfo at some point to grab additional information about the user (groups membership, etc) - // userInfo, err := oidcProvider.UserInfo(context.Background(), oauth2.StaticTokenSource(oauth2Token)) - // if err != nil { - // c.String(http.StatusBadRequest, fmt.Sprintf("Failed to retrieve userinfo")) - // return - // } - - claims, err := extractIDTokenClaims(writer, idToken) + if err := validateOIDCAllowedDomains(a.cfg.AllowedDomains, &claims); err != nil { + http.Error(writer, err.Error(), http.StatusUnauthorized) + return + } + + if err := validateOIDCAllowedGroups(a.cfg.AllowedGroups, &claims); err != nil { + http.Error(writer, err.Error(), http.StatusUnauthorized) + return + } + + if err := validateOIDCAllowedUsers(a.cfg.AllowedUsers, &claims); err != nil { + http.Error(writer, err.Error(), http.StatusUnauthorized) + return + } + + user, err := a.createOrUpdateUserFromClaim(&claims) if err != nil { + http.Error(writer, err.Error(), http.StatusInternalServerError) return } - if err := validateOIDCAllowedDomains(writer, h.cfg.OIDC.AllowedDomains, claims); err != nil { + // Retrieve the node and the machine key from the state cache and + // database. + // If the node exists, then the node should be reauthenticated, + // if the node does not exist, and the machine key exists, then + // this is a new node that should be registered. + node, mKey := a.getMachineKeyFromState(state) + + // Reauthenticate the node if it does exists. + if node != nil { + err := a.reauthenticateNode(node, nodeExpiry) + if err != nil { + http.Error(writer, err.Error(), http.StatusInternalServerError) + return + } + + // TODO(kradalby): replace with go-elem + var content bytes.Buffer + if err := oidcCallbackTemplate.Execute(&content, oidcCallbackTemplateConfig{ + User: user.DisplayNameOrUsername(), + Verb: "Reauthenticated", + }); err != nil { + http.Error(writer, fmt.Errorf("rendering OIDC callback template: %w", err).Error(), http.StatusInternalServerError) + return + } + + writer.Header().Set("Content-Type", "text/html; charset=utf-8") + writer.WriteHeader(http.StatusOK) + _, err = writer.Write(content.Bytes()) + if err != nil { + util.LogErr(err, "Failed to write response") + } + return } - if err := validateOIDCAllowedGroups(writer, h.cfg.OIDC.AllowedGroups, claims); err != nil { + // Register the node if it does not exist. + if mKey != nil { + if err := a.registerNode(user, mKey, nodeExpiry); err != nil { + http.Error(writer, err.Error(), http.StatusInternalServerError) + return + } + + content, err := renderOIDCCallbackTemplate(user) + if err != nil { + http.Error(writer, err.Error(), http.StatusInternalServerError) + return + } + + writer.Header().Set("Content-Type", "text/html; charset=utf-8") + writer.WriteHeader(http.StatusOK) + if _, err := writer.Write(content.Bytes()); err != nil { + util.LogErr(err, "Failed to write response") + } + return } - if err := validateOIDCAllowedUsers(writer, h.cfg.OIDC.AllowedUsers, claims); err != nil { - return - } - - machineKey, nodeExists, err := h.validateNodeForOIDCCallback( - writer, - state, - claims, - idTokenExpiry, - ) - if err != nil || nodeExists { - return - } - - userName, err := getUserName(writer, claims, h.cfg.OIDC.StripEmaildomain) - if err != nil { - return - } - - // register the node if it's new - log.Debug().Msg("Registering new node after successful callback") - - user, err := h.findOrCreateNewUserForOIDCCallback(writer, userName) - if err != nil { - return - } - - if err := h.registerNodeForOIDCCallback(writer, user, machineKey, idTokenExpiry); err != nil { - return - } - - content, err := renderOIDCCallbackTemplate(writer, claims) - if err != nil { - return - } - - writer.Header().Set("Content-Type", "text/html; charset=utf-8") - writer.WriteHeader(http.StatusOK) - if _, err := writer.Write(content.Bytes()); err != nil { - util.LogErr(err, "Failed to write response") - } + // Neither node nor machine key was found in the state cache meaning + // that we could not reauth nor register the node. + http.Error(writer, err.Error(), http.StatusInternalServerError) + return } -func validateOIDCCallbackParams( - writer http.ResponseWriter, +func extractCodeAndStateParamFromRequest( req *http.Request, ) (string, string, error) { code := req.URL.Query().Get("code") state := req.URL.Query().Get("state") if code == "" || state == "" { - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("Wrong params")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - return "", "", errEmptyOIDCCallbackParams } return code, state, nil } -func (h *Headscale) getIDTokenForOIDCCallback( +// extractIDToken takes the code parameter from the callback +// and extracts the ID token from the oauth2 token. +func (a *AuthProviderOIDC) extractIDToken( ctx context.Context, - writer http.ResponseWriter, - code, state string, -) (string, error) { - oauth2Token, err := h.oauth2Config.Exchange(ctx, code) - if err != nil { - util.LogErr(err, "Could not exchange code for token") - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, werr := writer.Write([]byte("Could not exchange code for token")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } - - return "", err - } - - log.Trace(). - Caller(). - Str("code", code). - Str("state", state). - Msg("Got oidc callback") - - rawIDToken, rawIDTokenOK := oauth2Token.Extra("id_token").(string) - if !rawIDTokenOK { - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("Could not extract ID Token")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - - return "", errNoOIDCIDToken - } - - return rawIDToken, nil -} - -func (h *Headscale) verifyIDTokenForOIDCCallback( - ctx context.Context, - writer http.ResponseWriter, - rawIDToken string, + code string, ) (*oidc.IDToken, error) { - verifier := h.oidcProvider.Verifier(&oidc.Config{ClientID: h.cfg.OIDC.ClientID}) + oauth2Token, err := a.oauth2Config.Exchange(ctx, code) + if err != nil { + return nil, fmt.Errorf("could not exchange code for token: %w", err) + } + + rawIDToken, ok := oauth2Token.Extra("id_token").(string) + if !ok { + return nil, errNoOIDCIDToken + } + + verifier := a.oidcProvider.Verifier(&oidc.Config{ClientID: a.cfg.ClientID}) idToken, err := verifier.Verify(ctx, rawIDToken) if err != nil { - util.LogErr(err, "failed to verify id token") - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, werr := writer.Write([]byte("Failed to verify id token")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } - - return nil, err + return nil, fmt.Errorf("failed to verify ID token: %w", err) } return idToken, nil } -func extractIDTokenClaims( - writer http.ResponseWriter, - idToken *oidc.IDToken, -) (*IDTokenClaims, error) { - var claims IDTokenClaims - if err := idToken.Claims(&claims); err != nil { - util.LogErr(err, "Failed to decode id token claims") - - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, werr := writer.Write([]byte("Failed to decode id token claims")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } - - return nil, err - } - - return &claims, nil -} - // validateOIDCAllowedDomains checks that if AllowedDomains is provided, // that the authenticated principal ends with @. func validateOIDCAllowedDomains( - writer http.ResponseWriter, allowedDomains []string, - claims *IDTokenClaims, + claims *types.OIDCClaims, ) error { if len(allowedDomains) > 0 { if at := strings.LastIndex(claims.Email, "@"); at < 0 || !slices.Contains(allowedDomains, claims.Email[at+1:]) { - log.Trace().Msg("authenticated principal does not match any allowed domain") - - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("unauthorized principal (domain mismatch)")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - return errOIDCAllowedDomains } } @@ -387,9 +355,8 @@ func validateOIDCAllowedDomains( // claims.Groups can be populated by adding a client scope named // 'groups' that contains group membership. func validateOIDCAllowedGroups( - writer http.ResponseWriter, allowedGroups []string, - claims *IDTokenClaims, + claims *types.OIDCClaims, ) error { if len(allowedGroups) > 0 { for _, group := range allowedGroups { @@ -398,14 +365,6 @@ func validateOIDCAllowedGroups( } } - log.Trace().Msg("authenticated principal not in any allowed groups") - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("unauthorized principal (allowed groups)")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - return errOIDCAllowedGroups } @@ -415,249 +374,129 @@ func validateOIDCAllowedGroups( // validateOIDCAllowedUsers checks that if AllowedUsers is provided, // that the authenticated principal is part of that list. func validateOIDCAllowedUsers( - writer http.ResponseWriter, allowedUsers []string, - claims *IDTokenClaims, + claims *types.OIDCClaims, ) error { if len(allowedUsers) > 0 && !slices.Contains(allowedUsers, claims.Email) { log.Trace().Msg("authenticated principal does not match any allowed user") - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("unauthorized principal (user mismatch)")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - return errOIDCAllowedUsers } return nil } -// validateNode retrieves node information if it exist -// The error is not important, because if it does not -// exist, then this is a new node and we will move -// on to registration. -func (h *Headscale) validateNodeForOIDCCallback( - writer http.ResponseWriter, - state string, - claims *IDTokenClaims, - expiry time.Time, -) (*key.MachinePublic, bool, error) { - // retrieve nodekey from state cache - machineKeyIf, machineKeyFound := h.registrationCache.Get(state) - if !machineKeyFound { - log.Trace(). - Msg("requested node state key expired before authorisation completed") - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("state has expired")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - - return nil, false, errOIDCNodeKeyMissing - } - - var machineKey key.MachinePublic - machineKey, machineKeyOK := machineKeyIf.(key.MachinePublic) - if !machineKeyOK { - log.Trace(). - Interface("got", machineKeyIf). - Msg("requested node state key is not a nodekey") - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusBadRequest) - _, err := writer.Write([]byte("state is invalid")) - if err != nil { - util.LogErr(err, "Failed to write response") - } - - return nil, false, errOIDCInvalidNodeState +// getMachineKeyFromState retrieves the machine key from the state +// cache. If the machine key is found, it will try retrieve the +// node information from the database. +func (a *AuthProviderOIDC) getMachineKeyFromState(state string) (*types.Node, *key.MachinePublic) { + machineKey, ok := a.registrationCache.Get(state) + if !ok { + return nil, nil } // retrieve node information if it exist // The error is not important, because if it does not // exist, then this is a new node and we will move // on to registration. - node, _ := h.db.GetNodeByMachineKey(machineKey) + node, _ := a.db.GetNodeByMachineKey(machineKey) - if node != nil { - log.Trace(). - Caller(). - Str("node", node.Hostname). - Msg("node already registered, reauthenticating") - - err := h.db.NodeSetExpiry(node.ID, expiry) - if err != nil { - util.LogErr(err, "Failed to refresh node") - http.Error( - writer, - "Failed to refresh node", - http.StatusInternalServerError, - ) - - return nil, true, err - } - log.Debug(). - Str("node", node.Hostname). - Str("expiresAt", fmt.Sprintf("%v", expiry)). - Msg("successfully refreshed node") - - var content bytes.Buffer - if err := oidcCallbackTemplate.Execute(&content, oidcCallbackTemplateConfig{ - User: claims.Email, - Verb: "Reauthenticated", - }); err != nil { - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusInternalServerError) - _, werr := writer.Write([]byte("Could not render OIDC callback template")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } - - return nil, true, fmt.Errorf("rendering OIDC callback template: %w", err) - } - - writer.Header().Set("Content-Type", "text/html; charset=utf-8") - writer.WriteHeader(http.StatusOK) - _, err = writer.Write(content.Bytes()) - if err != nil { - util.LogErr(err, "Failed to write response") - } - - ctx := types.NotifyCtx(context.Background(), "oidc-expiry-self", node.Hostname) - h.nodeNotifier.NotifyByNodeID( - ctx, - types.StateUpdate{ - Type: types.StateSelfUpdate, - ChangeNodes: []types.NodeID{node.ID}, - }, - node.ID, - ) - - ctx = types.NotifyCtx(context.Background(), "oidc-expiry-peers", node.Hostname) - h.nodeNotifier.NotifyWithIgnore(ctx, types.StateUpdateExpire(node.ID, expiry), node.ID) - - return nil, true, nil - } - - return &machineKey, false, nil + return node, &machineKey } -func getUserName( - writer http.ResponseWriter, - claims *IDTokenClaims, - stripEmaildomain bool, -) (string, error) { - userName, err := util.NormalizeToFQDNRules( - claims.Email, - stripEmaildomain, - ) +// reauthenticateNode updates the node expiry in the database +// and notifies the node and its peers about the change. +func (a *AuthProviderOIDC) reauthenticateNode( + node *types.Node, + expiry time.Time, +) error { + err := a.db.NodeSetExpiry(node.ID, expiry) if err != nil { - util.LogErr(err, "couldn't normalize email") - - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusInternalServerError) - _, werr := writer.Write([]byte("couldn't normalize email")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } - - return "", err + return err } - return userName, nil + ctx := types.NotifyCtx(context.Background(), "oidc-expiry-self", node.Hostname) + a.notifier.NotifyByNodeID( + ctx, + types.StateUpdate{ + Type: types.StateSelfUpdate, + ChangeNodes: []types.NodeID{node.ID}, + }, + node.ID, + ) + + ctx = types.NotifyCtx(context.Background(), "oidc-expiry-peers", node.Hostname) + a.notifier.NotifyWithIgnore(ctx, types.StateUpdateExpire(node.ID, expiry), node.ID) + + return nil } -func (h *Headscale) findOrCreateNewUserForOIDCCallback( - writer http.ResponseWriter, - userName string, +func (a *AuthProviderOIDC) createOrUpdateUserFromClaim( + claims *types.OIDCClaims, ) (*types.User, error) { - user, err := h.db.GetUser(userName) - if errors.Is(err, db.ErrUserNotFound) { - user, err = h.db.CreateUser(userName) - if err != nil { - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusInternalServerError) - _, werr := writer.Write([]byte("could not create user")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } + var user *types.User + var err error + user, err = a.db.GetUserByOIDCIdentifier(claims.Sub) + if err != nil && !errors.Is(err, db.ErrUserNotFound) { + return nil, fmt.Errorf("creating or updating user: %w", err) + } - return nil, fmt.Errorf("creating new user: %w", err) - } - } else if err != nil { - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusInternalServerError) - _, werr := writer.Write([]byte("could not find or create user")) - if werr != nil { - util.LogErr(err, "Failed to write response") + // This check is for legacy, if the user cannot be found by the OIDC identifier + // look it up by username. This should only be needed once. + if user == nil { + user, err = a.db.GetUserByName(claims.Username) + if err != nil && !errors.Is(err, db.ErrUserNotFound) { + return nil, fmt.Errorf("creating or updating user: %w", err) } - return nil, fmt.Errorf("find or create user: %w", err) + // if the user is still not found, create a new empty user. + if user == nil { + user = &types.User{} + } + } + + user.FromClaim(claims) + err = a.db.DB.Save(user).Error + if err != nil { + return nil, fmt.Errorf("creating or updating user: %w", err) } return user, nil } -func (h *Headscale) registerNodeForOIDCCallback( - writer http.ResponseWriter, +func (a *AuthProviderOIDC) registerNode( user *types.User, machineKey *key.MachinePublic, expiry time.Time, ) error { - ipv4, ipv6, err := h.ipAlloc.Next() + ipv4, ipv6, err := a.ipAlloc.Next() if err != nil { return err } - if err := h.db.Write(func(tx *gorm.DB) error { - if _, err := db.RegisterNodeFromAuthCallback( - // TODO(kradalby): find a better way to use the cache across modules - tx, - h.registrationCache, - *machineKey, - user.Name, - &expiry, - util.RegisterMethodOIDC, - ipv4, ipv6, - ); err != nil { - return err - } - - return nil - }); err != nil { - util.LogErr(err, "could not register node") - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusInternalServerError) - _, werr := writer.Write([]byte("could not register node")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } - - return err + if _, err := a.db.RegisterNodeFromAuthCallback( + *machineKey, + types.UserID(user.ID), + &expiry, + util.RegisterMethodOIDC, + ipv4, ipv6, + ); err != nil { + return fmt.Errorf("could not register node: %w", err) } return nil } +// TODO(kradalby): +// Rewrite in elem-go func renderOIDCCallbackTemplate( - writer http.ResponseWriter, - claims *IDTokenClaims, + user *types.User, ) (*bytes.Buffer, error) { var content bytes.Buffer if err := oidcCallbackTemplate.Execute(&content, oidcCallbackTemplateConfig{ - User: claims.Email, + User: user.DisplayNameOrUsername(), Verb: "Authenticated", }); err != nil { - writer.Header().Set("Content-Type", "text/plain; charset=utf-8") - writer.WriteHeader(http.StatusInternalServerError) - _, werr := writer.Write([]byte("Could not render OIDC callback template")) - if werr != nil { - util.LogErr(err, "Failed to write response") - } - return nil, fmt.Errorf("rendering OIDC callback template: %w", err) } diff --git a/hscontrol/policy/acls.go b/hscontrol/policy/acls.go index 7a552456..ff73985b 100644 --- a/hscontrol/policy/acls.go +++ b/hscontrol/policy/acls.go @@ -743,15 +743,7 @@ func (pol *ACLPolicy) expandUsersFromGroup( ErrInvalidGroup, ) } - grp, err := util.NormalizeToFQDNRulesConfigFromViper(group) - if err != nil { - return []string{}, fmt.Errorf( - "failed to normalize group %q, err: %w", - group, - ErrInvalidGroup, - ) - } - users = append(users, grp) + users = append(users, group) } return users, nil @@ -940,7 +932,7 @@ func (pol *ACLPolicy) TagsOfNode( } var found bool for _, owner := range owners { - if node.User.Name == owner { + if node.User.Username() == owner { found = true } } @@ -964,7 +956,7 @@ func (pol *ACLPolicy) TagsOfNode( func filterNodesByUser(nodes types.Nodes, user string) types.Nodes { var out types.Nodes for _, node := range nodes { - if node.User.Name == user { + if node.User.Username() == user { out = append(out, node) } } diff --git a/hscontrol/policy/acls_test.go b/hscontrol/policy/acls_test.go index cfcba77a..1c6e4de8 100644 --- a/hscontrol/policy/acls_test.go +++ b/hscontrol/policy/acls_test.go @@ -635,25 +635,6 @@ func Test_expandGroup(t *testing.T) { want: []string{}, wantErr: true, }, - { - name: "Expand emails in group strip domains", - field: field{ - pol: ACLPolicy{ - Groups: Groups{ - "group:admin": []string{ - "joe.bar@gmail.com", - "john.doe@yahoo.fr", - }, - }, - }, - }, - args: args{ - group: "group:admin", - stripEmail: true, - }, - want: []string{"joe.bar", "john.doe"}, - wantErr: false, - }, { name: "Expand emails in group", field: field{ @@ -669,7 +650,7 @@ func Test_expandGroup(t *testing.T) { args: args{ group: "group:admin", }, - want: []string{"joe.bar.gmail.com", "john.doe.yahoo.fr"}, + want: []string{"joe.bar@gmail.com", "john.doe@yahoo.fr"}, wantErr: false, }, } diff --git a/hscontrol/suite_test.go b/hscontrol/suite_test.go index b03e5c98..fb64d18e 100644 --- a/hscontrol/suite_test.go +++ b/hscontrol/suite_test.go @@ -46,9 +46,7 @@ func (s *Suite) ResetDB(c *check.C) { Path: tmpDir + "/headscale_test.db", }, }, - OIDC: types.OIDCConfig{ - StripEmaildomain: false, - }, + OIDC: types.OIDCConfig{}, } app, err = NewHeadscale(&cfg) diff --git a/hscontrol/types/config.go b/hscontrol/types/config.go index 50ce2f07..f02b9758 100644 --- a/hscontrol/types/config.go +++ b/hscontrol/types/config.go @@ -71,8 +71,7 @@ type Config struct { ACMEURL string ACMEEmail string - DNSConfig *tailcfg.DNSConfig - DNSUserNameInMagicDNS bool + DNSConfig *tailcfg.DNSConfig UnixSocket string UnixSocketPermission fs.FileMode @@ -90,12 +89,11 @@ type Config struct { } type DNSConfig struct { - MagicDNS bool `mapstructure:"magic_dns"` - BaseDomain string `mapstructure:"base_domain"` - Nameservers Nameservers - SearchDomains []string `mapstructure:"search_domains"` - ExtraRecords []tailcfg.DNSRecord `mapstructure:"extra_records"` - UserNameInMagicDNS bool `mapstructure:"use_username_in_magic_dns"` + MagicDNS bool `mapstructure:"magic_dns"` + BaseDomain string `mapstructure:"base_domain"` + Nameservers Nameservers + SearchDomains []string `mapstructure:"search_domains"` + ExtraRecords []tailcfg.DNSRecord `mapstructure:"extra_records"` } type Nameservers struct { @@ -164,7 +162,6 @@ type OIDCConfig struct { AllowedDomains []string AllowedUsers []string AllowedGroups []string - StripEmaildomain bool Expiry time.Duration UseExpiryFromToken bool } @@ -274,7 +271,6 @@ func LoadConfig(path string, isFile bool) error { viper.SetDefault("database.sqlite.write_ahead_log", true) viper.SetDefault("oidc.scope", []string{oidc.ScopeOpenID, "profile", "email"}) - viper.SetDefault("oidc.strip_email_domain", true) viper.SetDefault("oidc.only_start_if_oidc_is_available", true) viper.SetDefault("oidc.expiry", "180d") viper.SetDefault("oidc.use_expiry_from_token", false) @@ -321,8 +317,22 @@ func validateServerConfig() error { depr.warn("dns_config.use_username_in_magic_dns") depr.warn("dns.use_username_in_magic_dns") + depr.fatal("oidc.strip_email_domain") + depr.fatal("dns.use_username_in_musername_in_magic_dns") + depr.fatal("dns_config.use_username_in_musername_in_magic_dns") + depr.Log() + for _, removed := range []string{ + "oidc.strip_email_domain", + "dns_config.use_username_in_musername_in_magic_dns", + } { + if viper.IsSet(removed) { + log.Fatal(). + Msgf("Fatal config error: %s has been removed. Please remove it from your config file", removed) + } + } + // Collect any validation errors and return them all at once var errorText string if (viper.GetString("tls_letsencrypt_hostname") != "") && @@ -572,12 +582,9 @@ func dns() (DNSConfig, error) { if err != nil { return DNSConfig{}, fmt.Errorf("unmarshaling dns extra records: %w", err) } - dns.ExtraRecords = extraRecords } - dns.UserNameInMagicDNS = viper.GetBool("dns.use_username_in_magic_dns") - return dns, nil } @@ -780,7 +787,12 @@ func LoadServerConfig() (*Config, error) { case string(IPAllocationStrategyRandom): alloc = IPAllocationStrategyRandom default: - return nil, fmt.Errorf("config error, prefixes.allocation is set to %s, which is not a valid strategy, allowed options: %s, %s", allocStr, IPAllocationStrategySequential, IPAllocationStrategyRandom) + return nil, fmt.Errorf( + "config error, prefixes.allocation is set to %s, which is not a valid strategy, allowed options: %s, %s", + allocStr, + IPAllocationStrategySequential, + IPAllocationStrategyRandom, + ) } dnsConfig, err := dns() @@ -814,10 +826,11 @@ func LoadServerConfig() (*Config, error) { // - DERP run on their own domains // - Control plane runs on login.tailscale.com/controlplane.tailscale.com // - MagicDNS (BaseDomain) for users is on a *.ts.net domain per tailnet (e.g. tail-scale.ts.net) - // - // TODO(kradalby): remove dnsConfig.UserNameInMagicDNS check when removed. - if !dnsConfig.UserNameInMagicDNS && dnsConfig.BaseDomain != "" && strings.Contains(serverURL, dnsConfig.BaseDomain) { - return nil, errors.New("server_url cannot contain the base_domain, this will cause the headscale server and embedded DERP to become unreachable from the Tailscale node.") + if dnsConfig.BaseDomain != "" && + strings.Contains(serverURL, dnsConfig.BaseDomain) { + return nil, errors.New( + "server_url cannot contain the base_domain, this will cause the headscale server and embedded DERP to become unreachable from the Tailscale node.", + ) } return &Config{ @@ -847,8 +860,7 @@ func LoadServerConfig() (*Config, error) { TLS: tlsConfig(), - DNSConfig: dnsToTailcfgDNS(dnsConfig), - DNSUserNameInMagicDNS: dnsConfig.UserNameInMagicDNS, + DNSConfig: dnsToTailcfgDNS(dnsConfig), ACMEEmail: viper.GetString("acme_email"), ACMEURL: viper.GetString("acme_url"), @@ -860,15 +872,14 @@ func LoadServerConfig() (*Config, error) { OnlyStartIfOIDCIsAvailable: viper.GetBool( "oidc.only_start_if_oidc_is_available", ), - Issuer: viper.GetString("oidc.issuer"), - ClientID: viper.GetString("oidc.client_id"), - ClientSecret: oidcClientSecret, - Scope: viper.GetStringSlice("oidc.scope"), - ExtraParams: viper.GetStringMapString("oidc.extra_params"), - AllowedDomains: viper.GetStringSlice("oidc.allowed_domains"), - AllowedUsers: viper.GetStringSlice("oidc.allowed_users"), - AllowedGroups: viper.GetStringSlice("oidc.allowed_groups"), - StripEmaildomain: viper.GetBool("oidc.strip_email_domain"), + Issuer: viper.GetString("oidc.issuer"), + ClientID: viper.GetString("oidc.client_id"), + ClientSecret: oidcClientSecret, + Scope: viper.GetStringSlice("oidc.scope"), + ExtraParams: viper.GetStringMapString("oidc.extra_params"), + AllowedDomains: viper.GetStringSlice("oidc.allowed_domains"), + AllowedUsers: viper.GetStringSlice("oidc.allowed_users"), + AllowedGroups: viper.GetStringSlice("oidc.allowed_groups"), Expiry: func() time.Duration { // if set to 0, we assume no expiry if value := viper.GetString("oidc.expiry"); value == "0" { @@ -903,9 +914,11 @@ func LoadServerConfig() (*Config, error) { // TODO(kradalby): Document these settings when more stable Tuning: Tuning{ - NotifierSendTimeout: viper.GetDuration("tuning.notifier_send_timeout"), - BatchChangeDelay: viper.GetDuration("tuning.batch_change_delay"), - NodeMapSessionBufferedChanSize: viper.GetInt("tuning.node_mapsession_buffered_chan_size"), + NotifierSendTimeout: viper.GetDuration("tuning.notifier_send_timeout"), + BatchChangeDelay: viper.GetDuration("tuning.batch_change_delay"), + NodeMapSessionBufferedChanSize: viper.GetInt( + "tuning.node_mapsession_buffered_chan_size", + ), }, }, nil } @@ -921,14 +934,26 @@ func (d *deprecator) warnWithAlias(newKey, oldKey string) { // NOTE: RegisterAlias is called with NEW KEY -> OLD KEY viper.RegisterAlias(newKey, oldKey) if viper.IsSet(oldKey) { - d.warns.Add(fmt.Sprintf("The %q configuration key is deprecated. Please use %q instead. %q will be removed in the future.", oldKey, newKey, oldKey)) + d.warns.Add( + fmt.Sprintf( + "The %q configuration key is deprecated. Please use %q instead. %q will be removed in the future.", + oldKey, + newKey, + oldKey, + ), + ) } } // fatal deprecates and adds an entry to the fatal list of options if the oldKey is set. -func (d *deprecator) fatal(newKey, oldKey string) { +func (d *deprecator) fatal(oldKey string) { if viper.IsSet(oldKey) { - d.fatals.Add(fmt.Sprintf("The %q configuration key is deprecated. Please use %q instead. %q has been removed.", oldKey, newKey, oldKey)) + d.fatals.Add( + fmt.Sprintf( + "The %q configuration key has been removed. Please see the changelog for more details.", + oldKey, + ), + ) } } @@ -936,7 +961,14 @@ func (d *deprecator) fatal(newKey, oldKey string) { // If the new key is set, a warning is emitted instead. func (d *deprecator) fatalIfNewKeyIsNotUsed(newKey, oldKey string) { if viper.IsSet(oldKey) && !viper.IsSet(newKey) { - d.fatals.Add(fmt.Sprintf("The %q configuration key is deprecated. Please use %q instead. %q has been removed.", oldKey, newKey, oldKey)) + d.fatals.Add( + fmt.Sprintf( + "The %q configuration key is deprecated. Please use %q instead. %q has been removed.", + oldKey, + newKey, + oldKey, + ), + ) } else if viper.IsSet(oldKey) { d.warns.Add(fmt.Sprintf("The %q configuration key is deprecated. Please use %q instead. %q has been removed.", oldKey, newKey, oldKey)) } @@ -945,14 +977,26 @@ func (d *deprecator) fatalIfNewKeyIsNotUsed(newKey, oldKey string) { // warn deprecates and adds an option to log a warning if the oldKey is set. func (d *deprecator) warnNoAlias(newKey, oldKey string) { if viper.IsSet(oldKey) { - d.warns.Add(fmt.Sprintf("The %q configuration key is deprecated. Please use %q instead. %q has been removed.", oldKey, newKey, oldKey)) + d.warns.Add( + fmt.Sprintf( + "The %q configuration key is deprecated. Please use %q instead. %q has been removed.", + oldKey, + newKey, + oldKey, + ), + ) } } // warn deprecates and adds an entry to the warn list of options if the oldKey is set. func (d *deprecator) warn(oldKey string) { if viper.IsSet(oldKey) { - d.warns.Add(fmt.Sprintf("The %q configuration key is deprecated and has been removed. Please see the changelog for more details.", oldKey)) + d.warns.Add( + fmt.Sprintf( + "The %q configuration key is deprecated and has been removed. Please see the changelog for more details.", + oldKey, + ), + ) } } diff --git a/hscontrol/types/config_test.go b/hscontrol/types/config_test.go index e6e8d6c2..70c0ce7a 100644 --- a/hscontrol/types/config_test.go +++ b/hscontrol/types/config_test.go @@ -42,8 +42,7 @@ func TestReadConfig(t *testing.T) { {Name: "grafana.myvpn.example.com", Type: "A", Value: "100.64.0.3"}, {Name: "prometheus.myvpn.example.com", Type: "A", Value: "100.64.0.4"}, }, - SearchDomains: []string{"test.com", "bar.com"}, - UserNameInMagicDNS: true, + SearchDomains: []string{"test.com", "bar.com"}, }, }, { @@ -99,8 +98,7 @@ func TestReadConfig(t *testing.T) { {Name: "grafana.myvpn.example.com", Type: "A", Value: "100.64.0.3"}, {Name: "prometheus.myvpn.example.com", Type: "A", Value: "100.64.0.4"}, }, - SearchDomains: []string{"test.com", "bar.com"}, - UserNameInMagicDNS: true, + SearchDomains: []string{"test.com", "bar.com"}, }, }, { @@ -234,11 +232,10 @@ func TestReadConfigFromEnv(t *testing.T) { { name: "unmarshal-dns-full-config", configEnv: map[string]string{ - "HEADSCALE_DNS_MAGIC_DNS": "true", - "HEADSCALE_DNS_BASE_DOMAIN": "example.com", - "HEADSCALE_DNS_NAMESERVERS_GLOBAL": `1.1.1.1 8.8.8.8`, - "HEADSCALE_DNS_SEARCH_DOMAINS": "test.com bar.com", - "HEADSCALE_DNS_USE_USERNAME_IN_MAGIC_DNS": "true", + "HEADSCALE_DNS_MAGIC_DNS": "true", + "HEADSCALE_DNS_BASE_DOMAIN": "example.com", + "HEADSCALE_DNS_NAMESERVERS_GLOBAL": `1.1.1.1 8.8.8.8`, + "HEADSCALE_DNS_SEARCH_DOMAINS": "test.com bar.com", // TODO(kradalby): Figure out how to pass these as env vars // "HEADSCALE_DNS_NAMESERVERS_SPLIT": `{foo.bar.com: ["1.1.1.1"]}`, @@ -266,8 +263,7 @@ func TestReadConfigFromEnv(t *testing.T) { ExtraRecords: []tailcfg.DNSRecord{ // {Name: "prometheus.myvpn.example.com", Type: "A", Value: "100.64.0.4"}, }, - SearchDomains: []string{"test.com", "bar.com"}, - UserNameInMagicDNS: true, + SearchDomains: []string{"test.com", "bar.com"}, }, }, } diff --git a/hscontrol/types/node.go b/hscontrol/types/node.go index 0eb937a1..6e6fd9a5 100644 --- a/hscontrol/types/node.go +++ b/hscontrol/types/node.go @@ -253,7 +253,7 @@ func (node *Node) Proto() *v1.Node { return nodeProto } -func (node *Node) GetFQDN(cfg *Config, baseDomain string) (string, error) { +func (node *Node) GetFQDN(baseDomain string) (string, error) { if node.GivenName == "" { return "", fmt.Errorf("failed to create valid FQDN: %w", ErrNodeHasNoGivenName) } @@ -268,19 +268,6 @@ func (node *Node) GetFQDN(cfg *Config, baseDomain string) (string, error) { ) } - if cfg.DNSUserNameInMagicDNS { - if node.User.Name == "" { - return "", fmt.Errorf("failed to create valid FQDN: %w", ErrNodeUserHasNoName) - } - - hostname = fmt.Sprintf( - "%s.%s.%s", - node.GivenName, - node.User.Name, - baseDomain, - ) - } - if len(hostname) > MaxHostnameLength { return "", fmt.Errorf( "failed to create valid FQDN (%s): %w", diff --git a/hscontrol/types/node_test.go b/hscontrol/types/node_test.go index 885edf5d..1d0e7939 100644 --- a/hscontrol/types/node_test.go +++ b/hscontrol/types/node_test.go @@ -1,7 +1,9 @@ package types import ( + "fmt" "net/netip" + "strings" "testing" "github.com/google/go-cmp/cmp" @@ -127,76 +129,10 @@ func TestNodeFQDN(t *testing.T) { tests := []struct { name string node Node - cfg Config domain string want string wantErr string }{ - { - name: "all-set-with-username", - node: Node{ - GivenName: "test", - User: User{ - Name: "user", - }, - }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: true, - }, - DNSUserNameInMagicDNS: true, - }, - domain: "example.com", - want: "test.user.example.com", - }, - { - name: "no-given-name-with-username", - node: Node{ - User: User{ - Name: "user", - }, - }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: true, - }, - DNSUserNameInMagicDNS: true, - }, - domain: "example.com", - wantErr: "failed to create valid FQDN: node has no given name", - }, - { - name: "no-user-name-with-username", - node: Node{ - GivenName: "test", - User: User{}, - }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: true, - }, - DNSUserNameInMagicDNS: true, - }, - domain: "example.com", - wantErr: "failed to create valid FQDN: node user has no name", - }, - { - name: "no-magic-dns-with-username", - node: Node{ - GivenName: "test", - User: User{ - Name: "user", - }, - }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: false, - }, - DNSUserNameInMagicDNS: true, - }, - domain: "example.com", - want: "test.user.example.com", - }, { name: "no-dnsconfig-with-username", node: Node{ @@ -216,12 +152,6 @@ func TestNodeFQDN(t *testing.T) { Name: "user", }, }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: true, - }, - DNSUserNameInMagicDNS: false, - }, domain: "example.com", want: "test.example.com", }, @@ -232,46 +162,16 @@ func TestNodeFQDN(t *testing.T) { Name: "user", }, }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: true, - }, - DNSUserNameInMagicDNS: false, - }, domain: "example.com", wantErr: "failed to create valid FQDN: node has no given name", }, { - name: "no-user-name", + name: "too-long-username", node: Node{ - GivenName: "test", - User: User{}, + GivenName: strings.Repeat("a", 256), }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: true, - }, - DNSUserNameInMagicDNS: false, - }, - domain: "example.com", - want: "test.example.com", - }, - { - name: "no-magic-dns", - node: Node{ - GivenName: "test", - User: User{ - Name: "user", - }, - }, - cfg: Config{ - DNSConfig: &tailcfg.DNSConfig{ - Proxied: false, - }, - DNSUserNameInMagicDNS: false, - }, - domain: "example.com", - want: "test.example.com", + domain: "example.com", + wantErr: fmt.Sprintf("failed to create valid FQDN (%s.example.com): hostname too long, cannot except 255 ASCII chars", strings.Repeat("a", 256)), }, { name: "no-dnsconfig", @@ -288,7 +188,9 @@ func TestNodeFQDN(t *testing.T) { for _, tc := range tests { t.Run(tc.name, func(t *testing.T) { - got, err := tc.node.GetFQDN(&tc.cfg, tc.domain) + got, err := tc.node.GetFQDN(tc.domain) + + t.Logf("GOT: %q, %q", got, tc.domain) if (err != nil) && (err.Error() != tc.wantErr) { t.Errorf("GetFQDN() error = %s, wantErr %s", err, tc.wantErr) diff --git a/hscontrol/types/users.go b/hscontrol/types/users.go index 3e934e34..35839f8e 100644 --- a/hscontrol/types/users.go +++ b/hscontrol/types/users.go @@ -1,6 +1,7 @@ package types import ( + "cmp" "strconv" v1 "github.com/juanfont/headscale/gen/go/headscale/v1" @@ -10,25 +11,65 @@ import ( "tailscale.com/tailcfg" ) +type UserID uint64 + // User is the way Headscale implements the concept of users in Tailscale // // At the end of the day, users in Tailscale are some kind of 'bubbles' or users // that contain our machines. type User struct { gorm.Model + + // Username for the user, is used if email is empty + // Should not be used, please use Username(). Name string `gorm:"unique"` + + // Typically the full name of the user + DisplayName string + + // Email of the user + // Should not be used, please use Username(). + Email string + + // Unique identifier of the user from OIDC, + // comes from `sub` claim in the OIDC token + // and is used to lookup the user. + ProviderIdentifier string `gorm:"index"` + + // Provider is the origin of the user account, + // same as RegistrationMethod, without authkey. + Provider string + + ProfilePicURL string +} + +// Username is the main way to get the username of a user, +// it will return the email if it exists, the name if it exists, +// the OIDCIdentifier if it exists, and the ID if nothing else exists. +// Email and OIDCIdentifier will be set when the user has headscale +// enabled with OIDC, which means that there is a domain involved which +// should be used throughout headscale, in information returned to the +// user and the Policy engine. +func (u *User) Username() string { + return cmp.Or(u.Email, u.Name, u.ProviderIdentifier, strconv.FormatUint(uint64(u.ID), 10)) +} + +// DisplayNameOrUsername returns the DisplayName if it exists, otherwise +// it will return the Username. +func (u *User) DisplayNameOrUsername() string { + return cmp.Or(u.DisplayName, u.Username()) } // TODO(kradalby): See if we can fill in Gravatar here. func (u *User) profilePicURL() string { - return "" + return u.ProfilePicURL } func (u *User) TailscaleUser() *tailcfg.User { user := tailcfg.User{ ID: tailcfg.UserID(u.ID), - LoginName: u.Name, - DisplayName: u.Name, + LoginName: u.Username(), + DisplayName: u.DisplayNameOrUsername(), ProfilePicURL: u.profilePicURL(), Logins: []tailcfg.LoginID{}, Created: u.CreatedAt, @@ -41,9 +82,9 @@ func (u *User) TailscaleLogin() *tailcfg.Login { login := tailcfg.Login{ ID: tailcfg.LoginID(u.ID), // TODO(kradalby): this should reflect registration method. - Provider: "", - LoginName: u.Name, - DisplayName: u.Name, + Provider: u.Provider, + LoginName: u.Username(), + DisplayName: u.DisplayNameOrUsername(), ProfilePicURL: u.profilePicURL(), } @@ -53,8 +94,8 @@ func (u *User) TailscaleLogin() *tailcfg.Login { func (u *User) TailscaleUserProfile() tailcfg.UserProfile { return tailcfg.UserProfile{ ID: tailcfg.UserID(u.ID), - LoginName: u.Name, - DisplayName: u.Name, + LoginName: u.Username(), + DisplayName: u.DisplayNameOrUsername(), ProfilePicURL: u.profilePicURL(), } } @@ -66,3 +107,27 @@ func (n *User) Proto() *v1.User { CreatedAt: timestamppb.New(n.CreatedAt), } } + +type OIDCClaims struct { + // Sub is the user's unique identifier at the provider. + Sub string `json:"sub"` + + // Name is the user's full name. + Name string `json:"name,omitempty"` + Groups []string `json:"groups,omitempty"` + Email string `json:"email,omitempty"` + EmailVerified bool `json:"email_verified,omitempty"` + ProfilePictureURL string `json:"picture,omitempty"` + Username string `json:"preferred_username,omitempty"` +} + +// FromClaim overrides a User from OIDC claims. +// All fields will be updated, except for the ID. +func (u *User) FromClaim(claims *OIDCClaims) { + u.ProviderIdentifier = claims.Sub + u.DisplayName = claims.Name + u.Email = claims.Email + u.Name = claims.Username + u.ProfilePicURL = claims.ProfilePictureURL + u.Provider = util.RegisterMethodOIDC +} diff --git a/hscontrol/util/dns.go b/hscontrol/util/dns.go index ab3c90b7..217b1fbc 100644 --- a/hscontrol/util/dns.go +++ b/hscontrol/util/dns.go @@ -7,7 +7,6 @@ import ( "regexp" "strings" - "github.com/spf13/viper" "go4.org/netipx" "tailscale.com/util/dnsname" ) @@ -25,38 +24,6 @@ var invalidCharsInUserRegex = regexp.MustCompile("[^a-z0-9-.]+") var ErrInvalidUserName = errors.New("invalid user name") -func NormalizeToFQDNRulesConfigFromViper(name string) (string, error) { - strip := viper.GetBool("oidc.strip_email_domain") - - return NormalizeToFQDNRules(name, strip) -} - -// NormalizeToFQDNRules will replace forbidden chars in user -// it can also return an error if the user doesn't respect RFC 952 and 1123. -func NormalizeToFQDNRules(name string, stripEmailDomain bool) (string, error) { - name = strings.ToLower(name) - name = strings.ReplaceAll(name, "'", "") - atIdx := strings.Index(name, "@") - if stripEmailDomain && atIdx > 0 { - name = name[:atIdx] - } else { - name = strings.ReplaceAll(name, "@", ".") - } - name = invalidCharsInUserRegex.ReplaceAllString(name, "-") - - for _, elt := range strings.Split(name, ".") { - if len(elt) > LabelHostnameLength { - return "", fmt.Errorf( - "label %v is more than 63 chars: %w", - elt, - ErrInvalidUserName, - ) - } - } - - return name, nil -} - func CheckForFQDNRules(name string) error { if len(name) > LabelHostnameLength { return fmt.Errorf( diff --git a/hscontrol/util/dns_test.go b/hscontrol/util/dns_test.go index 2559cae6..28a28520 100644 --- a/hscontrol/util/dns_test.go +++ b/hscontrol/util/dns_test.go @@ -7,100 +7,6 @@ import ( "github.com/stretchr/testify/assert" ) -func TestNormalizeToFQDNRules(t *testing.T) { - type args struct { - name string - stripEmailDomain bool - } - tests := []struct { - name string - args args - want string - wantErr bool - }{ - { - name: "normalize simple name", - args: args{ - name: "normalize-simple.name", - stripEmailDomain: false, - }, - want: "normalize-simple.name", - wantErr: false, - }, - { - name: "normalize an email", - args: args{ - name: "foo.bar@example.com", - stripEmailDomain: false, - }, - want: "foo.bar.example.com", - wantErr: false, - }, - { - name: "normalize an email domain should be removed", - args: args{ - name: "foo.bar@example.com", - stripEmailDomain: true, - }, - want: "foo.bar", - wantErr: false, - }, - { - name: "strip enabled no email passed as argument", - args: args{ - name: "not-email-and-strip-enabled", - stripEmailDomain: true, - }, - want: "not-email-and-strip-enabled", - wantErr: false, - }, - { - name: "normalize complex email", - args: args{ - name: "foo.bar+complex-email@example.com", - stripEmailDomain: false, - }, - want: "foo.bar-complex-email.example.com", - wantErr: false, - }, - { - name: "user name with space", - args: args{ - name: "name space", - stripEmailDomain: false, - }, - want: "name-space", - wantErr: false, - }, - { - name: "user with quote", - args: args{ - name: "Jamie's iPhone 5", - stripEmailDomain: false, - }, - want: "jamies-iphone-5", - wantErr: false, - }, - } - for _, tt := range tests { - t.Run(tt.name, func(t *testing.T) { - got, err := NormalizeToFQDNRules(tt.args.name, tt.args.stripEmailDomain) - if (err != nil) != tt.wantErr { - t.Errorf( - "NormalizeToFQDNRules() error = %v, wantErr %v", - err, - tt.wantErr, - ) - - return - } - if got != tt.want { - t.Errorf("NormalizeToFQDNRules() = %v, want %v", got, tt.want) - } - }) - } -} - func TestCheckForFQDNRules(t *testing.T) { type args struct { name string diff --git a/integration/auth_oidc_test.go b/integration/auth_oidc_test.go index 38435fdc..d0929c4e 100644 --- a/integration/auth_oidc_test.go +++ b/integration/auth_oidc_test.go @@ -62,7 +62,6 @@ func TestOIDCAuthenticationPingAll(t *testing.T) { "HEADSCALE_OIDC_CLIENT_ID": oidcConfig.ClientID, "CREDENTIALS_DIRECTORY_TEST": "/tmp", "HEADSCALE_OIDC_CLIENT_SECRET_PATH": "${CREDENTIALS_DIRECTORY_TEST}/hs_client_oidc_secret", - "HEADSCALE_OIDC_STRIP_EMAIL_DOMAIN": fmt.Sprintf("%t", oidcConfig.StripEmaildomain), } err = scenario.CreateHeadscaleEnv( @@ -121,7 +120,6 @@ func TestOIDCExpireNodesBasedOnTokenExpiry(t *testing.T) { "HEADSCALE_OIDC_ISSUER": oidcConfig.Issuer, "HEADSCALE_OIDC_CLIENT_ID": oidcConfig.ClientID, "HEADSCALE_OIDC_CLIENT_SECRET": oidcConfig.ClientSecret, - "HEADSCALE_OIDC_STRIP_EMAIL_DOMAIN": fmt.Sprintf("%t", oidcConfig.StripEmaildomain), "HEADSCALE_OIDC_USE_EXPIRY_FROM_TOKEN": "1", } @@ -276,7 +274,6 @@ func (s *AuthOIDCScenario) runMockOIDC(accessTTL time.Duration) (*types.OIDCConf ), ClientID: "superclient", ClientSecret: "supersecret", - StripEmaildomain: true, OnlyStartIfOIDCIsAvailable: true, }, nil }