-
-
Notifications
You must be signed in to change notification settings - Fork 775
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
Bug/4817 abuse of invite system #4945
Conversation
- invite_url variable removed from the dom - invite_url removed from the client request when sharing a bounty
Codecov Report
@@ Coverage Diff @@
## master #4945 +/- ##
==========================================
+ Coverage 30.63% 30.63% +<.01%
==========================================
Files 216 216
Lines 17496 17494 -2
Branches 2389 2388 -1
==========================================
Hits 5360 5360
+ Misses 11920 11917 -3
- Partials 216 217 +1
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #4945 +/- ##
==========================================
+ Coverage 30.63% 30.63% +<.01%
==========================================
Files 216 216
Lines 17496 17494 -2
Branches 2389 2388 -1
==========================================
Hits 5360 5360
+ Misses 11920 11917 -3
- Partials 216 217 +1
Continue to review full report at Codecov.
|
Not sure why codecoverage on my patch is coming in at 0, all i did was remove a couple of variables from global js scope and from the client/server request handler. |
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
Don't worry about codecoverage, its more of a reminder to add coverage if appropriate |
Description
Removed invite_url from the JavaScript global scope
Removed invite_url from the share bounty client request
Removed invite_url from the share bounty server request handler
Refers/Fixes
#4817
Testing
Email's that are sent contain an invite_url generated correctly by the server