Problem/Motivation
https://www.drupal.org/project/vendor_stream_wrapper/releases/2.0.0 is out, it would be good to have a new release using that.
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|
Issue fork jquery_colorpicker-3253237
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
davemybes commentedWell, https://www.drupal.org/project/vendor_stream_wrapper is now no longer supported due to a security issue, so this could get tricky. There is no movement on a new maintainer for the module yet either.
Comment #3
mondrakePretty weird, from a new release two months ago to completely abadoned? Anyone knows what's going on? I filed #3262541: Access bypass: view any file in vendor directory.
Comment #5
ashetkar commentedHere's patch which removes vendor stream wrapper dependency from Jquery Colorpicker module. I have changed status to need review.
Comment #6
ashetkar commentedUploaded new patch after removing whitespace and trailing whitespace
Comment #7
mondrake@ashetkar thank you, but your patch is removing the dependency, which is not the scope of this issue. I see what you are trying to do here, it has its own merit, but I'd suggest you open a separate issue to pursue that.
Here we'd bump the dependency if/when the security issue upstream is solved.
Comment #8
ashetkar commentedHi @mondrake,
I have opened new issue to remove vendor stream wrapper dependency due to security issues.
https://www.drupal.org/project/jquery_colorpicker/issues/3284430
Comment #9
zann1e commentedA few days ago https://www.drupal.org/project/vendor_stream_wrapper was released with the new security update 2.0.1.
I think it is time to update it for "jquery_colorpicker".
Comment #10
mondrakeActually, this is now critical since ^1 is no longer supported for a security flaw. ^2.0.1 is now the minimum secure release.
Comment #11
mondrakeComment #12
podarokComment #13
podarokThanks all