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

Raise statements in session.py do not specify an exception type derived from BaseException #213

Closed
adam-wolfe opened this issue Sep 8, 2023 · 1 comment · Fixed by #217
Labels
bug Something isn't working good first issue Good for newcomers priority-low severity-low

Comments

@adam-wolfe
Copy link

Describe the bug

If a user fails to specify a host, user && password, tokenType && tokenValue, or tokenValue in their connection object when creating a session, the user will be presented with:

Exception has occurred: TypeError
exceptions must derive from BaseException

From src/core/zowe/core_for_zowe_sdk/session.py, the lines to be revised are (line numbers may change):
Line 48: raise "Host must be supplied"
Line 64: raise "An authentication method must be supplied"

Describe your environment

Additional context

@adam-wolfe adam-wolfe added the bug Something isn't working label Sep 8, 2023
@github-actions
Copy link

github-actions bot commented Sep 8, 2023

Thank you for creating a bug report.
We will investigate the bug and evaluate its impact on the product.
If you haven't already, please ensure you have provided steps to reproduce the bug and as much context as possible.

@t1m0thyj t1m0thyj added the good first issue Good for newcomers label Sep 15, 2023
@t1m0thyj t1m0thyj linked a pull request Oct 2, 2023 that will close this issue
4 tasks
@t1m0thyj t1m0thyj closed this as completed Oct 2, 2023
@t1m0thyj t1m0thyj moved this to Closed in Zowe CLI Squad Dec 26, 2023
@t1m0thyj t1m0thyj removed this from Zowe CLI Squad Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers priority-low severity-low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants