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

Movendo patched_startup para asgard.app #166

Merged
merged 2 commits into from Aug 14, 2019
Merged

Conversation

@daltonmatos
Copy link
Member

daltonmatos commented Aug 12, 2019

Acabou ficando perdido no asgard.api.agents

Esse código existe por uma falta de funcionalidade no asynworker.
O asyncworker até possui uma infra-estrutura de startup() e
shutdown() mas ele atualmente faz isso após o "listen" da app
http. E o aiohttp não permite mexer na app após esse ponto.

O que esse patched_startup faz é configurar um CORS (ainda inseguro)
para todas as rotas. E precisamos fazer isso antes do "listen".

Quando o asyncworker tiver esse processo de startup divido em dois (pre
listen e pos-listen) poderemos remover esse código.

Acabou ficando perdido no asgard.api.agents

Esse código existe por uma falta de funcionalidade no asynworker.
O asyncworker até possui uma infra-estrutura de `startup()` e
`shutdown()` mas ele atualmente faz isso **após** o "listen" da app
http. E o aiohttp não permite mexer na app após esse ponto.

O que esse patched_startup faz é configurar um CORS (ainda inseguro)
para todas as rotas. E precisamos fazer isso antes do "listen".

Quando o asyncworker tiver esse processo de startup divido em dois (pre
listen e pos-listen) poderemos remover esse código.
@codecov

This comment has been minimized.

Copy link

codecov bot commented Aug 12, 2019

Codecov Report

Merging #166 into master will increase coverage by 0.42%.
The diff coverage is 79.16%.

Flag Coverage Δ
#typehint 44.94% <0%> (+0.03%) ⬆️
#unittest 96.29% <100%> (+0.47%) ⬆️
Impacted Files Coverage Δ
asgard/api/agents.py 84.05% <ø> (+16.31%) ⬆️
asgard/app.py 83.33% <79.16%> (-16.67%) ⬇️
@daltonmatos daltonmatos merged commit 8006d9e into master Aug 14, 2019
6 of 7 checks passed
6 of 7 checks passed
codecov/patch/typehint 0% of diff hit (target 44.91%)
Details
ci/circleci: py368 Your tests passed on CircleCI!
Details
ci/circleci: py36x Your tests passed on CircleCI!
Details
ci/circleci: py37x Your tests passed on CircleCI!
Details
codecov/patch/unittest 100% of diff hit (target 95.81%)
Details
codecov/project/typehint 44.94% (+0.03%) compared to ba6818f
Details
codecov/project/unittest 96.29% (+0.47%) compared to ba6818f
Details
@daltonmatos daltonmatos deleted the feature/move-patched-startup branch Aug 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.