-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
[spiral/router] Opening of http.request
scope added
#1069
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## feature/scopes #1069 +/- ##
====================================================
- Coverage 89.53% 89.49% -0.04%
- Complexity 6318 6327 +9
====================================================
Files 824 825 +1
Lines 17799 17831 +32
====================================================
+ Hits 15936 15958 +22
- Misses 1863 1873 +10 ☔ View full report in Codecov by Sentry. |
return $this->app->getHttp()->handle( | ||
$this->request($uri, 'POST', [], $headers, $cookies)->withParsedBody($data) | ||
return $this->app->getContainer()->runScope( | ||
new Scope(ScopeName::Http), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we will come up with better name for the enum before release
6117455
to
dd40fb1
Compare
dd40fb1
to
a82dcfe
Compare
What was changed
Opening of
http.request
scope added,Proxy
attributes added to get dependencies in the correct scope.