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

ZE-FTP: Avoid throwing empty error when we know the reason of the failure #2143

Closed
Tracked by #1982
zFernand0 opened this issue Feb 21, 2023 · 2 comments
Closed
Tracked by #1982
Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have Technical Debt Includes Architecture, Code, Testing, Automation debt

Comments

@zFernand0
Copy link
Member

There are multiple instances where we throw empty error in the FTP extension.
Since we know what happened, it'd be good to throw the error in case other APIs are expecting more information.
Example:
https://github.com/zowe/vscode-extension-for-zowe/blob/4aa6d95ec6d86a9cd818a6fb19b6c4d8e172de79/packages/zowe-explorer-ftp-extension/src/ZoweExplorerFtpUssApi.ts#L76

@zFernand0 zFernand0 added enhancement New feature or request Technical Debt Includes Architecture, Code, Testing, Automation debt priority-low Legit issue but cosmetic or nice-to-have labels Feb 21, 2023
@github-actions
Copy link

Thank you for raising this issue.
The community has 90 days to upvote 👍 the issue.
If it receives 10 upvotes, we will move it to our backlog. If not, we will close it.

@JillieBeanSim JillieBeanSim mentioned this issue Sep 25, 2023
31 tasks
@JillieBeanSim
Copy link
Contributor

fixed with PR #2321

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have Technical Debt Includes Architecture, Code, Testing, Automation debt
Projects
None yet
Development

No branches or pull requests

2 participants