You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.
As a project aiming to be used in Bitcoin wallets, quality of code and accountability are of high importance. Projects integrating with fio will ask if the source code is secure and audited. While knowing the origin of source code cannot replace audits, it does help estimate who else has audited the code or if issues are found, it is important to also estimate the broader implications.
This date library looks a lot like coming from this date library but I can't find any documentation of which revision it actually is. It was added July 2019 but even diffs with revisions from back then are massive.
Fork date and maintain a stripped-down version which gets added as submodule.
Document well which revision of date is being used.
If no changes are required, I would opt for the first option. Else for the second. Only those two options allow a relatively easy merging of upstream fixes and improvements.
The initial version of the kotlin SDK was a port of the EOS Java SDK from java to kotlin. The kotlin code was
then modified to specifically support the FIO protocol. Included in the port was the The Android Serialization Provider, ABI Provider
and the SoftKey Signature Provider.
The C++ code was also brought over from the EOS Java SDK. The only change to the C++ code was the Class Loader path
information.
The current relevant EOS SDK code base can be found at:
That's how far I got, too. It would really make things easier if you would know the revision you forked from. Then auditors could look into the EOS SDK from there, check what critical fixes came later and check if those were ported over to FIO and also check the diff with that revision to see what FIO added. I don't know EOS but I know EOS has way more eyes on it than FIO, so I would consider their code as more trustworthy than FIO code and I would concentrate on what FIO added.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
As a project aiming to be used in Bitcoin wallets, quality of code and accountability are of high importance. Projects integrating with fio will ask if the source code is secure and audited. While knowing the origin of source code cannot replace audits, it does help estimate who else has audited the code or if issues are found, it is important to also estimate the broader implications.
This date library looks a lot like coming from this date library but I can't find any documentation of which revision it actually is. It was added July 2019 but even diffs with revisions from back then are massive.
I see several options here:
If no changes are required, I would opt for the first option. Else for the second. Only those two options allow a relatively easy merging of upstream fixes and improvements.
Edit: This looks similar like this and this looks like this.
So the
date
and therapidjson
classes live in their own folders inside theabieos
folder which also should be cleaned up.The text was updated successfully, but these errors were encountered: