Skip to content
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

Outdated SDK package dependencies #126

Open
3 tasks
valentinyanakiev opened this issue Sep 2, 2021 · 2 comments
Open
3 tasks

Outdated SDK package dependencies #126

valentinyanakiev opened this issue Sep 2, 2021 · 2 comments

Comments

@valentinyanakiev
Copy link

Description

Jolocom SDK has a lot of outdated dependencies - and this creates dependency hell for 3rd party integration.

E.g. We, the Alkemio team, rely server-side on nodeJS with mysql as the primary storage driver, used with typeORM as ORM layer and nestJS as a framework. We recently moved to the latest version of nestJS and moved to typeORM 0.2.35 (there is already 0.2.37 out).

Jolocom SDK, on the other hand, relies on a 2-year old typeORM release - 0.2.24, which is already incompatible with our code. It also relies on jolocom/sdk-storage-typeorm that has the same typeORM release as a peer dependency (0.2.24).

As a direct result of this outdated package dependencies we have isolated the jolocom integration for now in our codebase. We'd like to soon re-integrate it and start using it.

TODO

  • Update jolocom/sdk-storage-typeorm peer dependency of typeORM to 0.2.35+
  • Update jolocom-sdk with updated jolocom/sdk-storage-typeorm
  • Update jolocom-sdk typeorm dependency to typeORM 0.2.35+

P.S. It is also worth noting that other dependencies are also quite outdated, e.g. you are a major vershion behind (3 vs 4) on typescript etc. While that is not a direct blocker for us it might be good to update it (and analyze + update the rest of the packages where possible) to streamline external contribution and prevent others from having similar dependency issues as we have.

@mnzaki
Copy link
Contributor

mnzaki commented Sep 2, 2021

Thank you @valentinyanakiev for the report!
We are busy updating our entire stack and preparing for a version 2 of the SDK. All these issues will be handled by then.

Of course we are open to PRs if you have already worked on this yourself! Just be sure to sign the CLA (follow the steps in the CONTRIBUTING document)

@techsmyth
Copy link

Hey @mnzaki what is the timeline for v2?

At the moment Jolocom SDK integration is completely removed due to the version compatibility issues above. And given we likely will need to work on our SSI integration this month as part of our recent eSSIF lab grant we need to decide how to proceed.

I see a few options:

  1. we fork, try to update the dependencies ourselves and if we succeed we submit a PR. Plus is that it is in our own hands. Minus is that we are not deeply familiar with the SDK dependencies, so outcome and effort is hard to predict.
  2. we wait for v2. Plus: latest version. Minus: need to know the timeline / when it would be usable by us.
  3. we ask you very nicely to fix this. Plus: most effective use I think of everyone's time. Minus: I know you guys are swamped.

Clearly we hope for #3. #2 depends on your answer re timeline. #1 is uncertain re outcome / effort.

This is more critical than the docker image base issue.

Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants