Skip to content

Basic lambda implementation#15

Merged
rampage644 merged 8 commits into
mainfrom
rampage644/lambda
Nov 17, 2025
Merged

Basic lambda implementation#15
rampage644 merged 8 commits into
mainfrom
rampage644/lambda

Conversation

@rampage644
Copy link
Copy Markdown
Contributor

Currently have auth disabled due to inability to store token in between requests.

Copy link
Copy Markdown
Contributor

@DanCodedThis DanCodedThis left a comment

Choose a reason for hiding this comment

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

@rampage644 Otherwise LGTM, is there anything that we have a problem here with? Other than the session token

Comment thread crates/embucket-lambda/src/main.rs
@DanCodedThis
Copy link
Copy Markdown
Contributor

@rampage644 good news we can use jwt (stateless token) instead of uuidv4 with the connector

@DanCodedThis
Copy link
Copy Markdown
Contributor

DanCodedThis commented Nov 15, 2025

@rampage644

  • Good news I added jwt token sessions.
  • Bad news is, that there is no way to prolong the jwt token, once it's reached the snowflake client, without logining again. No matter if it's active or inactive, it will become expired.
  • So I just set it for 24 hours to solve it temporarily. Dbt should also login with every query.
  • We can't just send a set_auth header, because snowflake connector is not a browser (though it's worth checking).
  • There may be a session's enpoint, like keep alive or something. I think Denys and I saw it in the snowflake connector implementation. Still, we don't support it now, even if it exists and works the way we need it.
  • And yeah there is some merge conflicts :(

Currently have auth disabled due to inability to store token in between requests.
@rampage644
Copy link
Copy Markdown
Contributor Author

Rebased on main

@DanCodedThis
Copy link
Copy Markdown
Contributor

Compile errors on tests, clippy probably the same

@rampage644 rampage644 merged commit 8331b64 into main Nov 17, 2025
3 checks passed
@rampage644 rampage644 deleted the rampage644/lambda branch November 17, 2025 13:52
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