chrisguitarguy
Repos
136
Followers
119
Following
30

A simple PHP task queue.

129
11

Add request ID's to your Symfony application.

29
18

Demeanor is a spec and XUnit-style test harness for PHP.

6
0

Skeleton for how I like to build Symfony apps.

0
0

Events

Monolog 3 is not supported

Hello

To reproduce bug try following commands:

Create a new project composer create-project symfony/skeleton request-id-bundle-example Install monolog composer require monolog Install bundle composer require chrisguitarguy/request-id-bundle Configure this bundle (enable integration with Monolog).

On PHP 8.1 you should get an error, when you try to access /

Chrisguitarguy\RequestId\Monolog\RequestIdProcessor::__invoke(): Argument #1 ($record) must be of type array, Monolog\LogRecord given, called in /home/marcin/projekty/requestidbundle-test/vendor/monolog/monolog/src/Monolog/Logger.php on line 331

Monolog 3 requires PHP 8.1, but also has introduced some changes in API - UPGRADE instructions for Monolog 3

Log records have been converted from an array to a Monolog\LogRecord object with public (and mostly readonly) properties. e.g. instead of doing $record['context'] use $record->context. In formatters or handlers if you rather need an array to work with you can use $record->toArray() to get back a Monolog 1/2 style record array. This will contain the enum values instead of enum cases in the level and level_name keys to be more backwards compatible and use simpler data types.

We need to change types for Chrisguitarguy\RequestId\Monolog\RequestIdProcessor::__invoke to support Monolog 3 and previous versions. Because this bundle support PHP 7.4 we cannot use Union types. My idea is to remove types and check variable's type $record in method.

I will prepare PR soon (to end of this week).

Created at 2 weeks ago
Monolog 3 is not supported

In https://github.com/chrisguitarguy/RequestIdBundle/pull/27 !!

Created at 2 weeks ago
Add support for Monolog3

Tagged v5.1.0 with this too

Created at 2 weeks ago
chrisguitarguy create tag v5.1.0
Created at 2 weeks ago

Add support for Monolog3

In Monolog3 log records have been converted from an array to a Monolog\LogRecord object.

This commit fix error: Chrisguitarguy\RequestId\Monolog\RequestIdProcessor::__invoke(): Argument #1 ($record) must be of type array, Monolog\LogRecord given

Add symfony/flux to allow-plugins

Merge pull request #27 from morawskim/monolog3

Add support for Monolog3

Created at 2 weeks ago
pull request closed
Add support for Monolog3

This PR closes issue #26.

Monolog3 made some changes in API.

Log records have been converted from an array to a Monolog\LogRecord object with public (and mostly readonly) properties. e.g. instead of doing $record['context'] use $record->context. In formatters or handlers if you rather need an array to work with you can use $record->toArray() to get back a Monolog 1/2 style record array. This will contain the enum values instead of enum cases in the level and level_name keys to be more backwards compatible and use simpler data types.

This PR fix error:

Chrisguitarguy\RequestId\Monolog\RequestIdProcessor::__invoke(): Argument #1 ($record) must be of type array, Monolog\LogRecord given, called in /app/vendor/monolog/monolog/src/Monolog/Logger.php on line 331

Monolog3 requires PHP 8.1.

You can use my demo - https://github.com/morawskim/RequestIdBundle-test Or use instruction from issue #26.

Fetch a git repository git clone https://github.com/morawskim/RequestIdBundle-test.git Call docker-compose up -d Install dependencies - docker-compose exec php composer install

Open web browser and go to localhost:8080 You should see a error message:

Chrisguitarguy\RequestId\Monolog\RequestIdProcessor::__invoke(): Argument #1 ($record) must be of type array, Monolog\LogRecord given, called in /app/vendor/monolog/monolog/src/Monolog/Logger.php on line 331

Switch branch to fork. Install dependencies - docker-compose exec php composer install Refresh webpage. You should see a default Symfony application homepage.

Created at 2 weeks ago
Add support for Monolog3

I honestly just forgot about it 😬 I will merge it now

Created at 2 weeks ago
chrisguitarguy create tag v1.0.1
Created at 1 month ago

Bump the Version

Created at 1 month ago
delete branch
chrisguitarguy delete branch bump_deps
Created at 1 month ago

Bump to Current Dependency Versions

Fix Security Issues

Merge pull request #26 from AgencyPMG/bump_deps

Bump Dependencies & Fix Security Issues

Created at 1 month ago
pull request closed
Bump Dependencies & Fix Security Issues
Created at 1 month ago
pull request opened
Bump Dependencies & Fix Security Issues
Created at 1 month ago
create branch
chrisguitarguy create branch bump_deps
Created at 1 month ago
Front End Coding Challenge for Dan Kinsley

Thanks, but we can't merge this as we usually have folks submit their solutions to this privately so we can code review them and chat during interviews.

Created at 1 month ago
pull request closed
Front End Coding Challenge for Dan Kinsley

Front End Coding Challenge Complete!

Created at 1 month ago
pull request closed
AliceHuongNguyen finish coding challenge

Hello, I have finished the coding challenge.

Created at 1 month ago
AliceHuongNguyen finish coding challenge

Thanks, but we can't merge this as we usually have folks submit their solutions to this privately so we can code review them and chat during interviews.

Created at 1 month ago
pull request closed
Completed Assessment

I've completed the assessment.

Created at 1 month ago
Completed Assessment

Thanks, but we can't merge this as we usually have folks submit their solutions to this privately so we can code review them and chat during interviews.

Created at 1 month ago
pull request closed
Solve Frontend React Challenge

This implements all specs to complete the frontend React challenge. Redux Toolkit was added, so make sure to run npm install to update local dependencies before opening the application. This challenge took approximately 2 hours and 15 minutes to complete, including a couple small breaks.

Created at 1 month ago
Solve Frontend React Challenge

Thanks, but we can't merge this as we usually have folks submit their solutions to this privately so we can code review them and chat during interviews.

Created at 1 month ago
pull request closed
Corrected Error in CSV file

This is a CSV file, not a JavaScript file, so the escape character backslash is not needed before the inner quotes which is causing Github to not display the file in tabular format (beautiful and searchable as Github says).

Created at 1 month ago
Corrected Error in CSV file

Hi, thank's for this, but those are on purpose. The backslashes are there to break someones code that doesn't use proper CSV parsing. I realize that a \ isn't actually an escape character officially, but quite a few CSV dialects use it.

Created at 1 month ago
delete branch
chrisguitarguy delete branch pg_serial_defaults
Created at 1 month ago
CompactToken parsing failed with error code: 80049217

For what it's worth here: I found that different oauth scopes associated with your token can cause different token formats. My case it was an msads scope that caused a different format and caused the /me endpoint on the graph API to not work with this same errror: CompactToken parsing failed with error code: 80049217

Adding the open id/email/profile scopes were no help in fixing this.

Created at 2 months ago