-
Notifications
You must be signed in to change notification settings - Fork 3
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
add support for the authenticator arg #9
Comments
+1 to authenticator argument, I can only use Okta SSO and not passwords |
Hi @dwiso and @FabrizioMFE , I'd be happy to work on this. However, I don't personally have access to a 3rd party authenticator, so I will need some help testing it. My current plan is to support all of the arguments listed here. Does that sound sufficient for your use case? |
@isaacwasserman Yes! At least for me that would be great! I'm happy to test, I'm trying to use the MCP server in Cursor (for some more context), Smithery has an easy to use command generator. |
@FabrizioMFE the latest commit (a64e093) should be able to accept the |
@isaacwasserman I can get it to work in a terminal with this command "uv --directory "C:/path/to/mcp-snowflake-server/src" run mcp_snowflake_server --account "PXA00000" --warehouse "EXAMPLE_WH" --user "[email protected]" --authenticator "externalbrowser" --role "USER_ROLE" --database "EXAMPLE_DB" --schema "EXAMPLE_SCHEMA"" but in Cursor it just says there's no tools available 🤔 when I try to add it, it like quickly flashes a terminal before closing it and I'm not sure where to find logs for this? Of note, when I run it in a terminal, it auto opens the default browser to the SSO login page, and won't move until I finish logging in. Not sure if Cursor maybe tries to kill the terminal too fast? not that it would be an issue with your code though if that were the case |
@FabrizioMFE It's possible that Cursor's terminal doesn't allow programs to open browser windows. Have you used any other servers where this type of login flow works? |
@isaacwasserman Sadly no sorry, I haven't ☹ this is the first MCP server I've felt I actually have a use for so I have not tested other MCP servers much less ones with this type of login flow Note that it's also completely feasible that my command is not correct. I think I got it right but like I mentioned, this is the first MCP server I've tried on Cursor so I don't have a ton of knowledge on it. |
@isaacwasserman I have cloned the repo, set up the local config, and gotten the server to initialize. When I open Claude, a browser window opens and I am able to properly authenticate. There are some error messages that pop up but I think they are related to handling all of the response. I have included part of the log file where I opened the app, got warning/errors and then closed the app. Despite the errors I am able to use the tools and Claude is able to respond to prompts with queries to the database. Let me know if there is anything else from the logs you could use or if more testing is needed. "snowflake_prod": {
"command": "/<local path>/uv",
"args": [
"--directory",
"/<path>/mcp-snowflake-server",
"run",
"mcp_snowflake_server",
"--account",
"<our account>",
"--warehouse",
"<our warehouse>",
"--user",
"<my user>",
"--authenticator",
"externalbrowser",
"--role",
"<target role>",
"--database",
"<our database>",
"--schema",
"<our schema>"
]
}
|
Thanks for creating this MCP.
It would be great it you could add support for the authenticator arg to be passed in the config without a password. My org does not allow password Auth.
The text was updated successfully, but these errors were encountered: