Nyholm
Repos
689
Followers
1146
Following
38

Writing test code should be simple

218
8

Symfony integration for Translations

311
84

A super lightweight PSR-7 implementation

1002
70

Symfony Framework Integration for HTTPlug

373
45

AWS SDK with readable code and async responses

383
104

Events

Nyholm create tag 1.0.1
Created at 1 day ago
Nyholm delete branch prepare-release
Created at 1 day ago

Prepare release 1.0.1 (#419)

  • Prepare release 1.0.1

  • Updated layers.json


Co-authored-by: Bref Bot github-bot@bref.sh

Created at 1 day ago
pull request closed
Prepare release 1.0.1

FYI @starred-gijs and @mnapoli

Created at 1 day ago
pull request opened
Prepare release 1.0.1

FYI @starred-gijs and @mnapoli

Created at 1 day ago
Nyholm create branch prepare-release
Created at 1 day ago
Added parallel extension

Thank you. I put this on hold until there is a decision about ZTS.

FYI, I think I suggested to remove it years ago because "nobody is using pthreads or parallel anyways".

Created at 1 day ago
add new layer Swoole

Thank you.

Can you please see to update this PR? Lot's of things have happened here in the past 2 weeks

Created at 1 day ago
add new layer bsdiff

Hey. Thank you

A lot of things have happened since this PR was opened. Can you rebase this and update your Dockerfile?

Created at 1 day ago
Nyholm delete branch blackfire-update
Created at 1 day ago

Update Blackfire extension (#415)

Created at 1 day ago
pull request closed
Update Blackfire extension

The Blackfire extension was updated to 1.86.5. Ping @nyholm, please release this.

Created at 1 day ago

Bum version newrelic (#417)

Created at 1 day ago
pull request closed
Bump newrelic version (compatibility PHP 82)

Just bump the version of new relic.

Created at 1 day ago
Bump newrelic version (compatibility PHP 82)

Hey. Sorry. My OSS game lately has been awful.

Im always happy to get help.

The process is basically:

  • Update the change log and push changes to prepare-release branch.
  • Make a PR. Let the CI job run
  • Merge and deploy
  • Then merge the extra PR that is created to bump version number.

I wrote something here a wile ago.

Created at 1 day ago
Nyholm create tag 1.2.0
Created at 1 week ago
Nyholm delete branch tag-update
Created at 1 week ago

Adding tag 1.0.0 (#414)

Created at 1 week ago
pull request closed
Update list of tags

Since we just tagged 1.0.0 we also need to update config.json

Created at 1 week ago
Update list of tags

Yepp. This is to make sure some automated script is versioned correctly

Created at 1 week ago

Fix blank line after if block in headers generation (#1382)

Fix blank line afweter if in Headers generation

Created at 1 week ago

Fix blank line after if block in headers generation (#1382)

Fix blank line afweter if in Headers generation

Created at 1 week ago
pull request closed
[Security] Make AccessDecisionManager much faster

| Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | | License | MIT | Doc PR |

When you are using a lot of voters, you need better caching. With inspiration from https://github.com/symfony/symfony/pull/43066 by @jderusse I managed to squeeze some more performance from the AccessDecisionManager.

I do about 7000 calls to AccessDecisionManager::getVoters(). It takes me 369ms. With this PR, it takes about 30ms.

The trick is that I don't loop over all Voters all the time. If I seen the object and attribute before then I just returns the voters I previously calculated.


Performance:

When do benchmark on an application with 40 voters I got:

| Metric | From | To | |------- |------|----| | Wall time | 1150ms | 859 ms | I/O Wait | 135 ms | 154 ms | CPU | 1020ms | 705 ms | Memory | 11.8 MB | 12.9 MB


I would love a few more set of eyes on this PR.

Created at 2 weeks ago