From 71e1ae6e3c09335ff4b180ce43d473a33395a0bd Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 15:54:23 +0300 Subject: [PATCH 1/8] add function registration service --- cmd/main.go | 132 ++++------------------------ internal/handlers/handlerservice.go | 99 +++++++++++++++++++++ 2 files changed, 115 insertions(+), 116 deletions(-) create mode 100644 internal/handlers/handlerservice.go diff --git a/cmd/main.go b/cmd/main.go index b9ca7aa..7e5bc0d 100644 --- a/cmd/main.go +++ b/cmd/main.go @@ -7,15 +7,11 @@ import ( "os" "path" - "git.defalsify.org/vise.git/asm" - "git.defalsify.org/vise.git/db" - fsdb "git.defalsify.org/vise.git/db/fs" - gdbmdb "git.defalsify.org/vise.git/db/gdbm" "git.defalsify.org/vise.git/engine" "git.defalsify.org/vise.git/logging" - "git.defalsify.org/vise.git/persist" "git.defalsify.org/vise.git/resource" - "git.grassecon.net/urdt/ussd/internal/handlers/ussd" + "git.grassecon.net/urdt/ussd/internal/handlers" + "git.grassecon.net/urdt/ussd/internal/storage" ) var ( @@ -23,106 +19,6 @@ var ( scriptDir = path.Join("services", "registration") ) -func getParser(fp string, debug bool) (*asm.FlagParser, error) { - flagParser := asm.NewFlagParser().WithDebug() - _, err := flagParser.Load(fp) - if err != nil { - return nil, err - } - return flagParser, nil -} - -func getHandler(appFlags *asm.FlagParser, rs *resource.DbResource, pe *persist.Persister, userdataStore db.Db) (*ussd.Handlers, error) { - - ussdHandlers, err := ussd.NewHandlers(appFlags, userdataStore) - if err != nil { - return nil, err - } - ussdHandlers = ussdHandlers.WithPersister(pe) - rs.AddLocalFunc("set_language", ussdHandlers.SetLanguage) - rs.AddLocalFunc("create_account", ussdHandlers.CreateAccount) - rs.AddLocalFunc("save_pin", ussdHandlers.SavePin) - rs.AddLocalFunc("verify_pin", ussdHandlers.VerifyPin) - rs.AddLocalFunc("check_identifier", ussdHandlers.CheckIdentifier) - rs.AddLocalFunc("check_account_status", ussdHandlers.CheckAccountStatus) - rs.AddLocalFunc("authorize_account", ussdHandlers.Authorize) - rs.AddLocalFunc("quit", ussdHandlers.Quit) - rs.AddLocalFunc("check_balance", ussdHandlers.CheckBalance) - rs.AddLocalFunc("validate_recipient", ussdHandlers.ValidateRecipient) - rs.AddLocalFunc("transaction_reset", ussdHandlers.TransactionReset) - rs.AddLocalFunc("max_amount", ussdHandlers.MaxAmount) - rs.AddLocalFunc("validate_amount", ussdHandlers.ValidateAmount) - rs.AddLocalFunc("reset_transaction_amount", ussdHandlers.ResetTransactionAmount) - rs.AddLocalFunc("get_recipient", ussdHandlers.GetRecipient) - rs.AddLocalFunc("get_sender", ussdHandlers.GetSender) - rs.AddLocalFunc("get_amount", ussdHandlers.GetAmount) - rs.AddLocalFunc("reset_incorrect", ussdHandlers.ResetIncorrectPin) - rs.AddLocalFunc("save_firstname", ussdHandlers.SaveFirstname) - rs.AddLocalFunc("save_familyname", ussdHandlers.SaveFamilyname) - rs.AddLocalFunc("save_gender", ussdHandlers.SaveGender) - rs.AddLocalFunc("save_location", ussdHandlers.SaveLocation) - rs.AddLocalFunc("save_yob", ussdHandlers.SaveYob) - rs.AddLocalFunc("save_offerings", ussdHandlers.SaveOfferings) - rs.AddLocalFunc("quit_with_balance", ussdHandlers.QuitWithBalance) - rs.AddLocalFunc("reset_account_authorized", ussdHandlers.ResetAccountAuthorized) - rs.AddLocalFunc("reset_allow_update", ussdHandlers.ResetAllowUpdate) - rs.AddLocalFunc("get_profile_info", ussdHandlers.GetProfileInfo) - rs.AddLocalFunc("verify_yob", ussdHandlers.VerifyYob) - rs.AddLocalFunc("reset_incorrect_date_format", ussdHandlers.ResetIncorrectYob) - rs.AddLocalFunc("set_reset_single_edit", ussdHandlers.SetResetSingleEdit) - rs.AddLocalFunc("initiate_transaction", ussdHandlers.InitiateTransaction) - rs.AddLocalFunc("save_temporary_pin", ussdHandlers.SaveTemporaryPin) - rs.AddLocalFunc("verify_new_pin", ussdHandlers.VerifyNewPin) - rs.AddLocalFunc("confirm_pin_change", ussdHandlers.ConfirmPinChange) - rs.AddLocalFunc("quit_with_help",ussdHandlers.QuitWithHelp) - - return ussdHandlers, nil -} - -func ensureDbDir(dbDir string) error { - err := os.MkdirAll(dbDir, 0700) - if err != nil { - return fmt.Errorf("state dir create exited with error: %v\n", err) - } - return nil -} - -func getPersister(dbDir string, ctx context.Context) (*persist.Persister, error) { - err := ensureDbDir(dbDir) - if err != nil { - return nil, err - } - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "state.gdbm") - store.Connect(ctx, storeFile) - pr := persist.NewPersister(store) - return pr, nil -} - -func getUserdataDb(dbDir string, ctx context.Context) db.Db { - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "userdata.gdbm") - store.Connect(ctx, storeFile) - - return store -} - -func getResource(resourceDir string, ctx context.Context) (resource.Resource, error) { - store := fsdb.NewFsDb() - err := store.Connect(ctx, resourceDir) - if err != nil { - return nil, err - } - rfs := resource.NewDbResource(store) - return rfs, nil -} - -func getEngine(cfg engine.Config, rs resource.Resource, pr *persist.Persister) *engine.DefaultEngine { - en := engine.NewEngine(cfg, rs) - en = en.WithPersister(pr) - return en -} - func main() { var dbDir string var size uint @@ -139,11 +35,6 @@ func main() { ctx := context.Background() ctx = context.WithValue(ctx, "SessionId", sessionId) pfp := path.Join(scriptDir, "pp.csv") - flagParser, err := getParser(pfp, true) - - if err != nil { - os.Exit(1) - } cfg := engine.Config{ Root: "root", @@ -152,19 +43,21 @@ func main() { FlagCount: uint32(16), } - rs, err := getResource(scriptDir, ctx) + menuStorageService := storage.MenuStorageService{} + + rs, err := menuStorageService.GetResource(scriptDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - pe, err := getPersister(dbDir, ctx) + pe, err := menuStorageService.GetPersister(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - store := getUserdataDb(dbDir, ctx) + userdatastore := menuStorageService.GetUserdataDb(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) @@ -176,13 +69,20 @@ func main() { os.Exit(1) } - hl, err := getHandler(flagParser, dbResource, pe, store) + lhs, err := handlers.NewLocalHandlerService(pfp, true, dbResource, pe, userdatastore, cfg, rs) + if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - en := getEngine(cfg, rs, pe) + hl, err := lhs.GetHandler() + if err != nil { + fmt.Fprintf(os.Stderr, err.Error()) + os.Exit(1) + } + + en := lhs.GetEngine() en = en.WithFirst(hl.Init) if debug { en = en.WithDebug(nil) diff --git a/internal/handlers/handlerservice.go b/internal/handlers/handlerservice.go new file mode 100644 index 0000000..e6d4fb6 --- /dev/null +++ b/internal/handlers/handlerservice.go @@ -0,0 +1,99 @@ +package handlers + +import ( + "git.defalsify.org/vise.git/asm" + "git.defalsify.org/vise.git/db" + "git.defalsify.org/vise.git/engine" + "git.defalsify.org/vise.git/persist" + "git.defalsify.org/vise.git/resource" + "git.grassecon.net/urdt/ussd/internal/handlers/ussd" +) + +type HandlerService interface { + GetHandler() (*ussd.Handlers, error) +} + +func getParser(fp string, debug bool) (*asm.FlagParser, error) { + flagParser := asm.NewFlagParser().WithDebug() + _, err := flagParser.Load(fp) + if err != nil { + return nil, err + } + return flagParser, nil +} + +type LocalHandlerService struct { + Parser *asm.FlagParser + DbRs *resource.DbResource + Pe *persist.Persister + UserdataStore db.Db + Cfg engine.Config + Rs resource.Resource +} + +func NewLocalHandlerService(fp string, debug bool, dbResource *resource.DbResource, Pe *persist.Persister, userDataStore db.Db, cfg engine.Config, rs resource.Resource) (*LocalHandlerService, error) { + parser, err := getParser(fp, debug) + if err != nil { + return nil, err + } + return &LocalHandlerService{ + Parser: parser, + DbRs: dbResource, + Pe: Pe, + UserdataStore: userDataStore, + Cfg: cfg, + Rs: rs, + }, nil +} + +func (localHandlerService *LocalHandlerService) GetHandler() (*ussd.Handlers, error) { + ussdHandlers, err := ussd.NewHandlers(localHandlerService.Parser, localHandlerService.UserdataStore) + if err != nil { + return nil, err + } + ussdHandlers = ussdHandlers.WithPersister(localHandlerService.Pe) + localHandlerService.DbRs.AddLocalFunc("set_language", ussdHandlers.SetLanguage) + localHandlerService.DbRs.AddLocalFunc("create_account", ussdHandlers.CreateAccount) + localHandlerService.DbRs.AddLocalFunc("save_pin", ussdHandlers.SavePin) + localHandlerService.DbRs.AddLocalFunc("verify_pin", ussdHandlers.VerifyPin) + localHandlerService.DbRs.AddLocalFunc("check_identifier", ussdHandlers.CheckIdentifier) + localHandlerService.DbRs.AddLocalFunc("check_account_status", ussdHandlers.CheckAccountStatus) + localHandlerService.DbRs.AddLocalFunc("authorize_account", ussdHandlers.Authorize) + localHandlerService.DbRs.AddLocalFunc("quit", ussdHandlers.Quit) + localHandlerService.DbRs.AddLocalFunc("check_balance", ussdHandlers.CheckBalance) + localHandlerService.DbRs.AddLocalFunc("validate_recipient", ussdHandlers.ValidateRecipient) + localHandlerService.DbRs.AddLocalFunc("transaction_reset", ussdHandlers.TransactionReset) + localHandlerService.DbRs.AddLocalFunc("max_amount", ussdHandlers.MaxAmount) + localHandlerService.DbRs.AddLocalFunc("validate_amount", ussdHandlers.ValidateAmount) + localHandlerService.DbRs.AddLocalFunc("reset_transaction_amount", ussdHandlers.ResetTransactionAmount) + localHandlerService.DbRs.AddLocalFunc("get_recipient", ussdHandlers.GetRecipient) + localHandlerService.DbRs.AddLocalFunc("get_sender", ussdHandlers.GetSender) + localHandlerService.DbRs.AddLocalFunc("get_amount", ussdHandlers.GetAmount) + localHandlerService.DbRs.AddLocalFunc("reset_incorrect", ussdHandlers.ResetIncorrectPin) + localHandlerService.DbRs.AddLocalFunc("save_firstname", ussdHandlers.SaveFirstname) + localHandlerService.DbRs.AddLocalFunc("save_familyname", ussdHandlers.SaveFamilyname) + localHandlerService.DbRs.AddLocalFunc("save_gender", ussdHandlers.SaveGender) + localHandlerService.DbRs.AddLocalFunc("save_location", ussdHandlers.SaveLocation) + localHandlerService.DbRs.AddLocalFunc("save_yob", ussdHandlers.SaveYob) + localHandlerService.DbRs.AddLocalFunc("save_offerings", ussdHandlers.SaveOfferings) + localHandlerService.DbRs.AddLocalFunc("quit_with_balance", ussdHandlers.QuitWithBalance) + localHandlerService.DbRs.AddLocalFunc("reset_account_authorized", ussdHandlers.ResetAccountAuthorized) + localHandlerService.DbRs.AddLocalFunc("reset_allow_update", ussdHandlers.ResetAllowUpdate) + localHandlerService.DbRs.AddLocalFunc("get_profile_info", ussdHandlers.GetProfileInfo) + localHandlerService.DbRs.AddLocalFunc("verify_yob", ussdHandlers.VerifyYob) + localHandlerService.DbRs.AddLocalFunc("reset_incorrect_date_format", ussdHandlers.ResetIncorrectYob) + localHandlerService.DbRs.AddLocalFunc("set_reset_single_edit", ussdHandlers.SetResetSingleEdit) + localHandlerService.DbRs.AddLocalFunc("initiate_transaction", ussdHandlers.InitiateTransaction) + localHandlerService.DbRs.AddLocalFunc("save_temporary_pin", ussdHandlers.SaveTemporaryPin) + localHandlerService.DbRs.AddLocalFunc("verify_new_pin", ussdHandlers.VerifyNewPin) + localHandlerService.DbRs.AddLocalFunc("confirm_pin_change", ussdHandlers.ConfirmPinChange) + localHandlerService.DbRs.AddLocalFunc("quit_with_help", ussdHandlers.QuitWithHelp) + + return ussdHandlers, nil +} + +func (localHandlerService *LocalHandlerService) GetEngine() *engine.DefaultEngine { + en := engine.NewEngine(localHandlerService.Cfg, localHandlerService.Rs) + en = en.WithPersister(localHandlerService.Pe) + return en +} -- 2.45.2 From 5bd51b280e47668d56be1be756fd50e767dbc24d Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 15:57:11 +0300 Subject: [PATCH 2/8] add storage service for menu --- internal/storage/storageservice.go | 76 ++++++++++++++++++++++++++++++ 1 file changed, 76 insertions(+) create mode 100644 internal/storage/storageservice.go diff --git a/internal/storage/storageservice.go b/internal/storage/storageservice.go new file mode 100644 index 0000000..4306153 --- /dev/null +++ b/internal/storage/storageservice.go @@ -0,0 +1,76 @@ +package storage + +import ( + "context" + "fmt" + "os" + "path" + + "git.defalsify.org/vise.git/db" + fsdb "git.defalsify.org/vise.git/db/fs" + gdbmdb "git.defalsify.org/vise.git/db/gdbm" + "git.defalsify.org/vise.git/persist" + "git.defalsify.org/vise.git/resource" +) + +func ensureDbDir(dbDir string) error { + err := os.MkdirAll(dbDir, 0700) + if err != nil { + return fmt.Errorf("state dir create exited with error: %v\n", err) + } + return nil +} + +type StorageService interface { + GetPersister(dbDir string, ctx context.Context) (*persist.Persister, error) + GetUserdataDb(dbDir string, ctx context.Context) db.Db + GetResource(resourceDir string, ctx context.Context) (resource.Resource, error) + EnsureDbDir(dbDir string) error +} + +type MenuStorageService struct{} + +func (menuStorageService *MenuStorageService) GetPersister(dbDir string, ctx context.Context) (*persist.Persister, error) { + err := ensureDbDir(dbDir) + if err != nil { + return nil, err + } + store := gdbmdb.NewGdbmDb() + storeFile := path.Join(dbDir, "state.gdbm") + store.Connect(ctx, storeFile) + pr := persist.NewPersister(store) + return pr, nil +} + +func (menuStorageService *MenuStorageService) GetUserdataDb(dbDir string, ctx context.Context) db.Db { + store := gdbmdb.NewGdbmDb() + storeFile := path.Join(dbDir, "userdata.gdbm") + store.Connect(ctx, storeFile) + + return store +} + +func (menuStorageService *MenuStorageService) GetResource(resourceDir string, ctx context.Context) (resource.Resource, error) { + store := fsdb.NewFsDb() + err := store.Connect(ctx, resourceDir) + if err != nil { + return nil, err + } + rfs := resource.NewDbResource(store) + return rfs, nil +} + +func (menuStorageService *MenuStorageService) GetStateStore(dbDir string, ctx context.Context) (db.Db, error) { + store := gdbmdb.NewGdbmDb() + storeFile := path.Join(dbDir, "state.gdbm") + store.Connect(ctx, storeFile) + return store, nil +} + +func (menuStorageService *MenuStorageService) EnsureDbDir(dbDir string) error { + err := os.MkdirAll(dbDir, 0700) + if err != nil { + return fmt.Errorf("state dir create exited with error: %v\n", err) + } + return nil +} -- 2.45.2 From 81159e77d1c8f680a051bf50024c7d48ba594d65 Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 16:15:29 +0300 Subject: [PATCH 3/8] refactor code --- cmd/africastalking/main.go | 122 ++++++++----------------------------- 1 file changed, 26 insertions(+), 96 deletions(-) diff --git a/cmd/africastalking/main.go b/cmd/africastalking/main.go index 69122b7..be2ff01 100644 --- a/cmd/africastalking/main.go +++ b/cmd/africastalking/main.go @@ -13,26 +13,23 @@ import ( "syscall" "git.defalsify.org/vise.git/asm" - "git.defalsify.org/vise.git/db" - fsdb "git.defalsify.org/vise.git/db/fs" - gdbmdb "git.defalsify.org/vise.git/db/gdbm" "git.defalsify.org/vise.git/engine" "git.defalsify.org/vise.git/logging" "git.defalsify.org/vise.git/resource" "git.grassecon.net/urdt/ussd/internal/handlers" - "git.grassecon.net/urdt/ussd/internal/handlers/ussd" httpserver "git.grassecon.net/urdt/ussd/internal/http" + "git.grassecon.net/urdt/ussd/internal/storage" ) var ( - logg = logging.NewVanilla() + logg = logging.NewVanilla() scriptDir = path.Join("services", "registration") ) -type atRequestParser struct {} +type atRequestParser struct{} -func(arp *atRequestParser) GetSessionId(rq any) (string, error) { +func (arp *atRequestParser) GetSessionId(rq any) (string, error) { rqv, ok := rq.(*http.Request) if !ok { return "", handlers.ErrInvalidRequest @@ -49,7 +46,7 @@ func(arp *atRequestParser) GetSessionId(rq any) (string, error) { return phoneNumber, nil } -func(arp *atRequestParser) GetInput(rq any) ([]byte, error) { +func (arp *atRequestParser) GetInput(rq any) ([]byte, error) { rqv, ok := rq.(*http.Request) if !ok { return nil, handlers.ErrInvalidRequest @@ -68,7 +65,6 @@ func(arp *atRequestParser) GetInput(rq any) ([]byte, error) { return []byte(parts[len(parts)-1]), nil } - func getFlags(fp string, debug bool) (*asm.FlagParser, error) { flagParser := asm.NewFlagParser().WithDebug() _, err := flagParser.Load(fp) @@ -78,86 +74,6 @@ func getFlags(fp string, debug bool) (*asm.FlagParser, error) { return flagParser, nil } -func getHandler(appFlags *asm.FlagParser, rs *resource.DbResource, userdataStore db.Db) (*ussd.Handlers, error) { - - ussdHandlers, err := ussd.NewHandlers(appFlags, userdataStore) - if err != nil { - return nil, err - } - rs.AddLocalFunc("set_language", ussdHandlers.SetLanguage) - rs.AddLocalFunc("create_account", ussdHandlers.CreateAccount) - rs.AddLocalFunc("save_pin", ussdHandlers.SavePin) - rs.AddLocalFunc("verify_pin", ussdHandlers.VerifyPin) - rs.AddLocalFunc("check_identifier", ussdHandlers.CheckIdentifier) - rs.AddLocalFunc("check_account_status", ussdHandlers.CheckAccountStatus) - rs.AddLocalFunc("authorize_account", ussdHandlers.Authorize) - rs.AddLocalFunc("quit", ussdHandlers.Quit) - rs.AddLocalFunc("check_balance", ussdHandlers.CheckBalance) - rs.AddLocalFunc("validate_recipient", ussdHandlers.ValidateRecipient) - rs.AddLocalFunc("transaction_reset", ussdHandlers.TransactionReset) - rs.AddLocalFunc("max_amount", ussdHandlers.MaxAmount) - rs.AddLocalFunc("validate_amount", ussdHandlers.ValidateAmount) - rs.AddLocalFunc("reset_transaction_amount", ussdHandlers.ResetTransactionAmount) - rs.AddLocalFunc("get_recipient", ussdHandlers.GetRecipient) - rs.AddLocalFunc("get_sender", ussdHandlers.GetSender) - rs.AddLocalFunc("get_amount", ussdHandlers.GetAmount) - rs.AddLocalFunc("reset_incorrect", ussdHandlers.ResetIncorrectPin) - rs.AddLocalFunc("save_firstname", ussdHandlers.SaveFirstname) - rs.AddLocalFunc("save_familyname", ussdHandlers.SaveFamilyname) - rs.AddLocalFunc("save_gender", ussdHandlers.SaveGender) - rs.AddLocalFunc("save_location", ussdHandlers.SaveLocation) - rs.AddLocalFunc("save_yob", ussdHandlers.SaveYob) - rs.AddLocalFunc("save_offerings", ussdHandlers.SaveOfferings) - rs.AddLocalFunc("quit_with_balance", ussdHandlers.QuitWithBalance) - rs.AddLocalFunc("reset_account_authorized", ussdHandlers.ResetAccountAuthorized) - rs.AddLocalFunc("reset_allow_update", ussdHandlers.ResetAllowUpdate) - rs.AddLocalFunc("get_profile_info", ussdHandlers.GetProfileInfo) - rs.AddLocalFunc("verify_yob", ussdHandlers.VerifyYob) - rs.AddLocalFunc("reset_incorrect_date_format", ussdHandlers.ResetIncorrectYob) - rs.AddLocalFunc("set_reset_single_edit", ussdHandlers.SetResetSingleEdit) - rs.AddLocalFunc("initiate_transaction", ussdHandlers.InitiateTransaction) - rs.AddLocalFunc("save_temporary_pin", ussdHandlers.SaveTemporaryPin) - rs.AddLocalFunc("verify_new_pin", ussdHandlers.VerifyNewPin) - rs.AddLocalFunc("confirm_pin_change", ussdHandlers.ConfirmPinChange) - rs.AddLocalFunc("quit_with_help",ussdHandlers.QuitWithHelp) - - return ussdHandlers, nil -} - -func ensureDbDir(dbDir string) error { - err := os.MkdirAll(dbDir, 0700) - if err != nil { - return fmt.Errorf("state dir create exited with error: %v\n", err) - } - return nil -} - -func getStateStore(dbDir string, ctx context.Context) (db.Db, error) { - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "state.gdbm") - store.Connect(ctx, storeFile) - return store, nil -} - -func getUserdataDb(dbDir string, ctx context.Context) db.Db { - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "userdata.gdbm") - store.Connect(ctx, storeFile) - - return store -} - -func getResource(resourceDir string, ctx context.Context) (resource.Resource, error) { - store := fsdb.NewFsDb() - err := store.Connect(ctx, resourceDir) - if err != nil { - return nil, err - } - rfs := resource.NewDbResource(store) - return rfs, nil -} - - func main() { var dbDir string var resourceDir string @@ -175,7 +91,7 @@ func main() { flag.UintVar(&port, "p", 7123, "http port") flag.Parse() - logg.Infof("start command", "dbdir", dbDir, "resourcedir", resourceDir, "outputsize", size) + logg.Infof("start command", "dbdir", dbDir, "resourcedir", resourceDir, "outputsize", size) ctx := context.Background() pfp := path.Join(scriptDir, "pp.csv") @@ -197,19 +113,21 @@ func main() { cfg.EngineDebug = true } - rs, err := getResource(resourceDir, ctx) + menuStorageService := storage.MenuStorageService{} + + rs, err := menuStorageService.GetResource(scriptDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - err = ensureDbDir(dbDir) + err = menuStorageService.EnsureDbDir(dbDir) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - userdataStore := getUserdataDb(dbDir, ctx) + userdataStore := menuStorageService.GetUserdataDb(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) @@ -220,14 +138,26 @@ func main() { if !ok { os.Exit(1) } + lhs := handlers.LocalHandlerService{ + Parser: flagParser, + DbRs: dbResource, + UserdataStore: userdataStore, + Cfg: cfg, + Rs: rs, + } - hl, err := getHandler(flagParser, dbResource, userdataStore) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - stateStore, err := getStateStore(dbDir, ctx) + hl, err := lhs.GetHandler() + if err != nil { + fmt.Fprintf(os.Stderr, err.Error()) + os.Exit(1) + } + + stateStore, err := menuStorageService.GetStateStore(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) @@ -238,7 +168,7 @@ func main() { bsh := handlers.NewBaseSessionHandler(cfg, rs, stateStore, userdataStore, rp, hl) sh := httpserver.NewATSessionHandler(bsh) s := &http.Server{ - Addr: fmt.Sprintf("%s:%s", host, strconv.Itoa(int(port))), + Addr: fmt.Sprintf("%s:%s", host, strconv.Itoa(int(port))), Handler: sh, } s.RegisterOnShutdown(sh.Shutdown) -- 2.45.2 From 13294b42d39bf293805c73106a09bb02913b570b Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 16:16:17 +0300 Subject: [PATCH 4/8] refactor code --- cmd/http/main.go | 104 +++++++++-------------------------------------- 1 file changed, 19 insertions(+), 85 deletions(-) diff --git a/cmd/http/main.go b/cmd/http/main.go index f869291..38fd441 100644 --- a/cmd/http/main.go +++ b/cmd/http/main.go @@ -12,20 +12,17 @@ import ( "syscall" "git.defalsify.org/vise.git/asm" - "git.defalsify.org/vise.git/db" - fsdb "git.defalsify.org/vise.git/db/fs" - gdbmdb "git.defalsify.org/vise.git/db/gdbm" "git.defalsify.org/vise.git/engine" - "git.defalsify.org/vise.git/resource" "git.defalsify.org/vise.git/logging" + "git.defalsify.org/vise.git/resource" - "git.grassecon.net/urdt/ussd/internal/handlers/ussd" "git.grassecon.net/urdt/ussd/internal/handlers" httpserver "git.grassecon.net/urdt/ussd/internal/http" + "git.grassecon.net/urdt/ussd/internal/storage" ) var ( - logg = logging.NewVanilla() + logg = logging.NewVanilla() scriptDir = path.Join("services", "registration") ) @@ -38,52 +35,6 @@ func getFlags(fp string, debug bool) (*asm.FlagParser, error) { return flagParser, nil } -func getHandler(appFlags *asm.FlagParser, rs *resource.DbResource, userdataStore db.Db) (*ussd.Handlers, error) { - - ussdHandlers, err := ussd.NewHandlers(appFlags, userdataStore) - if err != nil { - return nil, err - } - rs.AddLocalFunc("set_language", ussdHandlers.SetLanguage) - rs.AddLocalFunc("create_account", ussdHandlers.CreateAccount) - rs.AddLocalFunc("save_pin", ussdHandlers.SavePin) - rs.AddLocalFunc("verify_pin", ussdHandlers.VerifyPin) - rs.AddLocalFunc("check_identifier", ussdHandlers.CheckIdentifier) - rs.AddLocalFunc("check_account_status", ussdHandlers.CheckAccountStatus) - rs.AddLocalFunc("authorize_account", ussdHandlers.Authorize) - rs.AddLocalFunc("quit", ussdHandlers.Quit) - rs.AddLocalFunc("check_balance", ussdHandlers.CheckBalance) - rs.AddLocalFunc("validate_recipient", ussdHandlers.ValidateRecipient) - rs.AddLocalFunc("transaction_reset", ussdHandlers.TransactionReset) - rs.AddLocalFunc("max_amount", ussdHandlers.MaxAmount) - rs.AddLocalFunc("validate_amount", ussdHandlers.ValidateAmount) - rs.AddLocalFunc("reset_transaction_amount", ussdHandlers.ResetTransactionAmount) - rs.AddLocalFunc("get_recipient", ussdHandlers.GetRecipient) - rs.AddLocalFunc("get_sender", ussdHandlers.GetSender) - rs.AddLocalFunc("get_amount", ussdHandlers.GetAmount) - rs.AddLocalFunc("reset_incorrect", ussdHandlers.ResetIncorrectPin) - rs.AddLocalFunc("save_firstname", ussdHandlers.SaveFirstname) - rs.AddLocalFunc("save_familyname", ussdHandlers.SaveFamilyname) - rs.AddLocalFunc("save_gender", ussdHandlers.SaveGender) - rs.AddLocalFunc("save_location", ussdHandlers.SaveLocation) - rs.AddLocalFunc("save_yob", ussdHandlers.SaveYob) - rs.AddLocalFunc("save_offerings", ussdHandlers.SaveOfferings) - rs.AddLocalFunc("quit_with_balance", ussdHandlers.QuitWithBalance) - rs.AddLocalFunc("reset_account_authorized", ussdHandlers.ResetAccountAuthorized) - rs.AddLocalFunc("reset_allow_update", ussdHandlers.ResetAllowUpdate) - rs.AddLocalFunc("get_profile_info", ussdHandlers.GetProfileInfo) - rs.AddLocalFunc("verify_yob", ussdHandlers.VerifyYob) - rs.AddLocalFunc("reset_incorrect_date_format", ussdHandlers.ResetIncorrectYob) - rs.AddLocalFunc("set_reset_single_edit", ussdHandlers.SetResetSingleEdit) - rs.AddLocalFunc("initiate_transaction", ussdHandlers.InitiateTransaction) - rs.AddLocalFunc("save_temporary_pin", ussdHandlers.SaveTemporaryPin) - rs.AddLocalFunc("verify_new_pin", ussdHandlers.VerifyNewPin) - rs.AddLocalFunc("confirm_pin_change", ussdHandlers.ConfirmPinChange) - rs.AddLocalFunc("quit_with_help",ussdHandlers.QuitWithHelp) - - return ussdHandlers, nil -} - func ensureDbDir(dbDir string) error { err := os.MkdirAll(dbDir, 0700) if err != nil { @@ -92,32 +43,6 @@ func ensureDbDir(dbDir string) error { return nil } -func getStateStore(dbDir string, ctx context.Context) (db.Db, error) { - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "state.gdbm") - store.Connect(ctx, storeFile) - return store, nil -} - -func getUserdataDb(dbDir string, ctx context.Context) db.Db { - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "userdata.gdbm") - store.Connect(ctx, storeFile) - - return store -} - -func getResource(resourceDir string, ctx context.Context) (resource.Resource, error) { - store := fsdb.NewFsDb() - err := store.Connect(ctx, resourceDir) - if err != nil { - return nil, err - } - rfs := resource.NewDbResource(store) - return rfs, nil -} - - func main() { var dbDir string var resourceDir string @@ -135,7 +60,7 @@ func main() { flag.UintVar(&port, "p", 7123, "http port") flag.Parse() - logg.Infof("start command", "dbdir", dbDir, "resourcedir", resourceDir, "outputsize", size) + logg.Infof("start command", "dbdir", dbDir, "resourcedir", resourceDir, "outputsize", size) ctx := context.Background() pfp := path.Join(scriptDir, "pp.csv") @@ -157,19 +82,20 @@ func main() { cfg.EngineDebug = true } - rs, err := getResource(resourceDir, ctx) + menuStorageService := storage.MenuStorageService{} + rs, err := menuStorageService.GetResource(scriptDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - err = ensureDbDir(dbDir) + err = menuStorageService.EnsureDbDir(dbDir) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - userdataStore := getUserdataDb(dbDir, ctx) + userdataStore := menuStorageService.GetUserdataDb(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) @@ -181,13 +107,21 @@ func main() { os.Exit(1) } - hl, err := getHandler(flagParser, dbResource, userdataStore) + lhs := handlers.LocalHandlerService{ + Parser: flagParser, + DbRs: dbResource, + UserdataStore: userdataStore, + Cfg: cfg, + Rs: rs, + } + + hl, err := lhs.GetHandler() if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - stateStore, err := getStateStore(dbDir, ctx) + stateStore, err := menuStorageService.GetStateStore(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) @@ -198,7 +132,7 @@ func main() { bsh := handlers.NewBaseSessionHandler(cfg, rs, stateStore, userdataStore, rp, hl) sh := httpserver.ToSessionHandler(bsh) s := &http.Server{ - Addr: fmt.Sprintf("%s:%s", host, strconv.Itoa(int(port))), + Addr: fmt.Sprintf("%s:%s", host, strconv.Itoa(int(port))), Handler: sh, } s.RegisterOnShutdown(sh.Shutdown) -- 2.45.2 From ffa00ae15c1dbe2722eb65be4950e5b02b236ad2 Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 16:18:21 +0300 Subject: [PATCH 5/8] refactor code --- cmd/async/main.go | 120 +++++++++------------------------------------- 1 file changed, 23 insertions(+), 97 deletions(-) diff --git a/cmd/async/main.go b/cmd/async/main.go index e194d18..5b8aa36 100644 --- a/cmd/async/main.go +++ b/cmd/async/main.go @@ -10,32 +10,29 @@ import ( "syscall" "git.defalsify.org/vise.git/asm" - "git.defalsify.org/vise.git/db" - fsdb "git.defalsify.org/vise.git/db/fs" - gdbmdb "git.defalsify.org/vise.git/db/gdbm" "git.defalsify.org/vise.git/engine" - "git.defalsify.org/vise.git/resource" "git.defalsify.org/vise.git/logging" + "git.defalsify.org/vise.git/resource" - "git.grassecon.net/urdt/ussd/internal/handlers/ussd" "git.grassecon.net/urdt/ussd/internal/handlers" + "git.grassecon.net/urdt/ussd/internal/storage" ) var ( - logg = logging.NewVanilla() + logg = logging.NewVanilla() scriptDir = path.Join("services", "registration") ) type asyncRequestParser struct { sessionId string - input []byte + input []byte } -func(p *asyncRequestParser) GetSessionId(r any) (string, error) { +func (p *asyncRequestParser) GetSessionId(r any) (string, error) { return p.sessionId, nil } -func(p *asyncRequestParser) GetInput(r any) ([]byte, error) { +func (p *asyncRequestParser) GetInput(r any) ([]byte, error) { return p.input, nil } @@ -48,86 +45,6 @@ func getFlags(fp string, debug bool) (*asm.FlagParser, error) { return flagParser, nil } -func getHandler(appFlags *asm.FlagParser, rs *resource.DbResource, userdataStore db.Db) (*ussd.Handlers, error) { - - ussdHandlers, err := ussd.NewHandlers(appFlags, userdataStore) - if err != nil { - return nil, err - } - rs.AddLocalFunc("set_language", ussdHandlers.SetLanguage) - rs.AddLocalFunc("create_account", ussdHandlers.CreateAccount) - rs.AddLocalFunc("save_pin", ussdHandlers.SavePin) - rs.AddLocalFunc("verify_pin", ussdHandlers.VerifyPin) - rs.AddLocalFunc("check_identifier", ussdHandlers.CheckIdentifier) - rs.AddLocalFunc("check_account_status", ussdHandlers.CheckAccountStatus) - rs.AddLocalFunc("authorize_account", ussdHandlers.Authorize) - rs.AddLocalFunc("quit", ussdHandlers.Quit) - rs.AddLocalFunc("check_balance", ussdHandlers.CheckBalance) - rs.AddLocalFunc("validate_recipient", ussdHandlers.ValidateRecipient) - rs.AddLocalFunc("transaction_reset", ussdHandlers.TransactionReset) - rs.AddLocalFunc("max_amount", ussdHandlers.MaxAmount) - rs.AddLocalFunc("validate_amount", ussdHandlers.ValidateAmount) - rs.AddLocalFunc("reset_transaction_amount", ussdHandlers.ResetTransactionAmount) - rs.AddLocalFunc("get_recipient", ussdHandlers.GetRecipient) - rs.AddLocalFunc("get_sender", ussdHandlers.GetSender) - rs.AddLocalFunc("get_amount", ussdHandlers.GetAmount) - rs.AddLocalFunc("reset_incorrect", ussdHandlers.ResetIncorrectPin) - rs.AddLocalFunc("save_firstname", ussdHandlers.SaveFirstname) - rs.AddLocalFunc("save_familyname", ussdHandlers.SaveFamilyname) - rs.AddLocalFunc("save_gender", ussdHandlers.SaveGender) - rs.AddLocalFunc("save_location", ussdHandlers.SaveLocation) - rs.AddLocalFunc("save_yob", ussdHandlers.SaveYob) - rs.AddLocalFunc("save_offerings", ussdHandlers.SaveOfferings) - rs.AddLocalFunc("quit_with_balance", ussdHandlers.QuitWithBalance) - rs.AddLocalFunc("reset_account_authorized", ussdHandlers.ResetAccountAuthorized) - rs.AddLocalFunc("reset_allow_update", ussdHandlers.ResetAllowUpdate) - rs.AddLocalFunc("get_profile_info", ussdHandlers.GetProfileInfo) - rs.AddLocalFunc("verify_yob", ussdHandlers.VerifyYob) - rs.AddLocalFunc("reset_incorrect_date_format", ussdHandlers.ResetIncorrectYob) - rs.AddLocalFunc("set_reset_single_edit", ussdHandlers.SetResetSingleEdit) - rs.AddLocalFunc("initiate_transaction", ussdHandlers.InitiateTransaction) - rs.AddLocalFunc("save_temporary_pin", ussdHandlers.SaveTemporaryPin) - rs.AddLocalFunc("verify_new_pin", ussdHandlers.VerifyNewPin) - rs.AddLocalFunc("confirm_pin_change", ussdHandlers.ConfirmPinChange) - rs.AddLocalFunc("quit_with_help",ussdHandlers.QuitWithHelp) - - return ussdHandlers, nil -} - -func ensureDbDir(dbDir string) error { - err := os.MkdirAll(dbDir, 0700) - if err != nil { - return fmt.Errorf("state dir create exited with error: %v\n", err) - } - return nil -} - -func getStateStore(dbDir string, ctx context.Context) (db.Db, error) { - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "state.gdbm") - store.Connect(ctx, storeFile) - return store, nil -} - -func getUserdataDb(dbDir string, ctx context.Context) db.Db { - store := gdbmdb.NewGdbmDb() - storeFile := path.Join(dbDir, "userdata.gdbm") - store.Connect(ctx, storeFile) - - return store -} - -func getResource(resourceDir string, ctx context.Context) (resource.Resource, error) { - store := fsdb.NewFsDb() - err := store.Connect(ctx, resourceDir) - if err != nil { - return nil, err - } - rfs := resource.NewDbResource(store) - return rfs, nil -} - - func main() { var sessionId string var dbDir string @@ -147,7 +64,7 @@ func main() { flag.UintVar(&port, "p", 7123, "http port") flag.Parse() - logg.Infof("start command", "dbdir", dbDir, "resourcedir", resourceDir, "outputsize", size, "sessionId", sessionId) + logg.Infof("start command", "dbdir", dbDir, "resourcedir", resourceDir, "outputsize", size, "sessionId", sessionId) ctx := context.Background() pfp := path.Join(scriptDir, "pp.csv") @@ -169,19 +86,20 @@ func main() { cfg.EngineDebug = true } - rs, err := getResource(resourceDir, ctx) + menuStorageService := storage.MenuStorageService{} + rs, err := menuStorageService.GetResource(scriptDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - err = ensureDbDir(dbDir) + err = menuStorageService.EnsureDbDir(dbDir) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - userdataStore := getUserdataDb(dbDir, ctx) + userdataStore := menuStorageService.GetUserdataDb(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) @@ -193,13 +111,21 @@ func main() { os.Exit(1) } - hl, err := getHandler(flagParser, dbResource, userdataStore) + lhs := handlers.LocalHandlerService{ + Parser: flagParser, + DbRs: dbResource, + UserdataStore: userdataStore, + Cfg: cfg, + Rs: rs, + } + + hl, err := lhs.GetHandler() if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) } - stateStore, err := getStateStore(dbDir, ctx) + stateStore, err := menuStorageService.GetStateStore(dbDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) os.Exit(1) @@ -212,7 +138,7 @@ func main() { sh := handlers.NewBaseSessionHandler(cfg, rs, stateStore, userdataStore, rp, hl) cfg.SessionId = sessionId rqs := handlers.RequestSession{ - Ctx: ctx, + Ctx: ctx, Writer: os.Stdout, Config: cfg, } @@ -248,7 +174,7 @@ func main() { fmt.Println("") _, err = fmt.Scanln(&rqs.Input) if err != nil { - fmt.Errorf("error in input: %v", err) + fmt.Errorf("error in input: %v", err) os.Exit(1) } } -- 2.45.2 From bc9dfe4f65f5586a2edd929f4f021700a670ec3a Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 17:01:23 +0300 Subject: [PATCH 6/8] remove redundant code --- cmd/main.go | 6 ++++++ internal/storage/storageservice.go | 4 ---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/cmd/main.go b/cmd/main.go index 7e5bc0d..ab36f30 100644 --- a/cmd/main.go +++ b/cmd/main.go @@ -45,6 +45,12 @@ func main() { menuStorageService := storage.MenuStorageService{} + err := menuStorageService.EnsureDbDir(dbDir) + if err != nil { + fmt.Fprintf(os.Stderr, err.Error()) + os.Exit(1) + } + rs, err := menuStorageService.GetResource(scriptDir, ctx) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) diff --git a/internal/storage/storageservice.go b/internal/storage/storageservice.go index 4306153..b31b67d 100644 --- a/internal/storage/storageservice.go +++ b/internal/storage/storageservice.go @@ -31,10 +31,6 @@ type StorageService interface { type MenuStorageService struct{} func (menuStorageService *MenuStorageService) GetPersister(dbDir string, ctx context.Context) (*persist.Persister, error) { - err := ensureDbDir(dbDir) - if err != nil { - return nil, err - } store := gdbmdb.NewGdbmDb() storeFile := path.Join(dbDir, "state.gdbm") store.Connect(ctx, storeFile) -- 2.45.2 From 96358959b4444c13374ba2c8defcafc764389188 Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 17:04:08 +0300 Subject: [PATCH 7/8] remove unused code --- internal/storage/storageservice.go | 8 -------- 1 file changed, 8 deletions(-) diff --git a/internal/storage/storageservice.go b/internal/storage/storageservice.go index b31b67d..a3f50d6 100644 --- a/internal/storage/storageservice.go +++ b/internal/storage/storageservice.go @@ -13,14 +13,6 @@ import ( "git.defalsify.org/vise.git/resource" ) -func ensureDbDir(dbDir string) error { - err := os.MkdirAll(dbDir, 0700) - if err != nil { - return fmt.Errorf("state dir create exited with error: %v\n", err) - } - return nil -} - type StorageService interface { GetPersister(dbDir string, ctx context.Context) (*persist.Persister, error) GetUserdataDb(dbDir string, ctx context.Context) db.Db -- 2.45.2 From e581a1ad2f71c89be521d7a792cb988bed3df53f Mon Sep 17 00:00:00 2001 From: Carlosokumu Date: Thu, 19 Sep 2024 21:04:09 +0300 Subject: [PATCH 8/8] factor out getflag --- cmd/africastalking/main.go | 25 +++------------------ cmd/async/main.go | 24 ++------------------ cmd/http/main.go | 35 +++++------------------------ cmd/main.go | 4 +++- internal/handlers/handlerservice.go | 24 ++++++++++++-------- 5 files changed, 29 insertions(+), 83 deletions(-) diff --git a/cmd/africastalking/main.go b/cmd/africastalking/main.go index be2ff01..c0db723 100644 --- a/cmd/africastalking/main.go +++ b/cmd/africastalking/main.go @@ -12,7 +12,6 @@ import ( "strings" "syscall" - "git.defalsify.org/vise.git/asm" "git.defalsify.org/vise.git/engine" "git.defalsify.org/vise.git/logging" "git.defalsify.org/vise.git/resource" @@ -65,15 +64,6 @@ func (arp *atRequestParser) GetInput(rq any) ([]byte, error) { return []byte(parts[len(parts)-1]), nil } -func getFlags(fp string, debug bool) (*asm.FlagParser, error) { - flagParser := asm.NewFlagParser().WithDebug() - _, err := flagParser.Load(fp) - if err != nil { - return nil, err - } - return flagParser, nil -} - func main() { var dbDir string var resourceDir string @@ -95,11 +85,6 @@ func main() { ctx := context.Background() pfp := path.Join(scriptDir, "pp.csv") - flagParser, err := getFlags(pfp, true) - - if err != nil { - os.Exit(1) - } cfg := engine.Config{ Root: "root", @@ -138,13 +123,9 @@ func main() { if !ok { os.Exit(1) } - lhs := handlers.LocalHandlerService{ - Parser: flagParser, - DbRs: dbResource, - UserdataStore: userdataStore, - Cfg: cfg, - Rs: rs, - } + + lhs, err := handlers.NewLocalHandlerService(pfp, true, dbResource, cfg, rs) + lhs.WithDataStore(&userdataStore) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) diff --git a/cmd/async/main.go b/cmd/async/main.go index 5b8aa36..2ff0d1e 100644 --- a/cmd/async/main.go +++ b/cmd/async/main.go @@ -9,7 +9,6 @@ import ( "path" "syscall" - "git.defalsify.org/vise.git/asm" "git.defalsify.org/vise.git/engine" "git.defalsify.org/vise.git/logging" "git.defalsify.org/vise.git/resource" @@ -36,15 +35,6 @@ func (p *asyncRequestParser) GetInput(r any) ([]byte, error) { return p.input, nil } -func getFlags(fp string, debug bool) (*asm.FlagParser, error) { - flagParser := asm.NewFlagParser().WithDebug() - _, err := flagParser.Load(fp) - if err != nil { - return nil, err - } - return flagParser, nil -} - func main() { var sessionId string var dbDir string @@ -68,11 +58,6 @@ func main() { ctx := context.Background() pfp := path.Join(scriptDir, "pp.csv") - flagParser, err := getFlags(pfp, true) - - if err != nil { - os.Exit(1) - } cfg := engine.Config{ Root: "root", @@ -111,13 +96,8 @@ func main() { os.Exit(1) } - lhs := handlers.LocalHandlerService{ - Parser: flagParser, - DbRs: dbResource, - UserdataStore: userdataStore, - Cfg: cfg, - Rs: rs, - } + lhs, err := handlers.NewLocalHandlerService(pfp, true, dbResource, cfg, rs) + lhs.WithDataStore(&userdataStore) hl, err := lhs.GetHandler() if err != nil { diff --git a/cmd/http/main.go b/cmd/http/main.go index 38fd441..7b74e19 100644 --- a/cmd/http/main.go +++ b/cmd/http/main.go @@ -11,7 +11,6 @@ import ( "strconv" "syscall" - "git.defalsify.org/vise.git/asm" "git.defalsify.org/vise.git/engine" "git.defalsify.org/vise.git/logging" "git.defalsify.org/vise.git/resource" @@ -26,23 +25,6 @@ var ( scriptDir = path.Join("services", "registration") ) -func getFlags(fp string, debug bool) (*asm.FlagParser, error) { - flagParser := asm.NewFlagParser().WithDebug() - _, err := flagParser.Load(fp) - if err != nil { - return nil, err - } - return flagParser, nil -} - -func ensureDbDir(dbDir string) error { - err := os.MkdirAll(dbDir, 0700) - if err != nil { - return fmt.Errorf("state dir create exited with error: %v\n", err) - } - return nil -} - func main() { var dbDir string var resourceDir string @@ -64,11 +46,6 @@ func main() { ctx := context.Background() pfp := path.Join(scriptDir, "pp.csv") - flagParser, err := getFlags(pfp, true) - - if err != nil { - os.Exit(1) - } cfg := engine.Config{ Root: "root", @@ -107,12 +84,12 @@ func main() { os.Exit(1) } - lhs := handlers.LocalHandlerService{ - Parser: flagParser, - DbRs: dbResource, - UserdataStore: userdataStore, - Cfg: cfg, - Rs: rs, + lhs, err := handlers.NewLocalHandlerService(pfp, true, dbResource, cfg, rs) + lhs.WithDataStore(&userdataStore) + + if err != nil { + fmt.Fprintf(os.Stderr, err.Error()) + os.Exit(1) } hl, err := lhs.GetHandler() diff --git a/cmd/main.go b/cmd/main.go index ab36f30..7176bd3 100644 --- a/cmd/main.go +++ b/cmd/main.go @@ -75,7 +75,9 @@ func main() { os.Exit(1) } - lhs, err := handlers.NewLocalHandlerService(pfp, true, dbResource, pe, userdatastore, cfg, rs) + lhs, err := handlers.NewLocalHandlerService(pfp, true, dbResource, cfg, rs) + lhs.WithDataStore(&userdatastore) + lhs.WithPersister(pe) if err != nil { fmt.Fprintf(os.Stderr, err.Error()) diff --git a/internal/handlers/handlerservice.go b/internal/handlers/handlerservice.go index e6d4fb6..93c5a10 100644 --- a/internal/handlers/handlerservice.go +++ b/internal/handlers/handlerservice.go @@ -26,28 +26,34 @@ type LocalHandlerService struct { Parser *asm.FlagParser DbRs *resource.DbResource Pe *persist.Persister - UserdataStore db.Db + UserdataStore *db.Db Cfg engine.Config Rs resource.Resource } -func NewLocalHandlerService(fp string, debug bool, dbResource *resource.DbResource, Pe *persist.Persister, userDataStore db.Db, cfg engine.Config, rs resource.Resource) (*LocalHandlerService, error) { +func NewLocalHandlerService(fp string, debug bool, dbResource *resource.DbResource, cfg engine.Config, rs resource.Resource) (*LocalHandlerService, error) { parser, err := getParser(fp, debug) if err != nil { return nil, err } return &LocalHandlerService{ - Parser: parser, - DbRs: dbResource, - Pe: Pe, - UserdataStore: userDataStore, - Cfg: cfg, - Rs: rs, + Parser: parser, + DbRs: dbResource, + Cfg: cfg, + Rs: rs, }, nil } +func (localHandlerService *LocalHandlerService) WithPersister(Pe *persist.Persister) { + localHandlerService.Pe = Pe +} + +func (localHandlerService *LocalHandlerService) WithDataStore(db *db.Db) { + localHandlerService.UserdataStore = db +} + func (localHandlerService *LocalHandlerService) GetHandler() (*ussd.Handlers, error) { - ussdHandlers, err := ussd.NewHandlers(localHandlerService.Parser, localHandlerService.UserdataStore) + ussdHandlers, err := ussd.NewHandlers(localHandlerService.Parser, *localHandlerService.UserdataStore) if err != nil { return nil, err } -- 2.45.2