You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@curator.apache.org by Max-Pudov <gi...@git.apache.org> on 2018/10/09 12:12:58 UTC

[GitHub] curator pull request #280: CURATOR-481 Remove jackson-mapper-asl-version and...

GitHub user Max-Pudov opened a pull request:

    https://github.com/apache/curator/pull/280

    CURATOR-481 Remove jackson-mapper-asl-version and update jackson

    …rsion of jackson

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Max-Pudov/curator CURATOR-481

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/curator/pull/280.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #280
    
----
commit 20cda944243f834a216316ebd8d3c59758898282
Author: Max-Pudov <pu...@...>
Date:   2018-10-09T12:11:58Z

    CURATOR-481 Remove jackson-mapper-asl-version and update to latest version of jackson

----


---

[GitHub] curator issue #280: CURATOR-481 Remove jackson-mapper-asl-version and update...

Posted by Randgalt <gi...@git.apache.org>.

Github user Randgalt commented on the issue:

    https://github.com/apache/curator/pull/280
  
    Are there any implications for backward compatibility?


---

[GitHub] curator issue #280: CURATOR-481 Remove jackson-mapper-asl-version and update...

Posted by cammckenzie <gi...@git.apache.org>.

Github user cammckenzie commented on the issue:

    https://github.com/apache/curator/pull/280
  
    @Max-Pudov @matobet 
    Guys, any thoughts about backward compatibility issues?


---

[GitHub] curator issue #280: CURATOR-481 Remove jackson-mapper-asl-version and update...

Posted by leventov <gi...@git.apache.org>.

Github user leventov commented on the issue:

    https://github.com/apache/curator/pull/280
  
    @Randgalt @cammckenzie I believe such update is safe. See #301.


---

[GitHub] curator issue #280: CURATOR-481 Remove jackson-mapper-asl-version and update...

Posted by matobet <gi...@git.apache.org>.

Github user matobet commented on the issue:

    https://github.com/apache/curator/pull/280
  
    Hi, thanks @Max-Pudov for contributing this! The security scans can get pretty annoying continuously reporting Curator for the usage of `jackson-mapper-asl`. Is there anything preventing this from getting merged?


---

[GitHub] curator issue #280: CURATOR-481 Remove jackson-mapper-asl-version and update...

Posted by cammckenzie <gi...@git.apache.org>.

Github user cammckenzie commented on the issue:

    https://github.com/apache/curator/pull/280
  
    It looks OK to me, but my knowledge of this section of Curator is very limited. @Randgalt do you have any thoughts on this? If not, I'll merge.


---