Skip to content

Commit

Permalink
the default language added to the setting service
Browse files Browse the repository at this point in the history
  • Loading branch information
2403905 committed Oct 5, 2023
1 parent c1dcc42 commit 40795bf
Show file tree
Hide file tree
Showing 10 changed files with 124 additions and 13 deletions.
6 changes: 6 additions & 0 deletions changelog/unreleased/default-language.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
Enhancement: The default language added

The ability of configuration the default language has been added to the setting service.

https://github.com/owncloud/ocis/pull/7417
https://github.com/owncloud/enterprise/issues/5915
4 changes: 4 additions & 0 deletions services/settings/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,3 +71,7 @@ Services can set or query ocis *setting values* of a user from settings bundles.
## Service Accounts

The settings service needs to know the ID's of service accounts but it doesn't need their secrets. Currently only one service account can be configured which has the admin role. This can be set with the `SETTINGS_SERVICE_ACCOUNT_ID_ADMIN` envvar, but it will also pick up the global `OCIS_SERVICE_ACCOUNT_ID` envvar. Also see the 'auth-service' service description for additional details.

## Default language

The default language can be defined via SETTINGS_DEFAULT_LANGUAGE environment variable. If this variable is not defined, the English language will be used as default.
2 changes: 1 addition & 1 deletion services/settings/pkg/command/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ func Server(cfg *config.Config) *cli.Command {
mtrcs := metrics.New()
mtrcs.BuildInfo.WithLabelValues(version.GetString()).Set(1)

handle := svc.NewService(cfg, logger)
handle := svc.NewDefaultLanguageService(cfg, svc.NewService(cfg, logger))

// prepare an HTTP server and add it to the group run.
httpServer, err := http.Server(
Expand Down
2 changes: 2 additions & 0 deletions services/settings/pkg/config/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ type Config struct {

ServiceAccountIDAdmin string `yaml:"service_account_id_admin" env:"OCIS_SERVICE_ACCOUNT_ID;SETTINGS_SERVICE_ACCOUNT_ID_ADMIN" desc:"The ID of the service account having the admin role. See the 'auth-service' service description for more details."`

DefaultLanguage string `yaml:"default_language" env:"SETTINGS_DEFAULT_LANGUAGE" desc:"The default language. If not defined, the English will be used as default."`

Context context.Context `yaml:"-"`
}

Expand Down
6 changes: 3 additions & 3 deletions services/settings/pkg/server/grpc/option.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import (
"github.com/owncloud/ocis/v2/ocis-pkg/log"
"github.com/owncloud/ocis/v2/services/settings/pkg/config"
"github.com/owncloud/ocis/v2/services/settings/pkg/metrics"
svc "github.com/owncloud/ocis/v2/services/settings/pkg/service/v0"
"github.com/owncloud/ocis/v2/services/settings/pkg/settings"
"github.com/urfave/cli/v2"
"go.opentelemetry.io/otel/trace"
)
Expand All @@ -21,7 +21,7 @@ type Options struct {
Context context.Context
Config *config.Config
Metrics *metrics.Metrics
ServiceHandler svc.Service
ServiceHandler settings.ServiceHandler
Flags []cli.Flag
TraceProvider trace.TracerProvider
}
Expand Down Expand Up @@ -80,7 +80,7 @@ func Flags(val []cli.Flag) Option {
}

// ServiceHandler provides a function to set the ServiceHandler option
func ServiceHandler(val svc.Service) Option {
func ServiceHandler(val settings.ServiceHandler) Option {
return func(o *Options) {
o.ServiceHandler = val
}
Expand Down
6 changes: 3 additions & 3 deletions services/settings/pkg/server/http/option.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import (
"github.com/owncloud/ocis/v2/ocis-pkg/log"
"github.com/owncloud/ocis/v2/services/settings/pkg/config"
"github.com/owncloud/ocis/v2/services/settings/pkg/metrics"
svc "github.com/owncloud/ocis/v2/services/settings/pkg/service/v0"
"github.com/owncloud/ocis/v2/services/settings/pkg/settings"
"github.com/urfave/cli/v2"
"go.opentelemetry.io/otel/trace"
)
Expand All @@ -21,7 +21,7 @@ type Options struct {
Context context.Context
Config *config.Config
Metrics *metrics.Metrics
ServiceHandler svc.Service
ServiceHandler settings.ServiceHandler
Flags []cli.Flag
TraceProvider trace.TracerProvider
}
Expand Down Expand Up @@ -80,7 +80,7 @@ func Flags(val []cli.Flag) Option {
}

// ServiceHandler provides a function to set the ServiceHandler option
func ServiceHandler(val svc.Service) Option {
func ServiceHandler(val settings.ServiceHandler) Option {
return func(o *Options) {
o.ServiceHandler = val
}
Expand Down
2 changes: 1 addition & 1 deletion services/settings/pkg/service/v0/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ type Service struct {
}

// NewService returns a service implementation for Service.
func NewService(cfg *config.Config, logger log.Logger) Service {
func NewService(cfg *config.Config, logger log.Logger) settings.ServiceHandler {
service := Service{
id: "ocis-settings",
config: cfg,
Expand Down
83 changes: 83 additions & 0 deletions services/settings/pkg/service/v0/servicedecorator.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,83 @@
package svc

import (
"context"

settingsmsg "github.com/owncloud/ocis/v2/protogen/gen/ocis/messages/settings/v0"
settingssvc "github.com/owncloud/ocis/v2/protogen/gen/ocis/services/settings/v0"
"github.com/owncloud/ocis/v2/services/settings/pkg/settings"

"github.com/owncloud/ocis/v2/services/settings/pkg/config"
"github.com/owncloud/ocis/v2/services/settings/pkg/store/defaults"
)

// NewDefaultLanguageService returns a default language decorator for ServiceHandler.
func NewDefaultLanguageService(cfg *config.Config, serviceHandler settings.ServiceHandler) settings.ServiceHandler {
defaultLanguage := cfg.DefaultLanguage
if defaultLanguage == "" {
defaultLanguage = "en"
}
return &defaultLanguageDecorator{defaultLanguage: defaultLanguage, ServiceHandler: serviceHandler}
}

type defaultLanguageDecorator struct {
defaultLanguage string
settings.ServiceHandler
}

// GetValueByUniqueIdentifiers implements the ValueService interface
func (s *defaultLanguageDecorator) GetValueByUniqueIdentifiers(ctx context.Context, req *settingssvc.GetValueByUniqueIdentifiersRequest, res *settingssvc.GetValueResponse) error {
err := s.ServiceHandler.GetValueByUniqueIdentifiers(ctx, req, res)
if err != nil {
return err
}
if res.Value == nil {
res.Value = s.withDefaultLanguageSetting(req.AccountUuid)
}
return nil
}

// ListValues implements the ValueServiceHandler interface
func (s *defaultLanguageDecorator) ListValues(ctx context.Context, req *settingssvc.ListValuesRequest, res *settingssvc.ListValuesResponse) error {
err := s.ServiceHandler.ListValues(ctx, req, res)
if err != nil {
return err
}
for _, v := range res.Values {
if v.GetValue().GetSettingId() == defaults.SettingUUIDProfileLanguage {
return nil
}
}

res.Values = append(res.Values, s.withDefaultLanguageSetting(req.AccountUuid))
return nil
}

func (s *defaultLanguageDecorator) withDefaultLanguageSetting(accountUuid string) *settingsmsg.ValueWithIdentifier {
bundle := generateBundleProfileRequest()
return &settingsmsg.ValueWithIdentifier{
Identifier: &settingsmsg.Identifier{
Extension: bundle.Extension,
Bundle: bundle.Name,
Setting: "language",
},
Value: &settingsmsg.Value{
Id: "",
BundleId: defaults.BundleUUIDProfile,
SettingId: defaults.SettingUUIDProfileLanguage,
AccountUuid: accountUuid,
Resource: &settingsmsg.Resource{
Type: settingsmsg.Resource_TYPE_USER,
},
Value: &settingsmsg.Value_ListValue{
ListValue: &settingsmsg.ListValue{Values: []*settingsmsg.ListOptionValue{
{
Option: &settingsmsg.ListOptionValue_StringValue{
StringValue: s.defaultLanguage,
},
},
}},
},
},
}
}
11 changes: 11 additions & 0 deletions services/settings/pkg/settings/settings.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@ package settings
import (
"errors"

cs3permissions "github.com/cs3org/go-cs3apis/cs3/permissions/v1beta1"
settingsmsg "github.com/owncloud/ocis/v2/protogen/gen/ocis/messages/settings/v0"
settingssvc "github.com/owncloud/ocis/v2/protogen/gen/ocis/services/settings/v0"
"github.com/owncloud/ocis/v2/services/settings/pkg/config"
)

Expand All @@ -25,6 +27,15 @@ type RegisterFunc func(*config.Config) Manager

//go:generate mockery --name=Manager

// ServiceHandler combines handlers interfaces
type ServiceHandler interface {
settingssvc.BundleServiceHandler
settingssvc.ValueServiceHandler
settingssvc.RoleServiceHandler
settingssvc.PermissionServiceHandler
cs3permissions.PermissionsAPIServer
}

// Manager combines service interfaces for abstraction of storage implementations
type Manager interface {
BundleManager
Expand Down
15 changes: 10 additions & 5 deletions services/settings/pkg/store/defaults/defaults.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ const (
// BundleUUIDRoleUserLight represents the user light role.
BundleUUIDRoleUserLight = "38071a68-456a-4553-846a-fa67bf5596cc"

// BundleUUIDProfile represents the user profile
BundleUUIDProfile = "2a506de7-99bd-4f0d-994e-c38e72c28fd9"

// RoleManagementPermissionID is the hardcoded setting UUID for the role management permission
RoleManagementPermissionID string = "a53e601e-571f-4f86-8fec-d4576ef49c62"
// RoleManagementPermissionName is the hardcoded setting name for the role management permission
Expand Down Expand Up @@ -767,7 +770,7 @@ func generateBundleUserLightRole() *settingsmsg.Bundle {

func generateBundleProfileRequest() *settingsmsg.Bundle {
return &settingsmsg.Bundle{
Id: "2a506de7-99bd-4f0d-994e-c38e72c28fd9",
Id: BundleUUIDProfile,
Name: "profile",
Extension: "ocis-accounts",
Type: settingsmsg.Bundle_TYPE_DEFAULT,
Expand Down Expand Up @@ -837,7 +840,6 @@ var languageSetting = settingsmsg.Setting_SingleChoiceValue{
},
},
DisplayValue: "English",
Default: true,
},
{
Value: &settingsmsg.ListOptionValue{
Expand Down Expand Up @@ -885,18 +887,21 @@ func DefaultRoleAssignments(cfg *config.Config) []*settingsmsg.UserRoleAssignmen
{
AccountUuid: "4c510ada-c86b-4815-8820-42cdf82c3d51",
RoleId: BundleUUIDRoleUser,
}, {
},
{
AccountUuid: "f7fbf8c8-139b-4376-b307-cf0a8c2d0d9c",
RoleId: BundleUUIDRoleUser,
}, {
},
{
AccountUuid: "932b4540-8d16-481e-8ef4-588e4b6b151c",
RoleId: BundleUUIDRoleUser,
},
{
// additional admin user
AccountUuid: "058bff95-6708-4fe5-91e4-9ea3d377588b", // demo user "moss"
RoleId: BundleUUIDRoleAdmin,
}, {
},
{
// default users with role "spaceadmin"
AccountUuid: "534bb038-6f9d-4093-946f-133be61fa4e7",
RoleId: BundleUUIDRoleSpaceAdmin,
Expand Down

0 comments on commit 40795bf

Please sign in to comment.