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

xdebug + set_time_limit + Safe => Exception #443

Closed
josefsabl opened this issue Apr 24, 2024 · 2 comments
Closed

xdebug + set_time_limit + Safe => Exception #443

josefsabl opened this issue Apr 24, 2024 · 2 comments

Comments

@josefsabl
Copy link

Hello, this objectively is not a fault of Safe but I am wondering if this could/should be solved on Safe's side.

The problem is, that when xdebug extension is enabled. The set_time_limit returns false even if it was successful. Safe\set_time_limit then converts this false into exception that crashes the application.

It is annoying when debugging and I run into this several times a week.

Maybe this weird edge case could be handled and the false from this function could be ignored when xdebug is enabled.

Obviously there is a workaround to temporarily comment out the Safe/set_time_limit calls or not use them at all.

@theofidry
Copy link

Shouldn't it be rather reported to Xdebug as a bug?

@shish
Copy link
Collaborator

shish commented Feb 3, 2025

+1 for "sounds like an xdebug bug", though feel free to comment if fixing it in xdebug is impossible for some reason and we can try to find a workaround

@shish shish closed this as completed Feb 3, 2025
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

No branches or pull requests

3 participants