forked from TrueCloudLab/frostfs-node
[#6] services/util: Simplify response.Service
It has only 1 parameter. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
parent
5d13f70cb0
commit
761a0aaecc
2 changed files with 7 additions and 35 deletions
|
@ -590,7 +590,7 @@ func initCfg(appCfg *config.Config) *cfg {
|
||||||
key: key,
|
key: key,
|
||||||
binPublicKey: key.PublicKey().Bytes(),
|
binPublicKey: key.PublicKey().Bytes(),
|
||||||
localAddr: netAddr,
|
localAddr: netAddr,
|
||||||
respSvc: response.NewService(response.WithNetworkState(netState)),
|
respSvc: response.NewService(netState),
|
||||||
clientCache: cache.NewSDKClientCache(cacheOpts),
|
clientCache: cache.NewSDKClientCache(cacheOpts),
|
||||||
bgClientCache: cache.NewSDKClientCache(cacheOpts),
|
bgClientCache: cache.NewSDKClientCache(cacheOpts),
|
||||||
putClientCache: cache.NewSDKClientCache(cacheOpts),
|
putClientCache: cache.NewSDKClientCache(cacheOpts),
|
||||||
|
|
|
@ -11,45 +11,24 @@ import (
|
||||||
// Service represents universal v2 service
|
// Service represents universal v2 service
|
||||||
// that sets response meta header values.
|
// that sets response meta header values.
|
||||||
type Service struct {
|
type Service struct {
|
||||||
cfg *cfg
|
|
||||||
}
|
|
||||||
|
|
||||||
// Option is an option of Service constructor.
|
|
||||||
type Option func(*cfg)
|
|
||||||
|
|
||||||
type cfg struct {
|
|
||||||
version refs.Version
|
version refs.Version
|
||||||
|
|
||||||
state netmap.State
|
state netmap.State
|
||||||
}
|
}
|
||||||
|
|
||||||
func defaultCfg() *cfg {
|
|
||||||
var c cfg
|
|
||||||
|
|
||||||
version.Current().WriteToV2(&c.version)
|
|
||||||
|
|
||||||
return &c
|
|
||||||
}
|
|
||||||
|
|
||||||
// NewService creates, initializes and returns Service instance.
|
// NewService creates, initializes and returns Service instance.
|
||||||
func NewService(opts ...Option) *Service {
|
func NewService(nmState netmap.State) *Service {
|
||||||
c := defaultCfg()
|
s := &Service{state: nmState}
|
||||||
|
version.Current().WriteToV2(&s.version)
|
||||||
for i := range opts {
|
return s
|
||||||
opts[i](c)
|
|
||||||
}
|
|
||||||
|
|
||||||
return &Service{
|
|
||||||
cfg: c,
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// SetMeta sets adds meta-header to resp.
|
// SetMeta sets adds meta-header to resp.
|
||||||
func (s *Service) SetMeta(resp util.ResponseMessage) {
|
func (s *Service) SetMeta(resp util.ResponseMessage) {
|
||||||
meta := new(session.ResponseMetaHeader)
|
meta := new(session.ResponseMetaHeader)
|
||||||
meta.SetVersion(&s.cfg.version)
|
meta.SetVersion(&s.version)
|
||||||
meta.SetTTL(1) // FIXME: #1160 TTL must be calculated
|
meta.SetTTL(1) // FIXME: #1160 TTL must be calculated
|
||||||
meta.SetEpoch(s.cfg.state.CurrentEpoch())
|
meta.SetEpoch(s.state.CurrentEpoch())
|
||||||
|
|
||||||
if origin := resp.GetMetaHeader(); origin != nil {
|
if origin := resp.GetMetaHeader(); origin != nil {
|
||||||
// FIXME: #1160 what if origin is set by local server?
|
// FIXME: #1160 what if origin is set by local server?
|
||||||
|
@ -58,10 +37,3 @@ func (s *Service) SetMeta(resp util.ResponseMessage) {
|
||||||
|
|
||||||
resp.SetMetaHeader(meta)
|
resp.SetMetaHeader(meta)
|
||||||
}
|
}
|
||||||
|
|
||||||
// WithNetworkState returns option to set network state of Service.
|
|
||||||
func WithNetworkState(v netmap.State) Option {
|
|
||||||
return func(c *cfg) {
|
|
||||||
c.state = v
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
Loading…
Reference in a new issue