d-kv / signer Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 0.0 769 KB

Go 100.00%

signer's People

Contributors

Watchers

signer's Issues

fix incorrect location for some service

signer/command-executor/internal/entity/commands.go

Lines 3 to 5 in 4d26a41

 type DataBaseCommand interface { 

 Convert() ApiEntity 

 }

Насколько я понимаю это не сущность поэтому ей скорее всего следует лежать в interfaces

Добавить multi-stage сборку для экономии памяти

Для пущей скорости и стабильности сервиса - переписать обработку с синхрона на асинхроные операции - но оно пока и не нужно

Неверный формат ответов с bundleid

при запросе на создание подаётся на вход именно название и идентификатор в формате "com.mc. ..." в ответ в поле id даётся bundleId

Для багсерча и просто красоты можно подключить трэйсинг

https://habr.com/ru/articles/710644/

Vault Development mode should NOT be used in production installations!

vault | WARNING! dev mode is enabled! In this mode, Vault runs entirely in-memory
vault | and starts unsealed with a single unseal key. The root token is already
vault | authenticated to the CLI, so you can immediately begin using Vault.

Vault add 1 more field

keyId field needs to be added

Конвертация сущностей

Подумать над более удобной конвертацией сущностей

CI/CD pipelines with different tests

fix id type from uint to string

ID -> string

Tennants/integrations info in repo

На примере bunleId, у нас в бд у него зависимость с интеграцией, соответственно при создании нужно найти интеграцию по id, который мы таскаем в очереди. Открывается вопрос, какой сценарий занесения в бд информации о теннантах и интеграциях.

Подумать о дедупликации запросов

https://yandex.cloud/ru/docs/message-queue/concepts/deduplication?utm_referrer=https%3A%2F%2Fyandex.ru%2F

Дублирование кода

signer/command-executor/internal/repo/commands_postgres.go

Lines 16 to 46 in 51a874b

 type PostgresConfig struct { 

 Host string 

 User string 

 Password string 

 Name string 

 Port string 

 } 

 func New(conf PostgresConfig) *CommandRepo { 

 dsn := fmt.Sprintf( 

 "host=%s user=%s password=%s dbname=%s port=%s sslmode=disable", 

 conf.Host, 

 conf.User, 

 conf.Password, 

 conf.Name, 

 conf.Port, 

 ) 

 db, err := gorm.Open(postgres.Open(dsn), &gorm.Config{}) 

 if err != nil { 

 log.Fatal("Error open gorm connection wti db", err) 

 } 

 err = db.AutoMigrate( 

 &entity.EnableCapabilityType{}, 

 &entity.CreateDevice{}, 

 &entity.CreateBundleId{}, 

 ) 

 if err != nil { 

 log.Fatal(err) 

 } 

 return &CommandRepo{db: db} 

 }

Одни и те же структура и метод в command-executor/internal/repo/commands_postgres.go и command-executor/internal/repo/domain_repo.go

	type PostgresConfig struct {
	Host string
	User string
	Password string
	Name string
	Port string
	}

	func New(conf PostgresConfig) *CommandRepo {
	dsn := fmt.Sprintf(
	"host=%s user=%s password=%s dbname=%s port=%s sslmode=disable",
	conf.Host,
	conf.User,
	conf.Password,
	conf.Name,
	conf.Port,
	)
	db, err := gorm.Open(postgres.Open(dsn), &gorm.Config{})
	if err != nil {
	log.Fatal("Error open gorm connection wti db", err)
	}
	err = db.AutoMigrate(
	&entity.EnableCapabilityType{},
	&entity.CreateDevice{},
	&entity.CreateBundleId{},
	)
	if err != nil {
	log.Fatal(err)
	}
	return &CommandRepo{db: db}
	}

d-kv / signer Goto Github PK

signer's People

Contributors

Watchers

signer's Issues

Recommend Projects

Recommend Topics

Recommend Org