fix: display Snyk Code disabled in the tree view [IDE 1538]#339
Merged
nick-y-snyk merged 2 commits intofeat/IDE-1368_consume_org_from_serverfrom Nov 18, 2025
Merged
fix: display Snyk Code disabled in the tree view [IDE 1538]#339nick-y-snyk merged 2 commits intofeat/IDE-1368_consume_org_from_serverfrom
nick-y-snyk merged 2 commits intofeat/IDE-1368_consume_org_from_serverfrom
Conversation
|
|
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
e097d77 to
36a0a92
Compare
nick-y-snyk
commented
Nov 14, 2025
| String errorJson = ptn.getErrorMessage(); | ||
| if (errorJson != null && !errorJson.isBlank()) { | ||
| String error = ptn.getErrorMessage(); | ||
| if (error != null && !error.isBlank()) { |
Contributor
Author
There was a problem hiding this comment.
handle json CliError too
ae717bd to
66c42d5
Compare
70a7529
into
feat/IDE-1368_consume_org_from_server
6 checks passed
andrewrobinsonhodges-snyk
pushed a commit
that referenced
this pull request
Nov 21, 2025
* feat: add organization to project properties * fix: imports, pmd violations * fix: bump jackson deps * fix: bump core-text dep * fix: bump lang3 dep * fix: a few bugs with NPEs * fix: add checkbox * fix: add selection listeners to property page * fix: handle auto org correctly * fix: remove validation, so we can set an empty org in project settings and use global org * fix: remove unneeded org logic * fix: update the UI labels and make pref page behave even if ls has not started * fix: added a context menu for open project preferences * fix: clean up UI for auto org Added help text to Snyk Code Security checkbox: "Snyk Code scans must be enabled for the organization to run." Removed disableSnykCodeIfOrgDisabled() method and all calls to it Snyk Code checkbox is always selectable, regardless of SAST/org status Removed conditional logic in toggleIgnoresButtons() that hid filters based on CCI status Open Issues and Ignored Issues filters are always visible Updated tooltips in plugin.xml to mention: "Note: This filter will only take effect if Code Consistent Ignores (CCI) is enabled for the organization." * fix: remove LCE from sast settings * fix: remove (CCI) from tooltips * fix: fix linter on generic exception * fix: remove tooltip from checkbox * fix: add back the org enabled for code * fix: fixed duplicate of code must be enabled label. * fix: disable auto org by default * fix: display Snyk Code disabled in the tree view [IDE 1538] (#339) fix: error html view --------- Co-authored-by: Knut Funkel <knut.funkel@snyk.io> Co-authored-by: nick-y-snyk <nikita.yasnohorodskyi@snyk.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Removed json handling logic, because errorMessage is always a simple string, there is a cliError field on ScanParams, but it's not present in Eclipse. So looks like error handling has been broken for a while
Checklist
Screenshots / GIFs
Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!