Skip to content

Clarify info in the box.session module #5065

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

Merged
merged 1 commit into from
Jul 7, 2025
Merged

Conversation

xuniq
Copy link
Contributor

@xuniq xuniq commented Apr 4, 2025

  • Update description in box.session.su(), box.session.user(), and box.session.euid()
  • Add box.session.effective_user() page

Fixes #5062

Deployment: https://docs.d.tarantool.io/en/doc/gh-5062-clarify-box.session.su/reference/reference_lua/box_session/

@xuniq xuniq force-pushed the gh-5062-clarify-box.session.su branch 2 times, most recently from 4b4c35f to 090b2da Compare June 25, 2025 16:01
@xuniq xuniq marked this pull request as ready for review June 25, 2025 16:01
@xuniq xuniq requested review from sergepetrenko and a1div0 June 25, 2025 16:08
@xuniq xuniq force-pushed the gh-5062-clarify-box.session.su branch from 090b2da to e0c99bc Compare June 25, 2025 16:12
@sergepetrenko sergepetrenko self-assigned this Jun 26, 2025
@sergepetrenko sergepetrenko assigned xuniq and unassigned sergepetrenko Jun 26, 2025
@xuniq xuniq force-pushed the gh-5062-clarify-box.session.su branch from e0c99bc to e3e8dfd Compare June 30, 2025 08:46
@xuniq xuniq requested a review from lenkis June 30, 2025 09:28
@xuniq xuniq force-pushed the gh-5062-clarify-box.session.su branch from e3e8dfd to 06bd508 Compare July 4, 2025 12:10
- Update description in `box.session.su()`, `box.session.user()`, `box.session.uid()`, and `box.session.euid()`

- Add `box.session.effective_user()` page

Fixes #5062
@xuniq xuniq force-pushed the gh-5062-clarify-box.session.su branch from 06bd508 to afca314 Compare July 7, 2025 09:56
Copy link
Contributor

@sergepetrenko sergepetrenko left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes!
LGTM.

@xuniq xuniq merged commit 5ccdb23 into latest Jul 7, 2025
1 check passed
@xuniq xuniq deleted the gh-5062-clarify-box.session.su branch July 7, 2025 11:35
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.

Clarify the box.session.su() description
3 participants