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

Added string replace line to fix implementation error. #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Daktic
Copy link

@Daktic Daktic commented May 22, 2023

I was having an issue where Snowflake was not accepting the JWT because it expects the fingerprint to be formatted with "/" instead of "_".

for example => "p0T8x62Z4CX2_sQPeLX7brvOM6sK2j5ICVq_0vmQJY0o=" should be "p0T8x62Z4CX2/sQPeLX7brvOM6sK2j5ICVq/0vmQJY0o="

@Ripper53 Ripper53 self-requested a review May 22, 2023 21:28
@Ripper53
Copy link
Owner

So, I'm not too familiar with key generation, but do you know why your key contains underscores? Mine has slashes, and I can't find any information on why a key would have underscores instead. From the little research I did, I found this table which contains the characters that a key is made up of and I don't see an underscore. So, I'm wondering, would it be better to change the key itself instead of the code doing it for you?

@Daktic
Copy link
Author

Daktic commented Jun 7, 2023

This has been my first foray into key generation. When I wasn't able to connect to snowflake I tried creating my own JWT and failed at that so I just resorted to trying to debug where in your code it went wrong. That's where I discovered the fingerprint was "correct" but it had underscores instead of slashes.

If I find some time I will try running it from a different architecture and see if I have the same issue.

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.

2 participants