Skip to content

refactor: Use logger in Redis Session Handler #8568

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

refactor: Use logger in Redis Session Handler #8568

wants to merge 1 commit into from

Conversation

najdanovicivan
Copy link
Contributor

Description
log_message was left in some places this changes those instances to use logger object instead.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@kenjis kenjis changed the title fix: Use logger in Redis Session Handler refactor: Use logger in Redis Session Handler Feb 22, 2024
@kenjis kenjis added the refactor Pull requests that refactor code label Feb 22, 2024
@github-actions github-actions bot added the stale Pull requests with conflicts label Feb 23, 2024
@github-actions GitHub Actions
Copy link

👋 Hi, @najdanovicivan!

We detected conflicts in your PR against the base branch 🙊
You may want to sync 🔄 your branch with upstream!

Ref: Syncing Your Branch

@kenjis
Copy link
Member

kenjis commented Feb 23, 2024

Closed by #8323

@kenjis kenjis closed this Feb 23, 2024
@najdanovicivan najdanovicivan deleted the redis-session-logger branch March 7, 2024 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kenjis kenjis kenjis approved these changes

Assignees
No one assigned
Labels
refactor Pull requests that refactor code stale Pull requests with conflicts
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@najdanovicivan @kenjis