Skip to content

Add password authentication request validation#323

Merged
antoniomika merged 1 commit intoantoniomika:mainfrom
EpicEric:main
Sep 17, 2024
Merged

Add password authentication request validation#323
antoniomika merged 1 commit intoantoniomika:mainfrom
EpicEric:main

Conversation

@EpicEric
Copy link
Copy Markdown
Contributor

This adds a password-based authentication mechanism under the option --authentication-password-request-url, which functions similarly to --authentication-key-request-url, but takes a password instead of a public key.

Closes #322

This adds a password-based authentication mechanism under the option
`--authentication-password-request-url`, which functions similarly
to `--authentication-key-request-url`, but takes a password
instead of a public key.

Closes antoniomika#322
@EpicEric
Copy link
Copy Markdown
Contributor Author

This seems to work on my sish instance, with a simple verification server that checks for a single password and returns 200. I've used a command like this one for testing (which then prompts me for the password):

ssh -o PubkeyAuthentication=no -o PreferredAuthentications=password -R test:80:localhost:8080 tuns.sh

I haven't tested the timeout, however since this is basically a copy-pasted version of --authentication-key-request-url, it should work the same.

@antoniomika
Copy link
Copy Markdown
Owner

Thanks for the awesome contribution @EpicEric!

Copy link
Copy Markdown
Owner

@antoniomika antoniomika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm 👍

@antoniomika antoniomika merged commit 23ccc59 into antoniomika:main Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature-request] Password authentication with URL validation

2 participants