java-sproc-wrapper icon indicating copy to clipboard operation
java-sproc-wrapper copied to clipboard

update Readme

Open mesut opened this issue 4 years ago • 4 comments

@vadeg We should inform users that they need to move their custom objectTransformer to the new default package, or they need to provide scanning namespace instead of supporting the old namespace ("de.zalando") ( https://github.com/zalando-stups/java-sproc-wrapper/pull/68)

It is solution for https://github.com/zalando-stups/java-sproc-wrapper/issues/67

mesut avatar Apr 05 '21 14:04 mesut

@mesut why did you close #68 ?

vadeg avatar Apr 13 '21 21:04 vadeg

@vadeg It doesn't make sense to support old namespace for new version of the library since the library provides to the clients an alternative path to scan value-transformer via "global.value.transformer.search.namespace" system env key. So instead of making changing "global.value.transformer.search.namespace" to the array, it would be better to keep it as it is and inform the client that they need to change directory of their value-transformer to "de.zalando". Or if they don't want to change it they can provide their existing directory via "global.value.transformer.search.namespace". So that's why I closed https://github.com/zalando-stups/java-sproc-wrapper/pull/68 and created this one for updating README.md.

mesut avatar Apr 24 '21 20:04 mesut

@mesut readme update would be helpful, bu I think support of multiple packages also make sense. Library provide a UUIDValueTransformer global value transformers out of the box. If someone wants to use that, then it enforces to keep all transformers in the org.zalando package what may not match with packages in other projects.

vadeg avatar May 21 '21 19:05 vadeg

@vadeg your point is valid

mesut avatar May 24 '21 11:05 mesut

Vulnerabilities found. Need to fix them first

One or more dependencies were identified with vulnerabilities: Error:
Error: postgresql-42.2.14.jar: CVE-2022-21724, CVE-2022-31197, CVE-2022-26520 Error: spring-core-5.2.16.RELEASE.jar: CVE-2021-22060, CVE-2021-22096, CVE-2022-22968, CVE-2022-22965, CVE-2022-22971, CVE-2022-22950, CVE-2016-1000027, CVE-2022-22970

vadeg avatar Oct 10 '22 04:10 vadeg