Skip to content

Рассмотреть передачу зависимостей через context.Context #2

Description

@EgorBron

Поскольку встроенная структура Context поддерживает передачу зависимостей, можно использовать её, а не поле Deps у контекста команд.

Вопросы для обдумывания:

  • vk-sdk уже предоставляет контекст в обработчике события, нужно решить, передавать ли в обработчик этот контекст или новый context.Background();
    • сюда же: стоит ли объединять эти контексты в пользу языкового с небольшой обёрткой из библиотеки:
      func Handler(ctx context.Context) {
        vkCtx, err := vkc.SeparateContext(ctx)
        // обработка ошибки
        vkCtx.SendText("Hello world!")
      } 
  • так или иначе нужна обратная совместимость со старым подходом;
  • контекст из языка и контекст из библиотеки пересекаются по названиям.

Metadata

Metadata

Assignees

Labels

enhancementNew feature or requestpriority:normalWill be resolved with a normal priority

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions