Skip to content

Conversation

@tashik
Copy link

@tashik tashik commented Nov 13, 2025

Hello!

In raising this pull request, I confirm the following:

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I have updated the relevant CHANGELOG
  • I have created a PR for the documentation about this change

Small description of change:
Fixes merging numeric arrays keeping keys safe for \Memcached options constants

Thanks

Natalya

@niden niden merged commit 6cf172d into phalcon:5.0.x Nov 14, 2025
42 checks passed
@niden
Copy link
Member

niden commented Nov 14, 2025

@tashik Thank you. Good catch

@tashik
Copy link
Author

tashik commented Nov 14, 2025

@niden, OPT_CONNECT_TIMEOUT in Memcached is in milliseconds, I think to have the default 10 ms is not suitable for production with one memcached. Please, consider to set the default at least 50 ms

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.

2 participants