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

ini-values with equal sign doesn't work #676

Closed
2 of 5 tasks
stefanak-michal opened this issue Dec 14, 2022 · 3 comments
Closed
2 of 5 tasks

ini-values with equal sign doesn't work #676

stefanak-michal opened this issue Dec 14, 2022 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@stefanak-michal
Copy link

stefanak-michal commented Dec 14, 2022

Describe the bug
I want to add DSN alias for PDO into php.ini

ini-values: pdo.dsn.mybolt="bolt:host=localhost;port=7687;appname=pdo-bolt"

Version

  • I have checked releases, and the bug exists in the latest patch version of v1 or v2.
  • v2
  • v1

Runners

  • GitHub Hosted
  • Self Hosted

Operating systems
ubuntu-20.04

PHP versions
^8

Expected behavior
I expect to receive full string with this php code

$dsn = get_cfg_var('pdo.dsn.mybolt');

$dsn should contains bolt:host=localhost;port=7687;appname=pdo-bolt but currently this returns only bolt:host.

Are you willing to submit a PR?
no

@stefanak-michal stefanak-michal added the bug Something isn't working label Dec 14, 2022
@stefanak-michal
Copy link
Author

stefanak-michal commented Dec 14, 2022

https://github.com/stefanak-michal/pdo-bolt/actions/runs/3699091929/jobs/6266073032

==> Add php.ini values
✓ max_execution_time=0 Added to php.ini
✓ pdo.dsn.mybolt=bolt:host=localhost;port=7687;appname=pdo-bolt Added to php.ini
PHP:  syntax error, unexpected '=' in /etc/php/8.0/cli/conf.d/99-pecl.ini on line 3

@shivammathur
Copy link
Owner

Fixed in 3407e60. Will be in the next release.

@shivammathur shivammathur added the awaiting-release Added/Fixed and tested, awaiting release label Dec 15, 2022
@shivammathur
Copy link
Owner

Released 2.23.0 with the fix.

@shivammathur shivammathur removed the awaiting-release Added/Fixed and tested, awaiting release label Dec 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants