-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix overwriting security definitions on call Server.SetAPI() #2140
Fix overwriting security definitions on call Server.SetAPI() #2140
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2140 +/- ##
=======================================
Coverage 80.11% 80.11%
=======================================
Files 47 47
Lines 8254 8254
=======================================
Hits 6613 6613
Misses 1122 1122
Partials 519 519 Continue to review full report at Codecov.
|
Signed-off-by: v.votinov <v.votinov@xsolla.com>
8ca9ed5
to
2ad3fba
Compare
can you run |
Signed-off-by: v.votinov <v.votinov@xsolla.com>
5074009
to
ecce65d
Compare
Done |
@casualjim, hi! Can review PR? |
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.
Great job!
Just a few nitpicks on bindata version. Apologies: we sure should have warned you beforehand.
c95771e
to
e6acd6c
Compare
@h0mjam we are just a commit signature away... Hang on, this all looks good to me. |
Signed-off-by: v.votinov <v.votinov@xsolla.com>
e6acd6c
to
f4d5867
Compare
On call Server.SetAPI() trigger internal function configureAPI() which is overwrite above defined (into api object) secirutyDefinitions