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

Bump to DOMPurify 0.8.6 #4426

Merged
merged 2 commits into from
Apr 21, 2017
Merged

Bump to DOMPurify 0.8.6 #4426

merged 2 commits into from
Apr 21, 2017

Conversation

LukasReschke
Copy link
Member

Fixes #4424

Signed-off-by: Lukas Reschke lukas@statuscode.ch

Fixes #4424

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschke LukasReschke added the 3. to review Waiting for reviews label Apr 21, 2017
@LukasReschke LukasReschke added this to the Nextcloud 12.0 milestone Apr 21, 2017
@mention-bot
Copy link

@LukasReschke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @rullzer to be a potential reviewer.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
Copy link
Member

rullzer commented Apr 21, 2017

Updated core.js to actually use the new code ;)

@codecov
Copy link

codecov bot commented Apr 21, 2017

Codecov Report

Merging #4426 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4426      +/-   ##
============================================
+ Coverage     54.18%   54.18%   +<.01%     
  Complexity    21635    21635              
============================================
  Files          1327     1327              
  Lines         82778    82778              
  Branches       1312     1312              
============================================
+ Hits          44854    44855       +1     
+ Misses        37924    37923       -1
Impacted Files Coverage Δ Complexity Δ
apps/files_trashbin/lib/Trashbin.php 73.39% <0%> (-0.25%) 133% <0%> (ø)
apps/comments/lib/EventHandler.php 87.5% <0%> (+8.33%) 7% <0%> (ø) ⬇️

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine by me!

@MorrisJobke MorrisJobke merged commit 1666a17 into master Apr 21, 2017
@MorrisJobke MorrisJobke deleted the update-dom-purify branch April 21, 2017 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants