Skip to content

fix: Hot reloading when session is enabled - #8112

Merged
kenjis merged 3 commits into
codeigniter4:developfrom
michalsn:fix/hot-reloader
Oct 30, 2023
Merged

kenjis merged 3 commits into
codeigniter4:developfrom
michalsn:fix/hot-reloader

Conversation

@michalsn

@michalsn michalsn commented Oct 28, 2023

Copy link
Copy Markdown
Member

Description
When we use the session globally, it can block other requests as they wait for the session to be available for writing. The hot reloader runs in the background as a "long-running process".

This PR may potentially fix: 8093

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

@michalsn michalsn added the bug Verified issues on the current code behavior or pull requests that will fix them label Oct 28, 2023
@kenjis

kenjis commented Oct 28, 2023

Copy link
Copy Markdown
Member

Hot reloading does not work.
Screenshot 2023-10-28 22 28 12

@michalsn

Copy link
Copy Markdown
Member Author

Thank you. I totally forgot about the indexPage. Fixed.

@kenjis kenjis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The code looks good.

@neznaika0

Copy link
Copy Markdown
Contributor

@michalsn I applied the patch, but the problem remained. The page cannot be updated

$ patch -p1 < ~/Downloads/patch-diff.githubusercontent.com_raw_codeigniter4_CodeIgniter4_pull_8112.patch 
patching file system/Debug/Toolbar/Views/toolbar.js
patching file system/HotReloader/HotReloader.php
patching file system/Debug/Toolbar/Views/toolbar.js
patching file system/Debug/Toolbar/Views/toolbar.tpl.php

@michalsn

Copy link
Copy Markdown
Member Author

@neznaika0 Thank you for testing this out... unfortunately I have no other ideas for now.

Although this PR fixes a valid bug.

@kenjis
kenjis merged commit 0872a21 into codeigniter4:develop Oct 30, 2023
@kenjis

kenjis commented Oct 30, 2023

Copy link
Copy Markdown
Member

@michalsn Thank you!

@michalsn
michalsn deleted the fix/hot-reloader branch November 13, 2023 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Verified issues on the current code behavior or pull requests that will fix them

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants