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

expire-at param for a session token doesn't have any effect #2539

Closed
evgeniiz321 opened this issue Sep 1, 2023 · 0 comments · Fixed by #2546
Closed

expire-at param for a session token doesn't have any effect #2539

evgeniiz321 opened this issue Sep 1, 2023 · 0 comments · Fixed by #2546
Assignees
Labels
bug Something isn't working

Comments

@evgeniiz321
Copy link

Relates to nspcc-dev/neofs-testcases#621

Expected Behavior

Create session token with expire_at param=current_epoch + 1 and expect it to expire after 2 epoch ticks

Current Behavior

No expiration at all, tried different param values. Creating session token with 'lifetime' param works ok.

Steps to Reproduce (for bugs)

Allure - test_session_token_expiration_flags
allure.tar.gz

Regression

(No)

Your Environment

Run on current latest master of neofs-node - 9871712

@roman-khimov roman-khimov added bug Something isn't working and removed triage labels Sep 1, 2023
AliceInHunterland added a commit that referenced this issue Sep 5, 2023
Added a check for changes to the lifetime flag when it's passed by the
user to prevent the settlement of the default lifetime value even if
expire-at flag passed.

Closes #2539.

Signed-off-by: Ekaterina Pavlova <[email protected]>
AliceInHunterland added a commit that referenced this issue Sep 5, 2023
Added a check for changes to the lifetime flag when it's passed by the
user to prevent the settlement of the default lifetime value even if
expire-at flag passed.

Closes #2539.

Signed-off-by: Ekaterina Pavlova <[email protected]>
AliceInHunterland added a commit that referenced this issue Sep 5, 2023
Added a check for changes to the lifetime flag when it's passed by the
user to prevent the settlement of the default lifetime value even if
expire-at flag passed.

Closes #2539.

Signed-off-by: Ekaterina Pavlova <[email protected]>
AliceInHunterland added a commit that referenced this issue Sep 5, 2023
Added a check for changes to the lifetime flag when it's passed by the
user to prevent the settlement of the default lifetime value even if
expire-at flag passed.

Closes #2539.

Signed-off-by: Ekaterina Pavlova <[email protected]>
AliceInHunterland added a commit that referenced this issue Sep 5, 2023
Added a check for changes to the lifetime flag when it's passed by the
user to prevent the settlement of the default lifetime value even if
expire-at flag passed.

Closes #2539.

Signed-off-by: Ekaterina Pavlova <[email protected]>
AliceInHunterland added a commit that referenced this issue Sep 5, 2023
Added a check for changes to the lifetime flag when it's passed by the
user to prevent the settlement of the default lifetime value even if
expire-at flag passed.

Closes #2539.

Signed-off-by: Ekaterina Pavlova <[email protected]>
roman-khimov added a commit that referenced this issue Sep 6, 2023
Added check of passing `--lifetime` flag by the user and handling of
setting `defaultLifetime` value

Closes #2539.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants