-
Notifications
You must be signed in to change notification settings - Fork 31
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
api/auth: Allow authorization to be used for viewership metrics API #1279
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov Report
@@ Coverage Diff @@
## master #1279 +/- ##
===================================================
+ Coverage 51.06138% 51.37192% +0.31054%
===================================================
Files 66 66
Lines 4334 4337 +3
Branches 786 786
===================================================
+ Hits 2213 2228 +15
+ Misses 1842 1830 -12
Partials 279 279
Continue to review full report at Codecov.
|
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.
LGTM
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.
This reverts commit 010a96a.
8e6aad3
to
fb96cdd
Compare
What does this pull request do? Explain your changes. (required)
This allows the authorization API, currently used for stream health, to also
be used for the viewership API. This mostly means adding support for authorizing
access to a given asset as well, not only to a stream. Also adding CORS support
on the analyzer endpoint.
This goes together with livepeer/livepeer-data#60
Specific updates (required)
-
yarn test
Does this pull request close any open issues?
Closes #1278
Checklist: