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

SALClient can't connect to new server #67

Open
TOFarmer opened this issue Feb 22, 2021 · 1 comment
Open

SALClient can't connect to new server #67

TOFarmer opened this issue Feb 22, 2021 · 1 comment
Labels
Bug Issues that report incorrect or undesirable behaviour(s) Priority: 1 EVENTUAL: A low priority issue that is unlikely to be addressed until the priority increases.

Comments

@TOFarmer
Copy link
Contributor

Summary

If a SALClient has previously connected to server (i.e. during initialization), connecting to a new server is not possible if the original server required authentication.

Steps to reproduce

If host requires authentication:

sc = SALClient(host)
sc.host = new_host

What is the current bug behavior?

SALClient prompts for username and password (irrespective of whether new_host requires authentication.

What is the expected correct behavior?

No prompt, SALClient should simply connect to new host.

Possible fixes

This occurs because after a host has been set during initialization, SALClient.auth_required has a value determined by this host. When SALClient.host is called a GET request is made (to the new host), and before it is sent SALClient requests credentials because it thinks it has to authenticate (which is true of GET requests made to the original host but not the new one). This can be fixed by simply setting auth_required = False when host is set. If this is done, auth_required can be removed from __init__.

@TOFarmer TOFarmer added Bug Issues that report incorrect or undesirable behaviour(s) Priority: Triage TRIAGE: Yet to be prioritized. Priority: 1 EVENTUAL: A low priority issue that is unlikely to be addressed until the priority increases. and removed Priority: Triage TRIAGE: Yet to be prioritized. labels Feb 22, 2021
@TOFarmer
Copy link
Contributor Author

Minimal priority as this is unlikely to be commonly experienced by users, who will probably just connect to a single SAL host from a SALClient.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues that report incorrect or undesirable behaviour(s) Priority: 1 EVENTUAL: A low priority issue that is unlikely to be addressed until the priority increases.
Projects
None yet
Development

No branches or pull requests

1 participant