Skip to content

Credentials - Consider stripping leading/trailing spaces in credentials before storing and using the keys . #259

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.

Sign up for GitHub

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

Jump to bottom
Closed
sangee2004 opened this issue Apr 17, 2024 · 1 comment
Closed

Credentials - Consider stripping leading/trailing spaces in credentials before storing and using the keys . #259

sangee2004 opened this issue Apr 17, 2024 · 1 comment
Assignees
@drpebcak
Labels
enhancement New feature or request

Comments

@sangee2004
Copy link
Contributor

It would be good if we can strip leading/trailing spaces in credentials before storing and using the keys.

It is hard to figure out that the reason for invalid credentials error when it is relating to the presence of leading/trailing empty spaces.
@sangee2004 sangee2004 added the enhancement New feature or request label Apr 17, 2024
@g-linville g-linville assigned g-linville and unassigned g-linville Jul 1, 2024
@drpebcak drpebcak self-assigned this Jul 16, 2024
@drpebcak drpebcak mentioned this issue Jul 16, 2024
Merged
@sangee2004
Copy link
Contributor Author

Tested with gptscript version v0.0.0-dev-a5a0538d-dirty

leading/trailing spaces in the entered credentials gets stripped before being saved in credentials.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@drpebcak drpebcak

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@drpebcak @g-linville @sangee2004