Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Thrift over HTTP #1

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open

WIP: Thrift over HTTP #1

wants to merge 36 commits into from

Conversation

Ecialo
Copy link
Owner

@Ecialo Ecialo commented Aug 24, 2020

Добавлено:

  • Binary.Framed.Server.HTTP. Получает трифтовое сообщение в /thrift/ и перенапраляет в ручку соотвествующую вызываемому методу. В ручку можно сходить и отдельно через /thrift/:method.
  • Допустимые ошибки теперь нотируются в генерируемом поведении

Не сделано (пока что):

  • Пользовательские плаги на влёте и вылете из ручек
  • Осмысленная обработка односторонних сообщений

Сломано (пока что):

  • Поддержка новго поведения для старого framed сервера.

@Ecialo
Copy link
Owner Author

Ecialo commented Aug 24, 2020

Ещё на тип void стоит повнимательнее посмотреть. Сейчас там в спеках на него генерируется no_return() что явная неправда в случае http.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant