-
Notifications
You must be signed in to change notification settings - Fork 2
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
CryptoSwift dependency license need clarification #29
Comments
License: https://github.com/krzyzanowskim/CryptoSwift?tab=License-1-ov-file#readme License Text
|
The license appears to be a modification of the Zlib license, which takes the suggested but not required attribution requirements of Zlib and makes them mandatory and additionally adds various requirements concerning attribution in both source and binary form. We need to carefully review the strict attribution requirements and should also have a plan for how attribution and the other statements the license requires will be maintained in source and binary form. It might be worth considering alternatives that do not contain such strict attribution requirements. |
@hendrikebbers So there are other Crypto libraries for Swift that have less license restrictions but none that offer the wide range of functionality that CryptoSwift does. Would it be appropriate to add a |
@RickyLB "none that offer the wide range of functionality" does that mean that we need to replace it with like 2 libs? Because I assume the license will be a problem when moving it to Linux Foundation. |
@hendrikebbers Correct— this will require an overhaul of the key implementations in the SDK. In the meantime, i'll start looking into other options. When would this change need to take effect? |
In January as far as I know |
As written in #3 we have the direct dependency https://github.com/krzyzanowskim/[email protected] that has a license that needs clarification.
The text was updated successfully, but these errors were encountered: