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

customttl #503

Closed
wants to merge 2 commits into from
Closed

customttl #503

wants to merge 2 commits into from

Conversation

mwilde345
Copy link
Member

@mwilde345 mwilde345 commented Dec 10, 2024

Ticket(s): FE-6221

Problem

As part of exploring LSE-205, it looks like short lived keys are tripping up core router.

Solution

shell lends itself to easy repro of short term creds bugs if we just let an env var decide the ttl

added a soft restriction to no more than 15 minutes. but.... it's open source

Result

Can repro the lse bug:

export FAUNA_KEY_TTL_SECONDS=5
node ./src/user-entrypoint.mjs query "1 + 1" -d us/test --verbosity 5
...
(blast that over and over)

You'll get JSON unexpected end of Input which indicates the issue.

@mwilde345 mwilde345 changed the base branch from main to v3 December 10, 2024 22:37
@mwilde345 mwilde345 requested a review from a team as a code owner December 10, 2024 22:37
@ecooper ecooper closed this Dec 12, 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.

2 participants