Skip to content
This repository has been archived by the owner on Apr 27, 2021. It is now read-only.

SonarTS for SQ fails with Node 6 #432

Closed
nielsnoorlander opened this issue Jan 10, 2018 · 3 comments
Closed

SonarTS for SQ fails with Node 6 #432

nielsnoorlander opened this issue Jan 10, 2018 · 3 comments
Assignees
Milestone

Comments

@nielsnoorlander
Copy link

I want to report a bug.
The current prerequisite of the NodeJS version for SonarTS 1.4 seems no longer valid.

SonarTS version: 1.4

Node.js version: 6.12.3

SonarQube version: 6.7.1

Reproducer
Reproducable with the SonarTS example projects
Logs

ERROR: /projectpath/.sonar/sonarts-bundle/node_modules/tslint-sonarts/lib/runner/sonarUtils.js:17
ERROR:         Object.entries(element).forEach(([key, value]) => {
ERROR:                ^
ERROR: 
ERROR: TypeError: Object.entries is not a function
ERROR:     at data.forEach (/projectpath/.sonar/sonarts-bundle/node_modules/tslint-sonarts/lib/runner/sonarUtils.js:17:16)
ERROR:     at Array.forEach (native)
ERROR:     at Object.stringifyToStream (/projectpath/.sonar/sonarts-bundle/node_modules/tslint-sonarts/lib/runner/sonarUtils.js:14:10)
ERROR:     at Socket.process.stdin.on (/projectpath/.sonar/sonarts-bundle/node_modules/tslint-sonarts/lib/runner/tsrunner.js:33:18)
ERROR:     at emitNone (events.js:91:20)
ERROR:     at Socket.emit (events.js:185:7)
ERROR:     at endReadableNT (_stream_readable.js:974:12)
ERROR:     at _combinedTickCallback (internal/process/next_tick.js:80:11)
ERROR:     at process._tickCallback (internal/process/next_tick.js:104:9)

I want to provide the following feedback.
Upgrading to the latest NodeJS 8.x version solved the problem.

@ncooper09
Copy link

I can confirm this bug, though it's really an error in the documentation.

We saw the actual bug when we were on Node 6.(something.something). We switched to Node 8.9.4, and the issue was resolved.

We are on SonarTS 1.4 and SonarQube 6.7.1.

@vilchik-elena vilchik-elena added this to the 1.4.1 milestone Jan 11, 2018
@vilchik-elena
Copy link
Contributor

Thanks for reporting! I've added it to the bug fix milestone
Feel free to provide a PR

@vilchik-elena vilchik-elena self-assigned this Jan 12, 2018
@ghost ghost removed the in progress label Jan 12, 2018
@vilchik-elena vilchik-elena changed the title Prerequisite for NodeJS invalid SonarTS for SQ fails with Node 6 Jan 12, 2018
@vilchik-elena
Copy link
Contributor

Bug fix release is done https://github.com/SonarSource/SonarTS/releases/tag/1.4.1.1991. Should be available in update center shortly

@vilchik-elena vilchik-elena modified the milestones: 1.4.1, 1.5 Jan 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants