-
Notifications
You must be signed in to change notification settings - Fork 536
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
Option for per request authentication #51
Comments
+1 - same use case, same issue. |
I love it. It’s in the queue. |
I did a small workaround for that. However, you have to patch few files... |
@vbeskrovny - Would you be able to share your workaround? |
Sure, the link is under my profile or here: https://github.com/vbeskrovny/WebSSH2-session |
@vbeskrovny I'll take a look at this when I have a moment and try to integrate it if that's ok. |
Cool, thanks. By the way I wanted to add custom calls to the onDisconnect events, to add some nice prompts and fancy UI, but the "bundle.js" has no source... :( |
The provided patch, will add a /logout URL, which will re-request the authentication and then redirect you back to the referrer (e.g like http://localhost:2222/ssh/host/myhost.com). It also adds a menu entry "Switch User" that opens /logout and forces re-authentication. |
Thanks @vvalchev, I will look at this shortly! |
@richard-scott and @AODtorusan I've merged this into |
fixes #75 and potentially #51 thanks to both @vbeskrovny and @vvalchev
fixes billchurch#75 and potentially billchurch#51 thanks to both @vbeskrovny and @vvalchev
fixes billchurch#75 and potentially billchurch#51 thanks to both @vbeskrovny and @vvalchev
Would it be possible to add an option to re-request the authentication each time you do a new request for an SSH session ?
We're using webssh on a jumphost server, but most target devices use different credentials. This means that almost every time we need to flush the browser authentication cache when we want to connect to a device, which is cumbersome.
The text was updated successfully, but these errors were encountered: