Some checks failed
CI / Run tests and lints (push) Failing after 29s
CI / Build CLI docs (push) Has been skipped
CI / Run Playwright end-to-end tests (mysql) (push) Failing after 3s
CI / Run Playwright end-to-end tests (postgres) (push) Failing after 2s
CI / Run Playwright end-to-end tests (sqlite) (push) Failing after 3s
CodeQL / Analyze (go) (push) Failing after 4s
CodeQL / Analyze (javascript) (push) Failing after 3s
Docker Image Scanners / scanners (push) Failing after 4s
Synchronize Issue Labels / Synchronize Issue Labels (push) Has been skipped
CI / Run end-to-end tests (postgres) (push) Failing after 19s
CI / Run end-to-end tests (sqlite) (push) Failing after 2s
CI / Run Playwright end-to-end tests (cockroach) (push) Failing after 2s
CI / Draft newsletter (push) Has been skipped
CI / Generate release (push) Has been skipped
CI / Pending approval Slack notification (push) Has been skipped
CI / Send newsletter (push) Has been skipped
445 lines
15 KiB
Go
445 lines
15 KiB
Go
// Copyright © 2023 Ory Corp
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
package passkey
|
|
|
|
import (
|
|
"bytes"
|
|
"context"
|
|
_ "embed"
|
|
"encoding/json"
|
|
"fmt"
|
|
"net/http"
|
|
"strings"
|
|
|
|
"go.opentelemetry.io/otel/attribute"
|
|
|
|
"github.com/ory/x/otelx"
|
|
|
|
"github.com/ory/kratos/x/webauthnx/js"
|
|
|
|
"github.com/go-webauthn/webauthn/protocol"
|
|
"github.com/go-webauthn/webauthn/webauthn"
|
|
"github.com/gofrs/uuid"
|
|
"github.com/pkg/errors"
|
|
"github.com/tidwall/gjson"
|
|
"github.com/tidwall/sjson"
|
|
|
|
"github.com/ory/herodot"
|
|
"github.com/ory/kratos/continuity"
|
|
"github.com/ory/kratos/identity"
|
|
"github.com/ory/kratos/selfservice/flow"
|
|
"github.com/ory/kratos/selfservice/flow/settings"
|
|
"github.com/ory/kratos/session"
|
|
"github.com/ory/kratos/text"
|
|
"github.com/ory/kratos/ui/node"
|
|
"github.com/ory/kratos/x"
|
|
"github.com/ory/kratos/x/webauthnx"
|
|
"github.com/ory/x/decoderx"
|
|
"github.com/ory/x/randx"
|
|
"github.com/ory/x/sqlcon"
|
|
"github.com/ory/x/sqlxx"
|
|
)
|
|
|
|
func (s *Strategy) SettingsStrategyID() string { return s.ID().String() }
|
|
|
|
const (
|
|
InternalContextKeySessionData = "session_data"
|
|
InternalContextKeySessionOptions = "session_options"
|
|
)
|
|
|
|
func (s *Strategy) PopulateSettingsMethod(ctx context.Context, r *http.Request, id *identity.Identity, f *settings.Flow) (err error) {
|
|
ctx, span := s.d.Tracer(ctx).Tracer().Start(ctx, "selfservice.strategy.passkey.Strategy.PopulateSettingsMethod")
|
|
defer otelx.End(span, &err)
|
|
|
|
f.UI.SetCSRF(s.d.GenerateCSRFToken(r))
|
|
count, err := s.d.IdentityManager().CountActiveFirstFactorCredentials(ctx, id)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
webAuthns, err := s.identityListWebAuthn(id)
|
|
if err != nil && !errors.Is(err, sqlcon.ErrNoRows()) {
|
|
return err
|
|
}
|
|
|
|
var registrationOptions []webauthn.RegistrationOption
|
|
if webAuthns != nil {
|
|
exclude := make([]protocol.CredentialDescriptor, 0, len(webAuthns.Credentials))
|
|
for k := range webAuthns.Credentials {
|
|
// We only show the option to remove a credential, if it is not the last one when passwordless,
|
|
// or, if it is for MFA we show it always.
|
|
cred := &webAuthns.Credentials[k]
|
|
|
|
f.UI.Nodes.Append(webauthnx.NewPasskeyUnlink(cred, func(a *node.InputAttributes) {
|
|
// Do not remove this node if it is the last credential the identity can sign in with.
|
|
a.Disabled = count < 2
|
|
}))
|
|
exclude = append(exclude, cred.ToWebAuthn().Descriptor())
|
|
}
|
|
// Exclude every registered passkey: platform authenticators overwrite their stored entry on
|
|
// re-enrollment while Kratos keeps both server-side (WebAuthn Level 2, section 5.4.3).
|
|
registrationOptions = append(registrationOptions, webauthn.WithExclusions(exclude))
|
|
}
|
|
|
|
web, err := webauthn.New(s.d.Config().PasskeyConfig(ctx))
|
|
if err != nil {
|
|
return errors.WithStack(err)
|
|
}
|
|
|
|
identifier := s.PasskeyDisplayNameFromIdentity(ctx, id)
|
|
if identifier == "" {
|
|
f.UI.Messages.Add(text.NewErrorValidationIdentifierMissing())
|
|
return nil
|
|
}
|
|
|
|
user := &webauthnx.User{
|
|
Name: identifier,
|
|
ID: []byte(randx.MustString(64, randx.AlphaNum)),
|
|
Config: s.d.Config().PasskeyConfig(ctx),
|
|
}
|
|
option, sessionData, err := web.BeginRegistration(user, registrationOptions...)
|
|
if err != nil {
|
|
return errors.WithStack(err)
|
|
}
|
|
|
|
f.InternalContext, err = sjson.SetBytes(f.InternalContext, flow.PrefixInternalContextKey(s.ID(), InternalContextKeySessionData), sessionData)
|
|
if err != nil {
|
|
return errors.WithStack(err)
|
|
}
|
|
|
|
injectWebAuthnOptions, err := json.Marshal(option)
|
|
if err != nil {
|
|
return errors.WithStack(err)
|
|
}
|
|
|
|
if f.Type == flow.TypeBrowser {
|
|
f.UI.Nodes.Upsert(webauthnx.NewWebAuthnScript(s.d.Config().SelfPublicURL(ctx)))
|
|
}
|
|
|
|
f.UI.Nodes.Upsert(node.NewInputField(
|
|
node.PasskeyRegisterTrigger,
|
|
"",
|
|
node.PasskeyGroup,
|
|
node.InputAttributeTypeButton,
|
|
node.WithInputAttributes(func(a *node.InputAttributes) {
|
|
a.OnClickTrigger = js.WebAuthnTriggersPasskeySettingsRegistration
|
|
// Only attach raw JS handler for browser flows; API flows must stay script-free.
|
|
if f.Type == flow.TypeBrowser {
|
|
//nolint:staticcheck
|
|
a.OnClick = js.WebAuthnTriggersPasskeySettingsRegistration.String() + "()"
|
|
}
|
|
}),
|
|
).WithMetaLabel(text.NewInfoSelfServiceSettingsRegisterPasskey()))
|
|
|
|
f.UI.Nodes.Upsert(&node.Node{
|
|
Type: node.Input,
|
|
Group: node.PasskeyGroup,
|
|
Meta: &node.Meta{},
|
|
Attributes: &node.InputAttributes{
|
|
Name: node.PasskeySettingsRegister,
|
|
Type: node.InputAttributeTypeHidden,
|
|
},
|
|
})
|
|
|
|
f.UI.Nodes.Upsert(&node.Node{
|
|
Type: node.Input,
|
|
Group: node.PasskeyGroup,
|
|
Meta: &node.Meta{},
|
|
Attributes: &node.InputAttributes{
|
|
Name: node.PasskeyCreateData,
|
|
Type: node.InputAttributeTypeHidden,
|
|
FieldValue: string(injectWebAuthnOptions),
|
|
},
|
|
})
|
|
|
|
return nil
|
|
}
|
|
|
|
func (s *Strategy) identityListWebAuthn(id *identity.Identity) (*identity.CredentialsWebAuthnConfig, error) {
|
|
cred, ok := id.GetCredentials(s.ID())
|
|
if !ok {
|
|
return nil, errors.WithStack(sqlcon.ErrNoRows())
|
|
}
|
|
|
|
var cc identity.CredentialsWebAuthnConfig
|
|
if err := json.Unmarshal(cred.Config, &cc); err != nil {
|
|
return nil, errors.WithStack(err)
|
|
}
|
|
|
|
return &cc, nil
|
|
}
|
|
|
|
func (s *Strategy) Settings(ctx context.Context, w http.ResponseWriter, r *http.Request, f *settings.Flow, ss *session.Session) (_ *settings.UpdateContext, err error) {
|
|
ctx, span := s.d.Tracer(ctx).Tracer().Start(ctx, "selfservice.strategy.passkey.Strategy.Settings")
|
|
defer otelx.End(span, &err)
|
|
|
|
var p updateSettingsFlowWithPasskeyMethod
|
|
ctxUpdate, err := settings.PrepareUpdate(s.d, w, r, f, ss, settings.ContinuityKey(s.SettingsStrategyID()), &p)
|
|
if errors.Is(err, settings.ErrContinuePreviousAction) {
|
|
return ctxUpdate, s.continueSettingsFlow(ctx, w, r, ctxUpdate, p)
|
|
} else if err != nil {
|
|
return ctxUpdate, s.handleSettingsError(ctx, w, r, ctxUpdate, p, err)
|
|
}
|
|
|
|
if err := s.decodeSettingsFlow(r, &p); err != nil {
|
|
return ctxUpdate, s.handleSettingsError(ctx, w, r, ctxUpdate, p, err)
|
|
}
|
|
|
|
if len(p.Register)+len(p.Remove) > 0 {
|
|
// This method has only two submit buttons
|
|
p.Method = s.SettingsStrategyID()
|
|
if err := flow.MethodEnabledAndAllowed(ctx, f.GetFlowName(), s.SettingsStrategyID(), p.Method, s.d); err != nil {
|
|
return nil, s.handleSettingsError(ctx, w, r, ctxUpdate, p, err)
|
|
}
|
|
} else {
|
|
span.SetAttributes(attribute.String("not_responsible_reason", "neither register nor remove provided"))
|
|
return nil, errors.WithStack(flow.ErrStrategyNotResponsible)
|
|
}
|
|
|
|
// This does not come from the payload!
|
|
p.Flow = ctxUpdate.Flow.ID.String()
|
|
if err := s.continueSettingsFlow(ctx, w, r, ctxUpdate, p); err != nil {
|
|
return ctxUpdate, s.handleSettingsError(ctx, w, r, ctxUpdate, p, err)
|
|
}
|
|
|
|
return ctxUpdate, nil
|
|
}
|
|
|
|
// Update Settings Flow with Passkey Method
|
|
//
|
|
// swagger:model updateSettingsFlowWithPasskeyMethod
|
|
type updateSettingsFlowWithPasskeyMethod struct {
|
|
// Register a WebAuthn Security Key
|
|
//
|
|
// It is expected that the JSON returned by the WebAuthn registration process
|
|
// is included here.
|
|
Register string `json:"passkey_settings_register"`
|
|
|
|
// Remove a WebAuthn Security Key
|
|
//
|
|
// This must contain the ID of the WebAuthN connection.
|
|
Remove string `json:"passkey_remove"`
|
|
|
|
// CSRFToken is the anti-CSRF token
|
|
CSRFToken string `json:"csrf_token"`
|
|
|
|
// Method
|
|
//
|
|
// Should be set to "passkey" when trying to add, update, or remove a webAuthn pairing.
|
|
//
|
|
// required: true
|
|
Method string `json:"method"`
|
|
|
|
// Flow is flow ID.
|
|
//
|
|
// swagger:ignore
|
|
Flow string `json:"flow"`
|
|
}
|
|
|
|
func (p *updateSettingsFlowWithPasskeyMethod) GetFlowID() uuid.UUID {
|
|
return x.ParseUUID(p.Flow)
|
|
}
|
|
|
|
func (p *updateSettingsFlowWithPasskeyMethod) SetFlowID(rid uuid.UUID) {
|
|
p.Flow = rid.String()
|
|
}
|
|
|
|
func (s *Strategy) continueSettingsFlow(
|
|
ctx context.Context,
|
|
w http.ResponseWriter, r *http.Request,
|
|
ctxUpdate *settings.UpdateContext, p updateSettingsFlowWithPasskeyMethod,
|
|
) error {
|
|
if len(p.Register+p.Remove) > 0 {
|
|
if err := flow.MethodEnabledAndAllowed(ctx, flow.SettingsFlow, s.SettingsStrategyID(), s.SettingsStrategyID(), s.d); err != nil {
|
|
return err
|
|
}
|
|
|
|
if err := flow.EnsureCSRF(s.d, r, ctxUpdate.Flow.Type, s.d.Config().DisableAPIFlowEnforcement(ctx), s.d.GenerateCSRFToken, p.CSRFToken); err != nil {
|
|
return err
|
|
}
|
|
|
|
if !s.d.SessionManager().IsPrivileged(ctx, ctxUpdate.Session) {
|
|
return errors.WithStack(settings.NewFlowNeedsReAuth())
|
|
}
|
|
} else {
|
|
return errors.New("ended up in unexpected state")
|
|
}
|
|
|
|
switch {
|
|
case len(p.Remove) > 0:
|
|
return s.continueSettingsFlowRemove(ctx, w, r, ctxUpdate, p)
|
|
case len(p.Register) > 0:
|
|
return s.continueSettingsFlowAdd(ctx, ctxUpdate, p)
|
|
default:
|
|
return errors.New("ended up in unexpected state")
|
|
}
|
|
}
|
|
|
|
func (s *Strategy) continueSettingsFlowRemove(ctx context.Context, w http.ResponseWriter, r *http.Request, ctxUpdate *settings.UpdateContext, p updateSettingsFlowWithPasskeyMethod) error {
|
|
i, err := s.d.PrivilegedIdentityPool().GetIdentityConfidential(ctx, ctxUpdate.Session.IdentityID)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
cred, ok := i.GetCredentials(s.ID())
|
|
if !ok {
|
|
return errors.WithStack(herodot.ErrBadRequest().WithReasonf("You tried to remove a WebAuthn but you have no WebAuthn set up."))
|
|
}
|
|
|
|
var cc identity.CredentialsWebAuthnConfig
|
|
if err := json.Unmarshal(cred.Config, &cc); err != nil {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Unable to decode identity credentials.").WithDebug(err.Error()))
|
|
}
|
|
|
|
updated := make([]identity.CredentialWebAuthn, 0)
|
|
for k, cred := range cc.Credentials {
|
|
if fmt.Sprintf("%x", cred.ID) != p.Remove {
|
|
updated = append(updated, cc.Credentials[k])
|
|
}
|
|
}
|
|
|
|
if len(updated) == len(cc.Credentials) {
|
|
return errors.WithStack(herodot.ErrBadRequest().WithReasonf("You tried to remove a passkey which does not exist."))
|
|
}
|
|
|
|
count, err := s.d.IdentityManager().CountActiveFirstFactorCredentials(ctx, i)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
if count < 2 {
|
|
return s.handleSettingsError(ctx, w, r, ctxUpdate, p, errors.WithStack(webauthnx.ErrNotEnoughCredentials()))
|
|
}
|
|
|
|
if len(updated) == 0 {
|
|
i.DeleteCredentialsType(identity.CredentialsTypePasskey)
|
|
ctxUpdate.UpdateIdentity(i)
|
|
return nil
|
|
}
|
|
|
|
cc.Credentials = updated
|
|
cred.Config, err = json.Marshal(cc)
|
|
if err != nil {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Unable to encode identity credentials.").WithDebug(err.Error()))
|
|
}
|
|
|
|
i.SetCredentials(s.ID(), *cred)
|
|
ctxUpdate.UpdateIdentity(i)
|
|
return nil
|
|
}
|
|
|
|
func (s *Strategy) continueSettingsFlowAdd(ctx context.Context, ctxUpdate *settings.UpdateContext, p updateSettingsFlowWithPasskeyMethod) error {
|
|
webAuthnSession := gjson.GetBytes(ctxUpdate.Flow.InternalContext, flow.PrefixInternalContextKey(s.ID(), InternalContextKeySessionData))
|
|
if !webAuthnSession.IsObject() {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Expected WebAuthN in internal context to be an object."))
|
|
}
|
|
|
|
var webAuthnSess webauthn.SessionData
|
|
if err := json.Unmarshal([]byte(gjson.GetBytes(ctxUpdate.Flow.InternalContext, flow.PrefixInternalContextKey(s.ID(), InternalContextKeySessionData)).Raw), &webAuthnSess); err != nil {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Expected WebAuthN in internal context to be an object but got: %s", err))
|
|
}
|
|
|
|
webAuthnResponse, err := protocol.ParseCredentialCreationResponseBody(strings.NewReader(p.Register))
|
|
if err != nil {
|
|
return errors.WithStack(herodot.ErrBadRequest().WithReasonf("Unable to parse WebAuthn response: %s", err))
|
|
}
|
|
|
|
web, err := webauthn.New(s.d.Config().PasskeyConfig(ctx))
|
|
if err != nil {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Unable to get webAuthn config.").WithDebug(err.Error()))
|
|
}
|
|
|
|
credential, err := web.CreateCredential(&webauthnx.User{
|
|
ID: webAuthnSess.UserID,
|
|
Config: web.Config,
|
|
}, webAuthnSess, webAuthnResponse)
|
|
if err != nil {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Unable to create WebAuthn credential: %s", err))
|
|
}
|
|
|
|
i, err := s.d.PrivilegedIdentityPool().GetIdentityConfidential(ctx, ctxUpdate.Session.IdentityID)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
cred := i.GetCredentialsOr(s.ID(), &identity.Credentials{Config: sqlxx.JSONRawMessage("{}")})
|
|
|
|
var cc identity.CredentialsWebAuthnConfig
|
|
if err := json.Unmarshal(cred.Config, &cc); err != nil {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Unable to decode identity credentials.").WithDebug(err.Error()))
|
|
}
|
|
|
|
for k := range cc.Credentials {
|
|
if bytes.Equal(cc.Credentials[k].ID, credential.ID) {
|
|
return errors.WithStack(webauthnx.ErrCredentialAlreadyRegistered("#/" + node.PasskeySettingsRegister))
|
|
}
|
|
}
|
|
|
|
credentialWebAuthn := identity.CredentialFromWebAuthn(credential, true)
|
|
cc.UserHandle = webAuthnSess.UserID
|
|
cc.Credentials = append(cc.Credentials, *credentialWebAuthn)
|
|
credentialsConfig, err := json.Marshal(cc)
|
|
if err != nil {
|
|
return errors.WithStack(herodot.ErrInternalServerError().WithReasonf("Unable to encode identity credentials.").WithDebug(err.Error()))
|
|
}
|
|
|
|
i.UpsertCredentialsConfig(s.ID(), credentialsConfig, 1, identity.WithAdditionalIdentifier(string(webAuthnSess.UserID)))
|
|
if err := s.validateCredentials(ctx, i); err != nil {
|
|
return err
|
|
}
|
|
|
|
// Remove the WebAuthn URL from the internal context now that it is set!
|
|
ctxUpdate.Flow.InternalContext, err = sjson.DeleteBytes(ctxUpdate.Flow.InternalContext, flow.PrefixInternalContextKey(s.ID(), InternalContextKeySessionData))
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
if err := s.d.SettingsFlowPersister().UpdateSettingsFlow(ctx, ctxUpdate.Flow); err != nil {
|
|
return err
|
|
}
|
|
|
|
aal := identity.AuthenticatorAssuranceLevel1
|
|
|
|
// Since we added the method, it also means that we have authenticated it
|
|
if err := s.d.SessionManager().SessionAddAuthenticationMethods(ctx, ctxUpdate.Session.ID, session.AuthenticationMethod{
|
|
Method: s.ID(),
|
|
AAL: aal,
|
|
}); err != nil {
|
|
return err
|
|
}
|
|
|
|
ctxUpdate.UpdateIdentity(i)
|
|
return nil
|
|
}
|
|
|
|
func (s *Strategy) decodeSettingsFlow(r *http.Request, dest interface{}) error {
|
|
compiler, err := decoderx.HTTPRawJSONSchemaCompiler(settingsSchema)
|
|
if err != nil {
|
|
return errors.WithStack(err)
|
|
}
|
|
|
|
return decoderx.Decode(r, dest, compiler,
|
|
decoderx.HTTPKeepRequestBody(true),
|
|
decoderx.HTTPDecoderAllowedMethods("POST", "GET"),
|
|
decoderx.HTTPDecoderSetValidatePayloads(true),
|
|
decoderx.HTTPDecoderJSONFollowsFormFormat(),
|
|
)
|
|
}
|
|
|
|
func (s *Strategy) handleSettingsError(ctx context.Context, w http.ResponseWriter, r *http.Request, ctxUpdate *settings.UpdateContext, p updateSettingsFlowWithPasskeyMethod, err error) error {
|
|
// Do not pause flow if the flow type is an API flow as we can't save cookies in those flows.
|
|
if e := new(settings.FlowNeedsReAuth); errors.As(err, &e) && ctxUpdate.Flow != nil && ctxUpdate.Flow.Type == flow.TypeBrowser {
|
|
if _, err := s.d.ContinuityManager().Pause(ctx, w, r, settings.ContinuityKey(s.SettingsStrategyID()), continuity.NewCookieReferenceStore(s.d.ContinuityCookieManager(ctx)), settings.ContinuityOptions(p, ctxUpdate.GetSessionIdentity())...); err != nil {
|
|
return err
|
|
}
|
|
}
|
|
|
|
if ctxUpdate.Flow != nil {
|
|
ctxUpdate.Flow.UI.ResetMessages()
|
|
ctxUpdate.Flow.UI.SetCSRF(s.d.GenerateCSRFToken(r))
|
|
}
|
|
|
|
return err
|
|
}
|