search

zircote / swagger-php

A php swagger annotation and parsing library
http://zircote.github.io/swagger-php/
Apache License 2.0
5.08k stars 934 forks source link

Allowed memory size of 1073741824 bytes exhausted (tried to allocate 534790144 bytes) #1109

Closed kriit24 closed 2 years ago

kriit24 commented 2 years ago

Its not possible to see where, but i got this error.

NB! solution is not do increase memory usage, but solution is do reduce memory usage in code.

The biggest problem is that no one is optimizing.

For now i look a another swagger builder.

For testing i set "ini_set('memory_limit', '-1');" unlimited memory, and then it works, but this is not a solution.

DerManoMann commented 2 years ago

Sure, reducing memory used is good and I'd be happy to look into that. If there is any way to create a reduced test case that would be awesome and help a lot. Also a stacktrace or any other information other than just'runs out of memory ' would be good.

kriit24 commented 2 years ago

One thing i found a error in vendor\zircote\swagger-php\src\Annotations\AbstractAnnotation.php line 422 Looks like if some error occurs in OpenApi definition then it crashes.

Maybe u but error logs with backtrace into array ?

DerManoMann commented 2 years ago

The exception stack would be a good start and perhaps the exact version of swagger-php and PHP itself? Cut/paste from a logfile would also be ok.

kriit24 commented 2 years ago

I accidentally repeated definition. I downloaded latest version of swagger two days a go. PHP version is 7.4

@OA\JsonContent(

@OA\Property(property="logistic_route_sheet_client_id", type="int", example=1, description="EMPLOYEE"),

@OA\Property(property="logistic_route_sheet_client_id", type="int", example=1, description="EMPLOYEE"),

)

DerManoMann commented 2 years ago

Strange - that alone is not enough to trigger the OOM. Adding that to one of the examples I just get this error

Unexpected log line: error("Multiple @OA\Property() with the same property="logistic_route_sheet_client_id":
...

Either way, sounds like this can be closed?

kriit24 commented 2 years ago

Yes.

On 09.02.2022 21:55, Martin Rademacher wrote:

Strange - that alone is not enough to trigger the OOM. Adding that to one of the examples I just get this error

Unexpected log line: error("Multiple @OA\Property() with the same property="logistic_route_sheet_client_id": ...

Either way, sounds like this can be closed?

— Reply to this email directly, view it on GitHub https://github.com/zircote/swagger-php/issues/1109#issuecomment-1034137960, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALLRPM4RNWIO4VRSAXBWKLU2LBC5ANCNFSM5NWUE7FA. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

-- Lugupidamisega, Kuno Vaik IT Project Partner OÜ +372 58 4444 84 www.projectpartner.ee