Bug 1184130 - Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
Summary: Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Josef Stribny
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1184910 1184920
TreeView+ depends on / blocked
 
Reported: 2015-01-20 15:47 UTC by Vít Ondruch
Modified: 2016-01-04 05:53 UTC (History)
3 users (show)

Fixed In Version: rubygem-rails-deprecated_sanitizer-1.0.3-2.fc22
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-01-23 09:02:44 UTC
Type: ---
Embargoed:
jstribny: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Vít Ondruch 2015-01-20 15:47:26 UTC
Spec URL: https://fedorapeople.org/cgit/vondruch/public_git/rubygem-rails-deprecated_sanitizer.git/tree/rubygem-rails-deprecated_sanitizer.spec
SRPM URL: http://people.redhat.com/vondruch/rubygem-rails-deprecated_sanitizer-1.0.3-1.fc22.src.rpm
Description: 
In Rails 4.2 HTML sanitization has been rewritten using a more secure library.

This gem includes the old behavior shipping with Rails 4.2 and before. It is
strictly provided to ease migration. It will be supported until Rails 5.


Fedora Account System Username: vondruch

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=8675371

Comment 1 Josef Stribny 2015-01-22 13:48:30 UTC
The package builds and test suite passes although cannot be installed with older ActiveSupport. If the package works with the old ActiveSupport as well, perhaps we can adjust the dependency requirement.

Otherwise the spec looks alright => APPROVED.

Comment 2 Vít Ondruch 2015-01-22 14:13:17 UTC
(In reply to Josef Stribny from comment #1)
> The package builds and test suite passes although cannot be installed with
> older ActiveSupport. If the package works with the old ActiveSupport as
> well, perhaps we can adjust the dependency requirement.

Yes, you are right. I already noticed it and fixed locally.

 
> Otherwise the spec looks alright => APPROVED.

Thanks for the review.


New Package SCM Request
=======================
Package Name: rubygem-rails-deprecated_sanitizer
Short Description: Deprecated sanitizer API extracted from Action View
Upstream URL: https://github.com/rails/rails-deprecated_sanitizer
Owners: vondruch jstribny
Branches: 
InitialCC:

Comment 3 Gwyn Ciesla 2015-01-22 17:11:50 UTC
Git done (by process-git-requests).


Note You need to log in before you can comment on or make changes to this bug.