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

Issue with winston dependency breaking builds relying on cloud-sdk/util #206

Closed
edonadei opened this issue Jun 22, 2020 · 4 comments · Fixed by #205
Closed

Issue with winston dependency breaking builds relying on cloud-sdk/util #206

edonadei opened this issue Jun 22, 2020 · 4 comments · Fixed by #205
Labels
bug Something isn't working

Comments

@edonadei
Copy link

Describe the bug
Winston dependencies break multiples jenkins builds since the update 3.3.0/3.3.1 to winstonjs/winston#1814

Expected behavior
We should use the cloud-sdk dependency without errors from this dependency
It is actually "winston": "^3.2.1" but it should be "winston": "3.2.1" or "winston": "~3.2.1"

Used Versions:

  • @sap/cloud-sdk-util@1.17.2 but it happens on other versions aswell
@edonadei edonadei added the bug Something isn't working label Jun 22, 2020
@edonadei edonadei changed the title Issue with winston dependency Issue with winston dependency breaking builds relying on cloud-sdk/util Jun 22, 2020
@marikaner marikaner mentioned this issue Jun 22, 2020
5 tasks
@marikaner
Copy link
Contributor

marikaner commented Jun 22, 2020

Hey @edonadei, I locked the version to 3.3.0. We will probably release a new version tomorrow or Wednesday. You will have to upgrade to the latest version of the SDK though, which will then be 1.22.0. As you are using a very old version, consider the migration guide.

@HoffmannThomas
Copy link
Member

There is a fixed version available now: winstonjs/winston#1814 (comment)

@marikaner
Copy link
Contributor

@marikaner I think 3.3.0 has the same issue. The last one that worked was 3.2.1.

I checked the code, the reference to git was introduced in v3.3.1 and fixed in v3.3.2, so theoretically your builds should work without any action now. If there is still something broken, maybe it is related to something else?

@HoffmannThomas
Copy link
Member

@marikaner I think 3.3.0 has the same issue. The last one that worked was 3.2.1.

I checked the code, the reference to git was introduced in v3.3.1 and fixed in v3.3.2, so theoretically your builds should work without any action now. If there is still something broken, maybe it is related to something else?

Yeah sorry that was a misfire on my part. It worked with npm install + resolution, but our preinstall resolution script was ignored during an explicit npm install of the module inside of our pipeline.

So yes, only 3.3.1 was affected and our builds are running fine after the release of 3.3.2 without any action on our side (since the sdk defined "winston@^3.2.1").

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants